Skip to content

cmd: remove bibupload as it was never exposed to the public#405

Merged
mvo5 merged 1 commit intoosbuild:mainfrom
mvo5:kill-bibupload
Dec 12, 2025
Merged

cmd: remove bibupload as it was never exposed to the public#405
mvo5 merged 1 commit intoosbuild:mainfrom
mvo5:kill-bibupload

Conversation

@mvo5
Copy link
Contributor

@mvo5 mvo5 commented Dec 12, 2025

The bibupload command was originally part of bootc-image-builder and when we moved the go code it moved here. But after talking to Ondrej we decided it can be removed because it was never part of the bootc-image-builder container. It is also problematic as we don't have enough information from the image alone how to upload it (i.e. what boot configuration etc). So lets just remove it, if (and that is a big if) we ever need it again we can alwas resurrect it from the git history.

The bibupload command was originally part of bootc-image-builder
and when we moved the go code it moved here. But after talking
to Ondrej we decided it can be removed because it was never
part of the bootc-image-builder container. It is also problematic
as we don't have enough information from the image alone how
to upload it (i.e. what boot configuration etc). So lets just
remove it, *if* (and that is a big if) we ever need it again
we can alwas resurrect it from the git history.
@mvo5 mvo5 requested a review from ondrejbudai December 12, 2025 10:29
@mvo5 mvo5 requested a review from a team as a code owner December 12, 2025 10:29
@mvo5 mvo5 requested review from bcl, lzap and thozza and removed request for a team December 12, 2025 10:29
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.

🔥 😍

@mvo5 mvo5 added this pull request to the merge queue Dec 12, 2025
Merged via the queue into osbuild:main with commit b38aeb1 Dec 12, 2025
38 of 39 checks passed
@mvo5 mvo5 deleted the kill-bibupload branch December 12, 2025 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants