tools: owners: handle matchers

Enhance the owners tool to parse the 'matchers' directives in order
to add additional owners and reviewers.

Tested:

* Created a bogus commit in 'docs' to detect GPIO

tools: owners: handle matchers

Enhance the owners tool to parse the 'matchers' directives in order
to add additional owners and reviewers.

Tested:

* Created a bogus commit in 'docs' to detect GPIO and MCTP matchers.
* Created a bogus commit in 'smbios-mdr' to detect change to
speed_select.hpp.

```
$ tools/owners --path ../docs reviewers && echo "..." && tools/owners --path ../smbios-mdr reviewers
{"reviewer": "bradleyb@fuzziesquirrel.com", "state": "REVIEWER"}
{"reviewer": "patrick@stwcx.xyz", "state": "REVIEWER"}
{"reviewer": "andrew@aj.id.au", "state": "CC"}
{"reviewer": "anoo@us.ibm.com", "state": "CC"}
{"reviewer": "gmills@linux.vnet.ibm.com", "state": "CC"}
{"reviewer": "jk@ozlabs.org", "state": "CC"}
{"reviewer": "joel@jms.id.au", "state": "CC"}
...
{"reviewer": "jonathan.doman@intel.com", "state": "REVIEWER"}
{"reviewer": "kuiying.wang@intel.com", "state": "REVIEWER"}
{"reviewer": "yugang.chen@linux.intel.com", "state": "REVIEWER"}
{"reviewer": "zhikui.ren@intel.com", "state": "REVIEWER"}
```

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

show more ...

Upgrade clang-tidy version to 14.0.0

Fix a clang issue appearing in clang-tidy-13.0.1
Details refer to: https://reviews.llvm.org/rG1f2d40c47f5f8fd01d91d73a1f52044fe1c83225
[clang-tidy] fix duplicate

Upgrade clang-tidy version to 14.0.0

Fix a clang issue appearing in clang-tidy-13.0.1
Details refer to: https://reviews.llvm.org/rG1f2d40c47f5f8fd01d91d73a1f52044fe1c83225
[clang-tidy] fix duplicate '{}' in cppcoreguidelines-pro-type-member-init

Issue: The overload of the constructor will repeatedly fix the member
variables that need to be initialized, Removed the duplicate '{}'.

In current ubuntu:jammy, clang version 14.0.0 is available, so we can use
clang-tidy 14.0.0 to fix the clang issue.

Change-Id: I312888b13ed0ed39e0edf502eca46b11b9e9a62f
Signed-off-by: Jian Zhang <zhangjian.3032@bytedance.com>

show more ...

run-unit-test-docker: add comment for TEST_ONLY

The NO_FORMAT_CODE option is in the comment block. It also makes sense
to add the TEST_ONLY option to that comment block.

Tested: tested locally thou

run-unit-test-docker: add comment for TEST_ONLY

The NO_FORMAT_CODE option is in the comment block. It also makes sense
to add the TEST_ONLY option to that comment block.

Tested: tested locally though it only changes a comment.

Signed-off-by: Nan Zhou <nanzhoumails@gmail.com>
Change-Id: I49f48e813c633bd89b9c82df128e84e3c59c8993

show more ...

tools: owners: initial parsing of OWNERS file

Tested:

```
$ tools/owners reviewers
{"reviewer": "geissonator@yahoo.com", "state": "REVIEWER"}
{"reviewer": "joel@jms.id.au", "state": "REVIEWER"}
{"r

tools: owners: initial parsing of OWNERS file

Tested:

```
$ tools/owners reviewers
{"reviewer": "geissonator@yahoo.com", "state": "REVIEWER"}
{"reviewer": "joel@jms.id.au", "state": "REVIEWER"}
{"reviewer": "patrick@stwcx.xyz", "state": "REVIEWER"}
{"reviewer": "andrew@aj.id.au", "state": "CC"}

