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: Two prep changes for pulling bound images at install time #707

Merged
merged 2 commits into from
Jul 18, 2024

Conversation

cgwalters
Copy link
Collaborator

Code similar to this was part of the original #559 - this is prep for pulling bound images at install time


install: Split internal main phase into two

Split up:

  • Initialize empty sysroot
  • Install container to sysroot

This makes it easier to do more things that still
reference the sysroot.

(The git diff for this change is...really nice IMO,
very straightforward)

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


install: Change helper to return deployment too

Prep for further changes.

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


Split up:

- Initialize empty sysroot
- Install container to sysroot

This makes it easier to do more things that still
reference the sysroot.

(The git diff for this change is...really nice IMO,
 very straightforward)

Signed-off-by: Colin Walters <walters@verbum.org>
@github-actions github-actions bot added the area/install Issues related to `bootc install` label Jul 18, 2024
@cgwalters cgwalters enabled auto-merge July 18, 2024 14:21
@cgwalters
Copy link
Collaborator Author

mount: /run/bootc/mounts/rootfs: mount point is busy.
ERROR Installing to disk: Task Finalizing filesystem rootfs failed: ExitStatus(unix_wait_status(8192))

Huh, hadn't seen that one before...oh oh wait, I know.

Prep for further changes.

Signed-off-by: Colin Walters <walters@verbum.org>
@cgwalters cgwalters merged commit 83a57e5 into containers:main Jul 18, 2024
8 of 28 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.

2 participants