Fixes #9841 - adds a release_type field to .composeinfo so we can tell Beta and Final releases apart. #13029
No reviewers
releng/members
releng/Owners
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
dev
docs
easyfix
epel
f26
f27
f28
f29
f30
f31
f32
f33
f34
f35
f36
f37
f38
f39
f40
f41
f42
f43
f44
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
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
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
3 participants
Notifications
Due date
No due date set.
Blocks
Reference
releng/tooling!13029
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "jnsamyak/tooling:update_composeinfo_file"
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?
Problem
The
.composeinfofile never included release type. PXE boot menus in RH Beaker labs couldn't tell Beta from GOLD for the same version - both showed up as "fedora-33". Beta entries stuck around after GOLD shipped, which confused the menu.Solution
So, as discussed in the ticket, I modified the script to add one more variant in the product section:
Modified
build_composeinfo.py:extract_release_type_from_path()to parse release type from directory paths("42_Beta" → "Beta", "42_RC" → "Final")
--release-typeparameter for explicit controlrelease_typeto the[product]sectionModified
stage_release.sh:RELEASEVER(Beta → Beta, RC/Final/no suffix → Final)Related Issue: releng/tickets#9841
Seems all pretty plausable, I haven't tested it though...
And you seem to have conflicts... can you get those sorted out?
Looking at RHEL composes, this doesn't match. Here's a sample:
so the name should be
typeand the value for a final release should bega.I see quite a lot of other discrepancies, e.g. we're entirely missing the
composesection:Possibly we need to compare notes with however this generation is done for RHEL composes and sync things up?
Ping me if you need a link to the RHEL compose server so you can cross-reference.
Okay so added builds/compose/build_composeinfo_wrapper.py as a follow-up to the release_type work.
When composeinfo.json is present (typical Pungi compose tree), the wrapper uses compose-utils to write the legacy .composeinfo INI format and adds release_type under [product]. If there’s no JSON e.g. staged mirror trees it falls back to the existing treeinfo logic in build_composeinfo.py.
stage_release.sh is unchanged for now; planning to switch that in a follow-up once this looks good.
Okay, and now the compose file looks like this: releng/tickets#9841 (comment)
e99bc8f08ato9d5d0cfdb09d5d0cfdb0to2cc78d2395As discussed in the releng call yesterday, we will merge this. And at the time of beta release, we will check for discrepancies and check with the beaker folks to see if this is as expected.