tof-voters: Record name and email in analysis

With this we can make the data a little easier to consume.

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

tof-voters: Record name and email in analysis

With this we can make the data a little easier to consume.

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

show more ...

Fix two CI code formatting errors

The CI reports the following two errors:
* typing.Any unused in tof-voters/libvoters/subcmd/analyzer-commits.py
* the "function" keyword in prepare-emmc-qemu is not

Fix two CI code formatting errors

The CI reports the following two errors:
* typing.Any unused in tof-voters/libvoters/subcmd/analyzer-commits.py
* the "function" keyword in prepare-emmc-qemu is not needed

Fixing as suggested by the CI.

Tested: Ran the CI locally, the CI passes.

Signed-off-By: Sui Chen <suichen6@gmail.com>
Change-Id: Idbd3c47895c6ec6f923ede264264006ffc08e2f7

show more ...

treewide: lint and format

Run all the latest linters and formatters from openbmc-build-scripts.
These were all robotically generated except for fixes due to
markdownlint warnings.

Signed-off-by: Pa

treewide: lint and format

Run all the latest linters and formatters from openbmc-build-scripts.
These were all robotically generated except for fixes due to
markdownlint warnings.

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

show more ...

python: fix flake8 warnings and format with black

Most of the flake8 warnings in this repository were fairly trivial,
so fixed them. The "openbmctool" is 7000+ lines of pretty heavily
warned code,

python: fix flake8 warnings and format with black

Most of the flake8 warnings in this repository were fairly trivial,
so fixed them. The "openbmctool" is 7000+ lines of pretty heavily
warned code, so just disabling that one. Format everything with
black.

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

show more ...

Revert "Allow script to be used without a git alias"

This reverts commit 6b4b9df44144c1cc2072eb1847a9fb335d326710.

The change switches from using an SSH alias to using a direct
hostname / port argu

Revert "Allow script to be used without a git alias"

This reverts commit 6b4b9df44144c1cc2072eb1847a9fb335d326710.

The change switches from using an SSH alias to using a direct
hostname / port argument in the claims that it is better for most
users, but it is not.

- The original 'openbmc.gerrit' SSH alias has been documented
forever in openbmc/docs:CONTRIBUTING.md and it is expected that
everyone has it set up as it is part of our gerrit documentation.

- Almost no one runs their system with their Unix ID matching their
Github ID, which means the SSH also needs a username argument
in order to be useful for most people (which was not added by
this previous commit).

As we already document having some kind of SSH alias for interacting
with Gerrit, using an SSH alias is necessary for the majority of users
anyhow, and the SSH alias chosen by default is the project-wide
documented suggestion, I am reverting this back to the previous
behavior.

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

show more ...

Remove some errant semicolons from report script

Found by autopep8.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I79004eda034aa3549f954edc49092582b946fad5

Allow script to be used without a git alias

The script as it exists today relies on an ssh alias, which leads to
non-obvious error messages if it's not set up. This is a case where
99% of the time

Allow script to be used without a git alias

The script as it exists today relies on an ssh alias, which leads to
non-obvious error messages if it's not set up. This is a case where
99% of the time people will be using the defaults, so we should make the
defaults as helpful as we can.

Tested:
Ran the commands in the README and saw that they succeeded.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I754b31ad9dae47af20c7cd8f4bbb4951119d061d

show more ...

tof-voters: simplify sub-command loading

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

tof-voters: enable openpower repositories

Based on private discussion with other TOF members, enabling openpower
repositories but disable:
- 'openpower-libhei' chip data.
- 'openpower-hw-dia

tof-voters: enable openpower repositories

Based on private discussion with other TOF members, enabling openpower
repositories but disable:
- 'openpower-libhei' chip data.
- 'openpower-hw-diags' chip data.
- 'openpower-hw-diags' PEL code.

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

show more ...

tof-voters: add README

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

tof-voters: add report subcommand

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

tof-voters: add review analysis subcommand

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

tof-voters: add commit analysis subcommand

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

tof-voters: begin tool for voter membership

The TOF voting requires looking at Gerrit data to determine who is
eligible to vote. Start a tool that can be used to dump and analyze
this data.

First

tof-voters: begin tool for voter membership

The TOF voting requires looking at Gerrit data to determine who is
eligible to vote. Start a tool that can be used to dump and analyze
this data.

First sub-command added is a method to dump all Gerrit data after a
certain date (typically the cut-off for the previous election).

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

show more ...