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: Add /usr/share/doc/bootc/baseimage #988

Merged
merged 2 commits into from
Dec 20, 2024

Conversation

cgwalters
Copy link
Collaborator

This directory will contain expected files in the base image.

That said, I may change the container import path to auto-create
at least the sysroot dir and symlink at some point and these
can just be dropped.

And for that matter after
ostreedev/ostree@9a0acd7
"libostree/deploy: enable composefs by default"
we can likely just drop the prepare-root bit too.

But for now this is needed.

Motivated by improving base image generation from
https://gitlab.com/fedora/bootc/tracker/-/issues/32

@cgwalters cgwalters force-pushed the add-install-base-configs branch 2 times, most recently from 0132405 to 5631254 Compare December 20, 2024 13:03
@cgwalters
Copy link
Collaborator Author

Oh duh of course, can't ship empty directories in git...annoying.

@cgwalters cgwalters force-pushed the add-install-base-configs branch 2 times, most recently from 39c5625 to eba3732 Compare December 20, 2024 15:46
This directory will contain expected files in the base image.

That said, I may change the container import path to auto-create
at least the sysroot dir and symlink at some point and these
can just be dropped.

And for that matter after
ostreedev/ostree@9a0acd7
"libostree/deploy: enable composefs by default"
we can likely just drop the prepare-root bit too.

But for now this is needed.

Motivated by improving base image generation from
https://gitlab.com/fedora/bootc/tracker/-/issues/32

Signed-off-by: Colin Walters <[email protected]>
This is mainly a sanity check for people building custom base
images.

But it also unit tests our own reference content.

Signed-off-by: Colin Walters <[email protected]>
@cgwalters cgwalters force-pushed the add-install-base-configs branch from eba3732 to 7d9a160 Compare December 20, 2024 16:13
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.

👍

@cgwalters cgwalters merged commit 161bc31 into containers:main Dec 20, 2024
22 of 25 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.

2 participants