Mass rebuild scripts should check Bodhi and not bump/build anything that failed gating #13428
Labels
No labels
after freeze
automation
backlog
blocked
change-ack
change-nak
change-noreleng
changes
Closed As
Can't Fix
Closed As
Duplicate
Closed As
Fixed
Closed As
Fixed with Explanation
Closed As
Get back later
Closed As
Grooming
Closed As
Insufficient data
Closed As
Invalid
Closed As
It's all good
Closed As
taiga
Closed As
upstream
day-to-day
dev
docs
easyfix
epel
f26
f27
f28
f29
f30
f31
f32
f33
f34
f35
f36
f37
f38
f39
f40
f41
f42
f43
f44
f45
fedora
groomed
high-gain
high-trouble
in-progress
in-review
investigation
legal
low-gain
low-trouble
mass rebuild
medium-gain
medium-trouble
meeting
mini-initiative
new_artifact
ops
pdc_retirement
rawhide
RCA
review
script
sidetarget
sprint-0
sprint-1
sprint-2
sprint-3
sprint-4
sprint-5
unfrozen
waiting on external
Backlog Status
Needs Review
Backlog Status
Ready
chore
documentation
points
01
points
02
points
03
points
05
points
08
points
13
Priority
High
Priority
Low
Priority
Medium
release-process
Sprint Status
Blocked
Sprint Status
Done
Sprint Status
In Progress
Sprint Status
Review
Sprint Status
To Do
Technical Debt
Work Item
Bug
Work Item
Epic
Work Item
Spike
Work Item
Task
Work Item
User Story
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
releng/tickets#13428
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
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.
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.
Ahh, read the ticket again, maybe after all we can tweak the rebuild script only.
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.).
what does "latest tagged in desttag" mean?
here: https://forge.fedoraproject.org/releng/tooling/src/branch/main/release-process/mass-rebuilds/mass_rebuilds_info.py#L129
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.)