$ tools/owners reviewers --push-args
r=geissonator@yahoo.com,r=joel@jms.id.au,r=patrick@stwcx.xyz,cc=andrew@aj.id.au
```

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

show more ...

run-unit-test-docker.sh: add INTERACTIVE option

This allows running an interactive bash shell instead of the default
unit testing script.

Tested: Ran with and without "INTERACTIVE=true"

Signed-off

run-unit-test-docker.sh: add INTERACTIVE option

This allows running an interactive bash shell instead of the default
unit testing script.

Tested: Ran with and without "INTERACTIVE=true"

Signed-off-by: Benjamin Fair <benjaminfair@google.com>
Change-Id: Ifaee7d0dd705466ce70ad49aae48ac9a4a7ab522

show more ...

userid-validation: move to python3

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

move all build scripts to gerrit.openbmc.org

OpenBMC has transitioned the gerrit server to the openbmc.org domain

Change-Id: Ia65d5f968a4ffcf3bc65536148e90e42725c86aa
Signed-off-by: Andrew Geissler

move all build scripts to gerrit.openbmc.org

OpenBMC has transitioned the gerrit server to the openbmc.org domain

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

show more ...

unit-test.py: Format with autopep8

Signed-off-by: Lei YU <yulei.sh@bytedance.com>
Change-Id: If2af9b695ec6325aebf7fafb9f5ef17e43a52c40

run-ci: remove --network=host

There shouldn't be any reason to expose the Docker container
directly on the host network. The default in Docker is 'bridge'.

This was originally introduced in a comm

run-ci: remove --network=host

There shouldn't be any reason to expose the Docker container
directly on the host network. The default in Docker is 'bridge'.

This was originally introduced in a commit that added CMake support
back in 2017 but with no explanation. Removing it now...

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

show more ...

build-unit-test-docker: Don't generate markdown for PDI

Not used in the unit test container.

Change-Id: I19c053bf73cd08b19f30c855191f1f087f8bca1c
Signed-off-by: William A. Kennington III <wak@googl

build-unit-test-docker: Don't generate markdown for PDI

Not used in the unit test container.

Change-Id: I19c053bf73cd08b19f30c855191f1f087f8bca1c
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

scripts/unit-test: Leverage meson log output

Instead of trying to find the logs ourselves, just leverage meson to
print the logs directly.

Change-Id: Iab8b54936efb6c5230e538a06e045f9f23733017
Signe

scripts/unit-test: Leverage meson log output

Instead of trying to find the logs ourselves, just leverage meson to
print the logs directly.

Change-Id: Iab8b54936efb6c5230e538a06e045f9f23733017
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

build-setup.sh: remove python

Started seeing this with recent fresh builds:

16:24:36 Package python is not available, but is referred to by another package.
16:24:36 This may mean that the package

build-setup.sh: remove python

Started seeing this with recent fresh builds:

16:24:36 Package python is not available, but is referred to by another package.
16:24:36 This may mean that the package is missing, has been obsoleted, or
16:24:36 is only available from another source
16:24:36 However the following packages replace it:
16:24:36 python2-minimal python2 dh-python 2to3 python-is-python3

Remove python from our build script. This apparently brought in the file
package for free so explicitly include that now.

Tested:
- Verified build-setup.sh can now build the docker image and start a
bitbake on a system that was not working previously

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

show more ...

test-qemu: Account for new Jenkins constraints and configuration

Recent (?) Jenkins releases have disabled traversal of symlinks, which
breaks the mechanism the script was using to avoid constructin

test-qemu: Account for new Jenkins constraints and configuration

Recent (?) Jenkins releases have disabled traversal of symlinks, which
breaks the mechanism the script was using to avoid constructing file
paths containing date- and time- stamps. Instead, switch to downloading
and extracting the artifact zip for each platform locally, and glob our
way through the paths.

Further, the Jenkins configuration has been rearranged in the
intervening time between when I first wrote this script and now. Update
the script to account for the new job names and platforms.

On the subject of platforms, witherspoon-tacoma was replaced in CI with
rainier[1], which was subsequently renamed to p10bmc[2].

Finally, regarding the AST2600 and p10bmc, openbmc/qemu master contains
a patch that expects a customised eMMC layout that can describe boot
partitions as well as the user partition. Integrate support for
generating an eMMC image in this manner and drop the use of
-kernel/-dtb/-initrd from the qemu commandline, as the eMMC image must
be formatted in this manner regardless.

[1] https://lore.kernel.org/openbmc/3309719B-0B3F-44AC-BF30-7E7745A4BA67@gmail.com/
[2] https://gerrit.openbmc-project.xyz/c/openbmc/openbmc/+/41799/2

Change-Id: Ie2668420ab9c572d173116d587cc7ac861051ad2
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

show more ...

test-qemu: Extend timeouts to 300 seconds (5 minutes)

Be a bit more tolerant of slow boot processes.

Change-Id: I2c1afd48768c33e049f14ac65645364aedc77e1d
Signed-off-by: Andrew Jeffery <andrew@aj.id

test-qemu: Extend timeouts to 300 seconds (5 minutes)

Be a bit more tolerant of slow boot processes.

Change-Id: I2c1afd48768c33e049f14ac65645364aedc77e1d
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

show more ...

test-qemu: Re-join lines of broken ssh invocation

Change-Id: I49ff248e57d48b79e11811ef60413f632c2e02e0
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

bitbake: continue on failure

Add '-k' option to bitbake invocations to get Jenkins builds to continue
after falure as much as possible. This gives us a greater view of
everything that is broken rat

bitbake: continue on failure

Add '-k' option to bitbake invocations to get Jenkins builds to continue
after falure as much as possible. This gives us a greater view of
everything that is broken rather than just the first bitbake task which
encountered a problem, hopefully allowing more problems to be fixed at
once.

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

show more ...

build-unit-test-docker: update to ubuntu-jammy

Jammy was released on April 21st, 2022. Major changes
impacting our Docker container were:

- native support of OpenSSL3.
- python3.10
- libu

build-unit-test-docker: update to ubuntu-jammy

Jammy was released on April 21st, 2022. Major changes
impacting our Docker container were:

- native support of OpenSSL3.
- python3.10
- liburing2

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

show more ...

build-unit-test-docker: remove span-lite

There are no uses of span-lite within the openbmc org anymore and
everyone should be using `std::span` now. Remove it from the Docker
container.

Signed-off

build-unit-test-docker: remove span-lite

There are no uses of span-lite within the openbmc org anymore and
everyone should be using `std::span` now. Remove it from the Docker
container.

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

show more ...

build-unit-test-docker: remove fifo_map

No code and no recipes currently depend on fifo_map. Remove it from
our Docker unit-test container.

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

build-unit-test-docker: remove fifo_map

No code and no recipes currently depend on fifo_map. Remove it from
our Docker unit-test container.

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

show more ...

meson: update to 0.61.3

This matches the version used in Yocto now.

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

phosphor-logging: fix yaml option

The meson option in the repository is 'yamldir' and not 'yaml_dir'.
Fix this since later versions of Meson complain about an unknown
option being passed.

Signed-of

phosphor-logging: fix yaml option

The meson option in the repository is 'yamldir' and not 'yaml_dir'.
Fix this since later versions of Meson complain about an unknown
option being passed.

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

show more ...

autoconf-archive: manually patch C++20 support

autoconf-archive added support for C++20 finally in v2022.02.11
but none of the Ubuntu distro versions have picked this up
yet (checked impish and jamm

autoconf-archive: manually patch C++20 support

autoconf-archive added support for C++20 finally in v2022.02.11
but none of the Ubuntu distro versions have picked this up
yet (checked impish and jammy). Manually patch the file so that
we can upgrade our autoconf-based C++ repositories to use C++20.

The v2022.02.11 version is already included in Yocto.

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

show more ...

userid-validation: add quic

CLA is accepted. Add quic.

Signed-off-by: Jae Hyun Yoo <quic_jaehyoo@quicinc.com>
Change-Id: Ie54f9e8553e203679c0edccecb79057f9552ff22

unit-test: Update gtest source rev to fix std::span issue

The std::span does not support some gtest MACROS and is fixed in
https://github.com/google/googletest/pull/3195 and
https://github.com/googl

unit-test: Update gtest source rev to fix std::span issue

The std::span does not support some gtest MACROS and is fixed in
https://github.com/google/googletest/pull/3195 and
https://github.com/google/googletest/commit/e4717df71a4f45bf9f0ac88c6cd9846a0bc248dd

Including the fixes to support std::span in gtests.

Match the openembedded gtest version in
https://github.com/openembedded/meta-openembedded/pull/564

Change-Id: Ia0171160ee7a0a36c2fb27eb441e25cdec9ba9f1
Signed-off-by: Willy Tu <wltu@google.com>

show more ...

jenkins: README.md: Map jobs to scripts

Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
Change-Id: Ibd97e8d71776ac6b6df65af683c3f05d901fdb1b