Skip to content

distro: Make isoCustomizations usable by bootcImageType#2360

Merged
ondrejbudai merged 2 commits into
osbuild:mainfrom
brlane-rht:main-iso-custom
May 25, 2026
Merged

distro: Make isoCustomizations usable by bootcImageType#2360
ondrejbudai merged 2 commits into
osbuild:mainfrom
brlane-rht:main-iso-custom

Conversation

@brlane-rht
Copy link
Copy Markdown
Contributor

The function only depends on ISOLabel() and getDefaultISOConfig(). Make it take a new interface, ISOImageType, so that it can be shared between regular images and bootc images.

Related: HMS-10627

The function only depends on ISOLabel() and getDefaultISOConfig(). Make
it take a new interface, ISOImageType, so that it can be shared between
regular images and bootc images.

Related: HMS-10627
@brlane-rht brlane-rht requested a review from a team as a code owner May 21, 2026 18:41
This satisfies the ISOImageType interface so that isoCustomizations
function can be used with bootc image types.

Related: HMS-10627
Copy link
Copy Markdown
Member

@thozza thozza left a comment

Choose a reason for hiding this comment

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

Functionally LGTM, but there is one style / common practice inconsistency.

Comment thread pkg/distro/generic/bootc_imagetype.go
Copy link
Copy Markdown
Member

@thozza thozza left a comment

Choose a reason for hiding this comment

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

I got confused by a struct and method of the same name. All good.

@thozza thozza added this pull request to the merge queue May 25, 2026
@ondrejbudai ondrejbudai removed this pull request from the merge queue due to a manual request May 25, 2026
@ondrejbudai ondrejbudai added this pull request to the merge queue May 25, 2026
Merged via the queue into osbuild:main with commit df42f97 May 25, 2026
25 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.

4 participants