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

Remove 'iso' image type again #177

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

AdamWill
Copy link
Contributor

I believe adding this was incorrect. 'iso' makes no sense as an "image type" in productmd terms. See
https://pagure.io/pungi-fedora/issue/1342#comment-927846 and https://pagure.io/pungi-fedora/issue/1342#comment-927848 for a more detailed rationale. We need to fix the Kiwi/Pungi pipeline such that ISO images it produces have correct "type"s, not add conceptually-nonsensical types to productmd so the current pipeline works.

I believe adding this was incorrect. 'iso' makes no sense as an
"image type" in productmd terms. See
https://pagure.io/pungi-fedora/issue/1342#comment-927846 and
https://pagure.io/pungi-fedora/issue/1342#comment-927848 for a
more detailed rationale. We need to fix the Kiwi/Pungi pipeline
such that ISO images it produces have correct "type"s, not add
conceptually-nonsensical types to productmd so the current
pipeline works.

Signed-off-by: Adam Williamson <[email protected]>
@lubomir
Copy link
Contributor

lubomir commented Aug 29, 2024

I agree with your reasoning. Let's merge this.

@lubomir lubomir merged commit ba6a9cc into release-engineering:master Aug 29, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants