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.

Sign up for GitHub

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

Jump to bottom

Fix manifest parsing #115

Closed
wants to merge 1 commit into from
Closed

Fix manifest parsing #115

wants to merge 1 commit into from

Conversation

tkoscieln
Copy link
Contributor

No description provided.

@tkoscieln tkoscieln force-pushed the fix_manifest_parsing branch 3 times, most recently from 4f526a4 to ecb8769 Compare February 27, 2024 15:18
@ochosi
Copy link
Contributor

ochosi commented Mar 7, 2024

Could you please add some context to the PR description on the specific problem/bug this PR is trying to solve? (Ideally including instructions on how to reproduce the problem)

@tkoscieln tkoscieln closed this Mar 21, 2024
@tkoscieln
Copy link
Contributor Author

tkoscieln commented Mar 21, 2024 •
edited
Loading

Could you please add some context to the PR description on the specific problem/bug this PR is trying to solve? (Ideally including instructions on how to reproduce the problem)

This was part of an effort to generate new manifests for RHEL 9.3/8.9 GA in order to migrate manifest tests in osbuild. With the change in notation where dot separator was added between the major and minor version, the manifest generation broke as the old notation without the separator was hardcoded in multiple places. I closed this as it was already merged in the automated db update that was run from this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tkoscieln @ochosi