Skip to content

reports: limit regzbot tags and lore link to upstream trees#1987

Open
bhcopeland wants to merge 2 commits into
kernelci:mainfrom
bhcopeland:regzbot-upstream-gating
Open

reports: limit regzbot tags and lore link to upstream trees#1987
bhcopeland wants to merge 2 commits into
kernelci:mainfrom
bhcopeland:regzbot-upstream-gating

Conversation

@bhcopeland

Copy link
Copy Markdown
Member

Regression reports currently carry the #regzbot tags and a lore link to their own Message-ID regardless of the tree, but only mainline and next reports are CC'd to the regressions list. Reports for other trees (android, chromeos and so on) are out of regzbot's scope, and their lore link can point at a mail that never reached lore.

Gate the regzbot tags and the lore link on the same tree check that adds the regressions list CC, through a shared is_regzbot_tree() helper, so the tags only appear on reports that regzbot will actually see and the link only appears on mails guaranteed to be archived. Reports for other trees keep the Reported-by attribution and the

@alanpeixinho alanpeixinho left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested for different issues from different trees, seems ok.

@bhcopeland

Copy link
Copy Markdown
Member Author

@alanpeixinho I thought it would make sense to put stable in here. I've added it as an additional commit, since it isn't much to add.

KERNELCI_REPLYTO = "kernelci@lists.linux.dev"
REGRESSIONS_LIST = "regressions@lists.linux.dev"

REGZBOT_TREES = ("mainline", "next", "stable")

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit https://docs.astral.sh/ruff/rules/literal-membership/

Suggested change
REGZBOT_TREES = ("mainline", "next", "stable")
REGZBOT_TREES = {"mainline", "next", "stable"}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Amended and squashed; appreciated.

@tales-aparecida

Copy link
Copy Markdown

Tested for different issues from different trees, seems ok.

@alanpeixinho if you don't trust the test suite, please update it 🙏🏼

Regression reports currently carry the #regzbot tags and a lore link
to their own Message-ID regardless of the tree, but only mainline and
next reports are CC'd to the regressions list. Reports for other trees
(android, chromeos and so on) are out of regzbot's scope, and their
lore link can point at a mail that never reached lore.

Gate the regzbot tags and the lore link on the same tree check that
adds the regressions list CC, through a shared is_regzbot_tree()
helper, so the tags only appear on reports that regzbot will actually
see and the link only appears on mails guaranteed to be archived.
Reports for other trees keep the Reported-by attribution and the

Signed-off-by: Ben Copeland <ben.copeland@linaro.org>
Regzbot also tracks regressions in proper stable releases, so add the
stable tree to the set whose reports carry the regzbot tags, the lore
link and the regressions list CC.

Signed-off-by: Ben Copeland <ben.copeland@linaro.org>
@bhcopeland bhcopeland force-pushed the regzbot-upstream-gating branch from b49f047 to 9c32a47 Compare July 14, 2026 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants