Skip to content

Conversation

@mvo5
Copy link
Collaborator

@mvo5 mvo5 commented Aug 20, 2025

This PR (or the need for it) indicates once more that we really
need to consolidate the manifest generation into the images
library (c.f. osbuild/images#1736) as
most of the adjustments would have one with the various
API changing changes in images if it was part of "images".


go.mod: update images to v0.177


image: update blueprint import path

We no longer ship the blueprint package inside the images library
but its now part of its own "blueprint" library.


main: update for v0.177 API changes

This includes:

  • disk.ParitioningMode->parition.PartitioningMode
  • anaconda.AdditionalModules -> anaconda.EnabledModules
  • use of new InstallerCustomizations
  • aws uploader needs arch.Arch

bib: enable anaconda Module{Network,Payload,Runtime}

Previously images [0] would take AdditionalAnacondaModules,
i.e. we had a set of defaults in code with

  • anaconda.ModuleNetwork
  • anaconda.ModulePayloads
  • anaconda.ModuleRuntime
    enabled by default. With the PR landed there are no longer
    default so we need to update our code to include those
    in the list of EnabledAnacondaModules.

[0] Before osbuild/images#1737

@mvo5 mvo5 requested a review from supakeen August 20, 2025 14:42
Copy link
Member

@supakeen supakeen left a comment

Choose a reason for hiding this comment

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

Yea, soon this is hopefully less or not at all necessary.

@mvo5
Copy link
Collaborator Author

mvo5 commented Aug 20, 2025

Hrm, it looks like the centos9 based iso install test is hanging, I need to debug this further :/

@mvo5 mvo5 added this pull request to the merge queue Aug 21, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 21, 2025
@mvo5 mvo5 added this pull request to the merge queue Aug 21, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 21, 2025
@supakeen supakeen added this pull request to the merge queue Aug 21, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 21, 2025
mvo5 added 5 commits August 21, 2025 19:15
We no longer ship the blueprint package inside the images library
but its now part of its own "blueprint" library.
This includes:
- disk.ParitioningMode->parition.PartitioningMode
- anaconda.AdditionalModules -> anaconda.EnabledModules
- use of new InstallerCustomizations
- aws uploader needs arch.Arch
Previously images [0] would take `AdditionalAnacondaModules`,
i.e. we had a set of defaults in code with
- anaconda.ModuleNetwork
- anaconda.ModulePayloads
- anaconda.ModuleRuntime
- anaconda.ModuleStorage
enabled by default. With the PR landed there are no longer
default so we need to update our code to include those
in the list of EnabledAnacondaModules.

[0] Before osbuild/images#1737
@supakeen
Copy link
Member

I can keep trying to add this to the merge queue, but will it ever merge?

@supakeen supakeen added this pull request to the merge queue Aug 21, 2025
Merged via the queue into osbuild:main with commit 2da427a Aug 21, 2025
21 of 22 checks passed
mvo5 added a commit to mvo5/bootc-image-builder that referenced this pull request Aug 25, 2025
This commit fixes the issue that with an unset --target-arch
the code will try to convert an empty string to an arch.Arch
which then fails. This is a regression from
osbuild#1017
that was not caught.
mvo5 added a commit to mvo5/bootc-image-builder that referenced this pull request Aug 25, 2025
This commit fixes the issue that with an unset --target-arch
the code will try to convert an empty string to an arch.Arch
which then fails. This is a regression from
osbuild#1017
that was not caught.

Closes: osbuild#1029
mvo5 added a commit to mvo5/bootc-image-builder that referenced this pull request Aug 25, 2025
This commit fixes the issue that with an unset --target-arch
the code will try to convert an empty string to an arch.Arch
which then fails. This is a regression from
osbuild#1017
that was not caught.

Closes: osbuild#1029
github-merge-queue bot pushed a commit that referenced this pull request Aug 27, 2025
This commit fixes the issue that with an unset --target-arch
the code will try to convert an empty string to an arch.Arch
which then fails. This is a regression from
#1017
that was not caught.

Closes: #1029
mvo5 added a commit to mvo5/image-builder-cli that referenced this pull request Nov 19, 2025
This commit fixes the issue that with an unset --target-arch
the code will try to convert an empty string to an arch.Arch
which then fails. This is a regression from
osbuild/bootc-image-builder#1017
that was not caught.

Closes: osbuild/bootc-image-builder#1029
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