Skip to content

docs: fix injecting root keys example#209

Merged
vrothberg merged 1 commit intobootc-dev:mainfrom
runcom:fix-root-keys-docs
Nov 24, 2023
Merged

docs: fix injecting root keys example#209
vrothberg merged 1 commit intobootc-dev:mainfrom
runcom:fix-root-keys-docs

Conversation

@runcom
Copy link
Contributor

@runcom runcom commented Nov 22, 2023

Fails with the following otherwise:

STEP 9/11: RUN mkdir -p /usr/etc-system/ &&     echo 'AuthorizedKeysFile /usr/etc-system/%u.keys' >> /etc/ssh/sshd_config.d/30-auth-system.conf &&     echo 'ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIL7xFq1HtZKZiaD8MfkhNtn37m8GSc1W168NoSaT9RSf cardno:000F_C36A3FC0' > /usr/etc-system/root.keys && chmod 0600 /usr/etc-system/keys
chmod: cannot access '/usr/etc-system/keys': No such file or directory
Error: building at STEP "RUN mkdir -p /usr/etc-system/ &&     echo 'AuthorizedKeysFile /usr/etc-system/%u.keys' >> /etc/ssh/sshd_config.d/30-auth-system.conf &&     echo 'ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIL7xFq1HtZKZiaD8MfkhNtn37m8GSc1W168NoSaT9RSf cardno:000F_C36A3FC0' > /usr/etc-system/root.keys && chmod 0600 /usr/etc-system/keys": while running runtime: exit status 1

@runcom
docs: fix injecting root keys example
f856b6f 
Signed-off-by: Antonio Murdaca <antoniomurdaca@gmail.com>
@dougsland
Copy link

dougsland commented Nov 22, 2023

LGTM

vrothberg
vrothberg approved these changes Nov 24, 2023
View reviewed changes
Copy link
Contributor

@vrothberg vrothberg left a comment

Choose a reason for hiding this comment

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

LGTM

@vrothberg vrothberg merged commit 0137dcc into bootc-dev:main Nov 24, 2023
cgwalters added a commit to cgwalters/bootc that referenced this pull request Jan 16, 2026
@cgwalters
Update composefs-rs to merged PR bootc-dev#209
68225d6 
The composefs-rs PR 209 has been merged to main. This updates
bootc to use the containers/composefs-rs repository at the
merge commit.

Key API changes:
- Directory::default() -> Directory::new(Stat::uninitialized())
- read_filesystem() no longer takes stat_root parameter
- New read_container_root() for OCI containers (propagates /usr metadata to root)
- stat_root CLI flag renamed to no_propagate_usr_to_root with inverted logic

See containers/composefs-rs#209

Signed-off-by: Colin Walters <walters@verbum.org>
cgwalters added a commit to cgwalters/bootc that referenced this pull request Jan 16, 2026
@cgwalters
Update composefs-rs to merged PR bootc-dev#209
56490c1 
The composefs-rs PR 209 has been merged to main. This updates
bootc to use the containers/composefs-rs repository at the
merge commit.

Key API changes:
- Directory::default() -> Directory::new(Stat::uninitialized())
- read_filesystem() no longer takes stat_root parameter
- New read_container_root() for OCI containers (propagates /usr metadata to root)
- stat_root CLI flag renamed to no_propagate_usr_to_root with inverted logic

See containers/composefs-rs#209

Signed-off-by: Colin Walters <walters@verbum.org>
cgwalters added a commit to cgwalters/bootc that referenced this pull request Jan 16, 2026
@cgwalters
Update composefs-rs to merged PR bootc-dev#209
3189696 
The composefs-rs PR 209 has been merged to main. This updates
bootc to use the containers/composefs-rs repository at the
merge commit.

Key API changes:
- Directory::default() -> Directory::new(Stat::uninitialized())
- read_filesystem() no longer takes stat_root parameter
- New read_container_root() for OCI containers (propagates /usr metadata to root)
- stat_root CLI flag renamed to no_propagate_usr_to_root with inverted logic

See containers/composefs-rs#209

Signed-off-by: Colin Walters <walters@verbum.org>
cgwalters added a commit to cgwalters/bootc that referenced this pull request Jan 19, 2026
@cgwalters
Update composefs-rs to merged PR bootc-dev#209
481f75a 
The composefs-rs PR 209 has been merged to main. This updates
bootc to use the containers/composefs-rs repository at the
merge commit.

Key API changes:
- Directory::default() -> Directory::new(Stat::uninitialized())
- read_filesystem() no longer takes stat_root parameter
- New read_container_root() for OCI containers (propagates /usr metadata to root)
- stat_root CLI flag renamed to no_propagate_usr_to_root with inverted logic

See containers/composefs-rs#209

Signed-off-by: Colin Walters <walters@verbum.org>
cgwalters added a commit to cgwalters/bootc that referenced this pull request Jan 19, 2026
@cgwalters
Update composefs-rs to merged PR bootc-dev#209
6135a3f 
The composefs-rs PR 209 has been merged to main. This updates
bootc to use the containers/composefs-rs repository at the
merge commit.

Key API changes:
- Directory::default() -> Directory::new(Stat::uninitialized())
- read_filesystem() no longer takes stat_root parameter
- New read_container_root() for OCI containers (propagates /usr metadata to root)
- stat_root CLI flag renamed to no_propagate_usr_to_root with inverted logic

See containers/composefs-rs#209

Signed-off-by: Colin Walters <walters@verbum.org>
cgwalters added a commit that referenced this pull request Jan 22, 2026
@cgwalters
Update composefs-rs to merged PR #209
efa510a 
The composefs-rs PR 209 has been merged to main. This updates
bootc to use the containers/composefs-rs repository at the
merge commit.

Key API changes:
- Directory::default() -> Directory::new(Stat::uninitialized())
- read_filesystem() no longer takes stat_root parameter
- New read_container_root() for OCI containers (propagates /usr metadata to root)
- stat_root CLI flag renamed to no_propagate_usr_to_root with inverted logic

See containers/composefs-rs#209

Signed-off-by: Colin Walters <walters@verbum.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@vrothberg vrothberg vrothberg approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@runcom @dougsland @vrothberg