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 bcl, lzap, supakeen 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 [9.6.0] internal/blueprint: add JSON tailoring to bp conversion [9.6.0] internal/blueprint: add JSON tailoring to bp conversion (HMS-9303) Sep 8, 2025
@croissanne croissanne merged commit 62822b5 into osbuild:rhel-9.6.0 Sep 15, 2025
25 of 26 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