Make no-unsafe-values a warning

Commit 049555aab9b414251f06853d9fa1fb95fb100bba just moved these checks
to use eslint/json as our json formatter. These unintentionally enabled
some checks that fail

Make no-unsafe-values a warning

Commit 049555aab9b414251f06853d9fa1fb95fb100bba just moved these checks
to use eslint/json as our json formatter. These unintentionally enabled
some checks that fail our code.

The json/no-unsafe-values check appears to verify that javascript can
successfully parse an integer value. The majority of openbmc code uses
nlohmann, which internally has a uint64 representation, which means
those values aren't a concern. This is currently breaking libcper CI.
In libcper, 64 bit registers are routinely represented as a json uint,
so there's many failures on the examples.

Demote this check to a warning for now to unbreak CI.

Change-Id: I81a0c402dd641e92171d33229da97df083e1bb09
Signed-off-by: Ed Tanous <etanous@nvidia.com>

show more ...

node: install from nodesource

Ubuntu is on node-20, but the latest version of eslint requires a
newer version. Use nodesource apt archive to install node-24, which
is the current LTS.

Signed-off-b

node: install from nodesource

Ubuntu is on node-20, but the latest version of eslint requires a
newer version. Use nodesource apt archive to install node-24, which
is the current LTS.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I7d5ca691ced3bd397f3a567dfd333f38f99c8dce

show more ...

eslint: switch to eslint/json

eslint-plugin-json no longer works with eslint 10.x series. Switch
to the native eslint/json plugin.

Due to OpenBMC allowing comments in JSON files, use the JSON5 lan

eslint: switch to eslint/json

eslint-plugin-json no longer works with eslint 10.x series. Switch
to the native eslint/json plugin.

Due to OpenBMC allowing comments in JSON files, use the JSON5 language
type for all JSON files. Without this, eslint/json fails to parse
many files in the OpenBMC org (due to unexpected comments).

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ib616f5c678ddfa0784578b2908bc221f825b3f56

show more ...

build-unit-test-docker: update meson to 1.10.1

Update meson to the latest version, which improves formatting
and linting. Generally we update to the latest version that is
pointed to by openembedde

build-unit-test-docker: update meson to 1.10.1

Update meson to the latest version, which improves formatting
and linting. Generally we update to the latest version that is
pointed to by openembedded anyhow (which is already updated) but
openbmc is currently behind at syncing openembedded.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ia08ecc8050b52b6f0a69a1e416d5e76cf850976e

show more ...

userid-validation: add axiado

CCLA accepted and uploaded to google drive

Change-Id: I3110ad22d845e6ee96d50cede7d44f0ea0022c17
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>

black: format for 26.1.0

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I958ca4b9bb58686e195d3df293c42e2a69c73d30

userid-validation: add xslab

CCLA accepted and uploaded to google drive

Change-Id: Ib96049cb136a8e9fc9267d814999c71af226031c
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>

scripts: unit-test: remove obsolete memory sanitizer TODO

Remove commented-out memory sanitizer code that is not used.
Memory sanitization is not currently supported in the build pipeline
and the co

scripts: unit-test: remove obsolete memory sanitizer TODO

Remove commented-out memory sanitizer code that is not used.
Memory sanitization is not currently supported in the build pipeline
and the commented code serves no purpose. Clean up to reduce clutter.

Change-Id: I9d107e04d78bdbb831331213983f79815d8361af
Signed-off-by: Arun R <rarun@ami.com>

show more ...

stdexec: update to latest

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ia9cb50463d65108334a92457fa7cfefa45fde1e5

scripts: unit-test: use specific exception type

Replace broad 'except Exception' with 'except GitCommandError'
for Git clone operations. This provides better error handling
by catching only the expe

scripts: unit-test: use specific exception type

Replace broad 'except Exception' with 'except GitCommandError'
for Git clone operations. This provides better error handling
by catching only the expected exception type from GitPython.

Change-Id: I8615a8389f02e598d93d07e4dc314cd6a51d5c02
Signed-off-by: Arun R <rarun@ami.com>

show more ...

scripts: unit-test: fix TODO comment formatting

Change TODO - to TODO: for consistency with PEP 350 conventions

Change-Id: I6175dd3c5f90281078f048d20301aa0de4a8002a
Signed-off-by: Arun R <rarun@ami

scripts: unit-test: fix TODO comment formatting

Change TODO - to TODO: for consistency with PEP 350 conventions

Change-Id: I6175dd3c5f90281078f048d20301aa0de4a8002a
Signed-off-by: Arun R <rarun@ami.com>

