Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blueprints: Fix snapshots + latest release #2058

Merged

Conversation

regexowl
Copy link
Collaborator

@regexowl regexowl commented May 6, 2024

Previously created blueprints didn't take into account what the current latest RHEL release is and built the specific minor release that was included in the blueprint. This should fix the bug and also fix snapshots.

@regexowl regexowl force-pushed the blueprint-with-latest-release-version branch from f705ac7 to 0a51537 Compare May 6, 2024 13:21
@regexowl
Copy link
Collaborator Author

regexowl commented May 6, 2024

/retest

croissanne
croissanne previously approved these changes May 6, 2024
@lucasgarfield lucasgarfield force-pushed the blueprint-with-latest-release-version branch from 0a51537 to b9c617b Compare May 7, 2024 16:20
@lucasgarfield
Copy link
Collaborator

This might be a bit confusing a year from now when we forget about why we needed this. Would extracting it to a well named function with an explanatory comment help future us?

@regexowl
Copy link
Collaborator Author

regexowl commented May 8, 2024

/retest

@regexowl
Copy link
Collaborator Author

/retest

Previously created blueprints didn't take into account what the current latest RHEL release is and built the specific minor release that was included in the blueprint. This should fix the bug and also fix snapshots.
@regexowl regexowl force-pushed the blueprint-with-latest-release-version branch from 19e1e2f to 7cd3e8b Compare May 21, 2024 09:25
@regexowl
Copy link
Collaborator Author

#2060 is merged so I've updated the comments to reflect the currently used major version. This PR is ready for review again.

@ezr-ondrej
Copy link
Contributor

@croissanne could you merge this one? :)

@croissanne croissanne enabled auto-merge (rebase) May 21, 2024 11:55
@croissanne croissanne merged commit 38fe6d5 into osbuild:main May 21, 2024
5 checks passed
@regexowl regexowl deleted the blueprint-with-latest-release-version branch May 21, 2024 11:56
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.

None yet

4 participants