Skip to content

Conversation

kingsleyzissou
Copy link
Contributor

The blueprint convert function was missing the json tailoring case. This meant that if the json tailoring customization was provided in the blueprint, the customization would get ignored and the tailoring profile would not be applied to the image.

The blueprint convert function was missing the json tailoring case. This meant that
if the json tailoring customization was provided in the blueprint, the customization
would get ignored and the tailoring profile would not be applied to the image.
@kingsleyzissou kingsleyzissou requested a review from a team as a code owner September 8, 2025 13:28
@kingsleyzissou kingsleyzissou requested review from lzap, bcl, mvo5 and thozza and removed request for a team September 8, 2025 13:28
Copy link
Contributor

@lzap lzap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks clean. Tho I am unsure about the workflow - I see this is being done for EL9/EL10 but I do not see similar patch in main. If this was extracted from a patch, please include commit sha that this was extracted from.

A jira issue would be helpful too.

@kingsleyzissou
Copy link
Contributor Author

Looks clean. Tho I am unsure about the workflow - I see this is being done for EL9/EL10 but I do not see similar patch in main. If this was extracted from a patch, please include commit sha that this was extracted from.

A jira issue would be helpful too.

It's not in main since we moved to the external blueprint library and this issue isn't there.

Ya sorry, I'm trying to figure out the jira flow here. I have a bug that I'm trying to fix but it's affecting 9.6 & 10.0

@kingsleyzissou kingsleyzissou changed the title [10.0] internal/blueprint: add JSON tailoring to bp conversion [10.0] internal/blueprint: add JSON tailoring to bp conversion (HMS-9302) Sep 8, 2025
@croissanne croissanne merged commit 514f824 into osbuild:rhel-10.0 Sep 15, 2025
23 of 24 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.

3 participants