show more ...

build-unit-test-docker: add NVIDIA/stdexec

sdbusplus will soon use a non-vendored copy of stdexec, so add
it to the Docker container.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id:

build-unit-test-docker: add NVIDIA/stdexec

sdbusplus will soon use a non-vendored copy of stdexec, so add
it to the Docker container.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I7aa38d1ef47abfd9860891e9cad9b6b714b366df

show more ...

build-unit-test-docker: set NODE_PATH

nodejs programs, such as eslint, need NODE_PATH set to the location
where `npm` installed modules in order that it can find secondary
modules.

The new (eslint-

build-unit-test-docker: set NODE_PATH

nodejs programs, such as eslint, need NODE_PATH set to the location
where `npm` installed modules in order that it can find secondary
modules.

The new (eslint-9.x) config files are essentially JS files that load
modules and plugins. In order for these to work, the nodejs `Module`
module needs to have the `npm root` in the `Modules.path`, which
requires it to be in the NODE_PATH.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I8483d4b7e063ca5f2246be9a9ffff4bd7b70b4a6

show more ...

eslint: update from 8.x series

Upgrading from eslint 8.x to 9.x required some argument changes and
updating to a new config file format. I've adjusted accordingly and
tested out on a few repositori

eslint: update from 8.x series

Upgrading from eslint 8.x to 9.x required some argument changes and
updating to a new config file format. I've adjusted accordingly and
tested out on a few repositories that had .eslintignore files.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I9d409b7b7ea4a77047d60a40da86a5d4baf7419c

show more ...

build-seed: add clemente

Clemente is a Meta platform using the GB300 hardware from Nvidia. It
is similar in design to Catalina (GB200) but the next generation.

Signed-off-by: Patrick Williams <pat

build-seed: add clemente

Clemente is a Meta platform using the GB300 hardware from Nvidia. It
is similar in design to Catalina (GB200) but the next generation.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I4dc014c7346297e10cc6f6cb0ef9862495f26889

show more ...

gitlint: check for bad authors

Similar to how we expect Signed-off-by to be legal names matching
something from a CLA, it is helpful if the Author is formatted
similarly. Often, a new user especial

gitlint: check for bad authors

Similar to how we expect Signed-off-by to be legal names matching
something from a CLA, it is helpful if the Author is formatted
similarly. Often, a new user especially, we fix their git-config and
the Signed-off-by but leave the author as a username-like. This leads
to the SRCREV bump script creating a commit message referencing a
[likely non-public] username instead of the proper name of the author.

Tested:
```
Running commit_gitlint (0.19.1)
-: UC4 Author user has too few words; likely user id instead of legal name?: "joe-bob"
```

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I6a3da5b27cd57de5758ba9277ae5d8b2e0570a5e

show more ...

Prepare for building meta machines on jenkins

We are preparing to build anacapa, bletchley15, santabarbara, ventura2,
and yosemite5 on jenkins. This ensures that when those new jobs start
running on

Prepare for building meta machines on jenkins

We are preparing to build anacapa, bletchley15, santabarbara, ventura2,
and yosemite5 on jenkins. This ensures that when those new jobs start
running on the jenkins workers, they will be fast since the sstate
will have data for each of the new machines.

Signed-off-by: James Athappilly <jamesatha@gmail.com>
Depends-on: Iae149f1da8822f48783199b51858a07fdaa476e2
Change-Id: Id6bb685014ac7d4e4978f11717462e216b10c7f2

show more ...

Invalidate docker cache to fix clang upgrade error

This should address build errors from running
```
/bin/sh -c apt-get update && apt-get dist-upgrade -yy
```

that resulted in the following error:

Invalidate docker cache to fix clang upgrade error

This should address build errors from running
```
/bin/sh -c apt-get update && apt-get dist-upgrade -yy
```

that resulted in the following error:
```
dpkg: error processing archive /tmp/...60_arm64.deb (--unpack):
trying to overwrite '/usr/lib/llvm-21/bin/clang-scan-deps'
```

This forces docker to actually do an apt-get update/upgrade instead of
using the cached image on the host

Change-Id: I99aa148a9af51a3375ba813e93621b7dbd302680
Signed-off-by: James Athappilly <jamesatha@gmail.com>

show more ...

scripts: unit-test: Reduce NOFILE rlimit for valgrind

Avoid triggering its file descriptor limit test and failure exit:

```
--27:0:libcfile Valgrind: FATAL: Private file creation failed.
The cur

scripts: unit-test: Reduce NOFILE rlimit for valgrind

Avoid triggering its file descriptor limit test and failure exit:

```
--27:0:libcfile Valgrind: FATAL: Private file creation failed.
The current file descriptor limit is 1073741804.
If you are running in Docker please consider
lowering this limit with the shell built-in limit command.
--27:0:libcfile Exiting now.
```

Change-Id: I94b051c9843209fb0986575622735691f0320a13
Signed-off-by: Andrew Jeffery <andrew@codeconstruct.com.au>

show more ...

Fix valgrind safety check

Cleanup the code to not catch all errors, only errors when the valgrind
run is done. If the build, or anything else fails, that should be
promoted to a real exception. Pr

Fix valgrind safety check

Cleanup the code to not catch all errors, only errors when the valgrind
run is done. If the build, or anything else fails, that should be
promoted to a real exception. Previously, any error in the c file would
silently be treated as "valgrind not supported".

While we're here, clean up some other things. Use TemporaryDirectory as
a context rather than doing manual cleanup. Use the python DEVNULL
rather than constructing our own null pipe.

Change-Id: I0749b10cb3de579db007927fb0e3a58e815b5e22
Signed-off-by: Ed Tanous <etanous@nvidia.com>

show more ...

beautysh: fix to v6.2.1

The latest version of beautysh has a bug[1] that breaks formatting.
Fix to an older version until this is fixed.

[1]: https://github.com/lovesegfault/beautysh/issues/265

Si

beautysh: fix to v6.2.1

The latest version of beautysh has a bug[1] that breaks formatting.
Fix to an older version until this is fixed.

[1]: https://github.com/lovesegfault/beautysh/issues/265

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I6371098f68a2f61c0a901a1b527ac93d0e043044

show more ...

build-unit-test-docker: use ubuntu-toolchain-r for gcc

Ubuntu plucky uses GCC-15.0, which has a few known bugs. Switch
to picking up GCC from the ubuntu-toolchain-r repository, which
currently has

build-unit-test-docker: use ubuntu-toolchain-r for gcc

Ubuntu plucky uses GCC-15.0, which has a few known bugs. Switch
to picking up GCC from the ubuntu-toolchain-r repository, which
currently has GCC-15.2.

One specific issue the community has encountered included with
this fixes is reported by Google Test[1].

[1]: https://github.com/google/googletest/issues/4759

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Id7f9cfd9c6292334b79c6c417c11210910876f06

show more ...

build-unit-test-docker: utilize https for llvm install

Some of our internal networks have started to completely block http.
This llvm.sh script utilizes a http address by default for some wget
comma

build-unit-test-docker: utilize https for llvm install

Some of our internal networks have started to completely block http.
This llvm.sh script utilizes a http address by default for some wget
commands. An issue[1] has been opened on the llvm project but for now
just utilize a workaround to tell the script to use https.

Tested:
- Confirmed script now correctly builds in environment without http
support

[1]: https://github.com/llvm/llvm-project/issues/163216

Change-Id: Iadcb05e94480c3a91dc65c33e41f274ac545fd99
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>

show more ...

qemu-build.sh: Allow WORKSPACE to be anywhere while building

When getting latest-qemu-aarch64 working, I encountered an error
saying the script which should have been mounted wasn't there [1].
Upon

qemu-build.sh: Allow WORKSPACE to be anywhere while building

When getting latest-qemu-aarch64 working, I encountered an error
saying the script which should have been mounted wasn't there [1].
Upon investigating, I realized that this works only if WORKSPACE
is a child of HOME which does get bind mounted. When building on
the meta machines, the main writable mount is not a users home
directory.

This change makes it so that we just bind WORKSPACE
directly and it simplifies things by remapping the docker root
user to the caller's UID/GID so permissions line up.

I was able to manually run this on the meta graviton host and
verified that it made the build artifacts with the correct
username and group.

[1] https://jenkins.openbmc.org/job/latest-qemu-aarch64/1/console

Change-Id: Ief4fb34c3f160767122c0129c447114a9e86692a
Signed-off-by: James Athappilly <jamesatha@gmail.com>

show more ...

userid-validation: add scaleway

CLA update has been processed and there is an active user in gerrit so
add to approve list

Change-Id: I5d05624bf6e623d30c6e6a52ce7c01051f9c5cd4
Signed-off-by: Andrew

userid-validation: add scaleway

CLA update has been processed and there is an active user in gerrit so
add to approve list

Change-Id: I5d05624bf6e623d30c6e6a52ce7c01051f9c5cd4
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>

show more ...