Mass rebuild scripts should check Bodhi and not bump/build anything that failed gating #13428

Open
opened 2026-07-16 18:44:37 +00:00 by adamwill · 5 comments
Member

Describe the issue

Every cycle we do a mass rebuild, and every time I wind up hastily scrolling through the Bodhi list of gated builds while the rebuild is going on, or after it finishes, and trying to flag up cases where we're rebuilding something that was gated because it failed tests.

This is dumb and we shouldn't do it. What should happen is that the mass rebuild scripts should check Bodhi and omit any package if the most recent build of it failed gating. Or, I guess it could equally just check whether current dist-git state matches the most recently-tagged build, and omit the package if it doesn't. Whatever: we should not blindly rebuild and merge packages that have failed gating tests, that is a bad idea.

When do you need this? (YYYY/MM/DD)

Before the F46 mass rebuild, I guess.

When is this no longer needed or useful? (YYYY/MM/DD)

When we stop doing blind mass rebuilds?

If we cannot complete this, what is the impact? [Dependencies/Blocker]

We will continue to merge builds of known-broken states.

## Describe the issue Every cycle we do a mass rebuild, and every time I wind up hastily scrolling through the Bodhi list of gated builds while the rebuild is going on, or after it finishes, and trying to flag up cases where we're rebuilding something that was gated because it failed tests. This is dumb and we shouldn't do it. What should happen is that the mass rebuild scripts should check Bodhi and omit any package if the most recent build of it failed gating. Or, I guess it could equally just check whether current dist-git state matches the most recently-tagged build, and omit the package if it doesn't. Whatever: we should not blindly rebuild and merge packages that have failed gating tests, that is a bad idea. ## When do you need this? (YYYY/MM/DD) Before the F46 mass rebuild, I guess. ## When is this no longer needed or useful? (YYYY/MM/DD) When we stop doing blind mass rebuilds? ## If we cannot complete this, what is the impact? [Dependencies/Blocker] We will continue to merge builds of known-broken states.
Owner

Okay, I think these points to mass tagging script we use after the mass rebuild https://forge.fedoraproject.org/releng/tooling/src/branch/main/release-process/mass-rebuilds/mass_tag.py;

I'll try to brainstorm this idea on how we can achieve that.

Okay, I think these points to mass tagging script we use after the mass rebuild https://forge.fedoraproject.org/releng/tooling/src/branch/main/release-process/mass-rebuilds/mass_tag.py; I'll try to brainstorm this idea on how we can achieve that.
Owner

Ahh, read the ticket again, maybe after all we can tweak the rebuild script only.

clone pkg
  → latest tagged in desttag?
       no  → rebuild
       yes → HEAD matches tagged source (+ releng-only commits)?
              yes → rebuild
              no  → skip + report

WDYT @adamwill, I think adding bodhi check in the script will slow things down, also it would be bringing one failure mode as we add one more service for rebuild (what if bodhi spots responding, error, etc.).

Ahh, read the ticket again, maybe after all we can tweak the rebuild script only. ``` clone pkg → latest tagged in desttag? no → rebuild yes → HEAD matches tagged source (+ releng-only commits)? yes → rebuild no → skip + report ``` WDYT @adamwill, I think adding bodhi check in the script will slow things down, also it would be bringing one failure mode as we add one more service for rebuild (what if bodhi spots responding, error, etc.).
Author
Member

what does "latest tagged in desttag" mean?

what does "latest tagged in desttag" mean?
Owner
here: https://forge.fedoraproject.org/releng/tooling/src/branch/main/release-process/mass-rebuilds/mass_rebuilds_info.py#L129
Owner

A starting point to discuss this issue: releng/tooling#13051 (Update: We need more discussion to happen here, moving it to next sprint. The PR is here but again the work is not done unless the PR is merged.)

A starting point to discuss this issue: https://forge.fedoraproject.org/releng/tooling/pulls/13051 (Update: We need more discussion to happen here, moving it to next sprint. The PR is here but again the work is not done unless the PR is merged.)
Sign in to join this conversation.
No milestone
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
releng/tickets#13428
No description provided.