Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch inplace #264

Merged
merged 3 commits into from
Jan 17, 2024
Merged

Switch inplace #264

merged 3 commits into from
Jan 17, 2024

Conversation

cgwalters
Copy link
Collaborator

switch: Move option processing earlier

This logic just converts the provided CLI options
into data structures, so it can come before we do
other things.

Prep for further work.

Signed-off-by: Colin Walters walters@verbum.org


spec: impl Display for ImageReference

I want this in a followup patch.

Signed-off-by: Colin Walters walters@verbum.org


switch: Add hidden --mutate-in-place

This is intended to be runnable in current Anaconda %post
to fix up its lack of support for --target-imgref.

Signed-off-by: Colin Walters walters@verbum.org


This logic just converts the provided CLI options
into data structures, so it can come before we do
other things.

Prep for further work.

Signed-off-by: Colin Walters <walters@verbum.org>
I want this in a followup patch.

Signed-off-by: Colin Walters <walters@verbum.org>
This is intended to be runnable in current Anaconda `%post`
to fix up its lack of support for `--target-imgref`.

Signed-off-by: Colin Walters <walters@verbum.org>
@cgwalters
Copy link
Collaborator Author

I haven't yet tried this in an anaconda %post but being runnable in a unit test environment without /run/ostree and the bootloader configs etc. gives me medium-high confidence. Doing so now.

@cgwalters
Copy link
Collaborator Author

Yep, tested in Anaconda %post, this works.

@supakeen
Copy link

Awesome, thank you. I'll be using this for osbuild/images#380

Copy link
Contributor

@ondrejbudai ondrejbudai left a comment

Choose a reason for hiding this comment

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

Splendid! Thanks a lot for this! :)

Copy link
Member

@runcom runcom left a comment

Choose a reason for hiding this comment

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

lgtm!

@cgwalters cgwalters merged commit b75b647 into containers:main Jan 17, 2024
11 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.

None yet

4 participants