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

install: Support being passed --filesystem with no install config #480

Merged
merged 1 commit into from
Apr 23, 2024

Conversation

cgwalters
Copy link
Collaborator

This fixes a logical bug where for base images that don't have any install configuration at all, we errored out even if we were explicitly passed --filesystem.

This is just about making it easier to test base images that don't have an install config.

@github-actions github-actions bot added the area/install Issues related to `bootc install` label Apr 20, 2024
@cgwalters cgwalters changed the title install: Support being pased --filesystem with no install config install: Support being passed --filesystem with no install config Apr 20, 2024
@cgwalters cgwalters force-pushed the install-config-optional branch 2 times, most recently from 36074f9 to 3f86d56 Compare April 20, 2024 15:07
@jeckersb jeckersb self-assigned this Apr 22, 2024
Copy link
Contributor

@jeckersb jeckersb left a comment

Choose a reason for hiding this comment

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

LGTM, one minor nit to appease our benevolent clippy overlord

lib/src/install/baseline.rs Outdated Show resolved Hide resolved
This fixes a logical bug where for base images that don't
have any install configuration at all, we errored out even
if we were explicitly passed `--filesystem`.

This is just about making it easier to test base images
that don't have an install config.

Signed-off-by: Colin Walters <walters@verbum.org>
@cgwalters cgwalters merged commit 5b618b8 into containers:main Apr 23, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/install Issues related to `bootc install`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants