Skip to content

Comments

OSBuildOptions: Support --in-vm option#431

Merged
supakeen merged 1 commit intoosbuild:mainfrom
alexlarsson:in-vm
Jan 14, 2026
Merged

OSBuildOptions: Support --in-vm option#431
supakeen merged 1 commit intoosbuild:mainfrom
alexlarsson:in-vm

Conversation

@alexlarsson
Copy link
Contributor

No description provided.

@alexlarsson alexlarsson requested a review from a team as a code owner January 14, 2026 09:31
@alexlarsson alexlarsson requested review from bcl, lzap and thozza and removed request for a team January 14, 2026 09:31
Copy link
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.

Thanks for making the change.

However, this made me aware of the fact that we have a duplicate implementation of executing osbuild in this repository. Merging this PR would just add to the technical dept.

Would you be willing to look at actually replacing this with the implementation in https://github.com/osbuild/images/blob/main/pkg/osbuild/osbuild-exec.go, which should have all the knobs that you need and hopefully also that ibcli needs?

If you don't have the capacity to do this, because this is arguably much more work, I'd be OK with merging this PR and addressing this duplication later.

@supakeen
Copy link
Member

Thanks for making the change.

However, this made me aware of the fact that we have a duplicate implementation of executing osbuild in this repository. Merging this PR would just add to the technical dept.

Would you be willing to look at actually replacing this with the implementation in https://github.com/osbuild/images/blob/main/pkg/osbuild/osbuild-exec.go, which should have all the knobs that you need and hopefully also that ibcli needs?

If you don't have the capacity to do this, because this is arguably much more work, I'd be OK with merging this PR and addressing this duplication later.

I feel this is an 'us' problem so if @alexlarsson feels the same I'm good with merging this and then we fix the duplication :)

@alexlarsson
Copy link
Contributor Author

I agree that there should only be one copy of this API, but atm I don't have time to work on this. For now, this was just something I noticed when working on the vm support for bootc-image-builder, a work that is still WIP, so there is no need to immediately merge this.

@supakeen supakeen added this pull request to the merge queue Jan 14, 2026
Merged via the queue into osbuild:main with commit b8f4482 Jan 14, 2026
34 of 37 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