Skip to content

Conversation

@mvo5
Copy link
Collaborator

@mvo5 mvo5 commented Nov 19, 2025

This commit removes bib/cmd/bootc-image-builder and all pieces that it pulled in and replaces it with the new image-builder-cli multicall binary.

It also removes one test which is a bit unfortunate but with the move to ibcli this needs to be re-added there anyway.

@achilleas-k achilleas-k self-requested a review November 19, 2025 17:42
@mvo5 mvo5 force-pushed the use-ibcli branch 4 times, most recently from 26384e1 to 1118516 Compare November 25, 2025 09:02
@mvo5 mvo5 force-pushed the use-ibcli branch 2 times, most recently from b19dbb2 to 00c72f1 Compare December 5, 2025 13:08
@mvo5 mvo5 marked this pull request as ready for review December 5, 2025 13:09
@mvo5 mvo5 requested a review from a team as a code owner December 5, 2025 13:09
@mvo5 mvo5 requested review from croissanne and thozza and removed request for a team December 5, 2025 13:09
build.sh Outdated
# XXX2: remove bootc-image-buidler build here entirely and take it from the upstream
# image-builer-cli container in the containerfile instead?
#
# XXX: replace with git clone --depth 1 github.com/osbuild/image-builder-cli@rev
Copy link
Member

Choose a reason for hiding this comment

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

Isn't it possible to do this already?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oh boy, sorry for that too. I went over this again and I hope I did not miss another silly leftover.

@mvo5 mvo5 force-pushed the use-ibcli branch 2 times, most recently from 8707b55 to c2a182c Compare December 8, 2025 15:25
mvo5 added 3 commits December 8, 2025 16:35
This commit removes bib/cmd/bootc-image-builder and all pieces
that it pulled in and replaces it with the new image-builder-cli
multicall binary.
This test is no longer working, it seems its a result of the
merge of
osbuild/image-builder-cli#316

Drop it for now so that we can move forward with the merge.
There is a similar test in
https://github.com/osbuild/image-builder-cli/pull/316/files#diff-57c49abc8a31b46856c2eecb2edfeff6072cd8d0edf769a67b6d4fdaabd8321aR100
that does not cover "auto" but any fixes/regression should be
done in the ibcli repo.
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