Skip to content

Conversation

@mvo5
Copy link
Contributor

@mvo5 mvo5 commented Feb 6, 2025

This commit updates ibcli for the new API in images for the reporegistry.New(). The main incompatible change is that the /repositories part is not longer automatically added inside the library so we need to add it on the call-site.

This needs osbuild/images#1179

@mvo5 mvo5 force-pushed the update-for-image-api1179 branch from f23af59 to 602905d Compare February 10, 2025 12:26
@mvo5 mvo5 marked this pull request as ready for review February 10, 2025 19:28
mvo5 added 2 commits February 10, 2025 20:40
This commit updates ibcli for the new API in images for the
`reporegistry.New()`. The main incompatible change is that the
`/repositories` part is not longer automatically added inside
the library so we need to add it on the call-site.

This needs osbuild/images#1179
Copy link
Member

@achilleas-k achilleas-k left a comment

Choose a reason for hiding this comment

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

Splendid.

@mvo5 mvo5 added this pull request to the merge queue Feb 11, 2025
Merged via the queue into osbuild:main with commit 0678d8d Feb 11, 2025
32 checks passed
@mvo5 mvo5 deleted the update-for-image-api1179 branch February 11, 2025 12:35
mvo5 added a commit to mvo5/image-builder-cli that referenced this pull request Oct 30, 2025
This commit fixes the missing appending of `repositories` when
`--data-dirs` is used. It also improves the tests a bit to
ensure this won't happen again.

Thanks to Achilleas for finding this!

This was missing when we did
osbuild#128
github-merge-queue bot pushed a commit that referenced this pull request Oct 30, 2025
This commit fixes the missing appending of `repositories` when
`--data-dirs` is used. It also improves the tests a bit to
ensure this won't happen again.

Thanks to Achilleas for finding this!

This was missing when we did
#128
mvo5 added a commit to mvo5/image-builder-cli that referenced this pull request Dec 8, 2025
This commit fixes the missing appending of `repositories` when
`--data-dirs` is used. It also improves the tests a bit to
ensure this won't happen again.

Thanks to Achilleas for finding this!

This was missing when we did
osbuild#128
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