Skip to content

Reinstall ssh followup cleanup #1153

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

Merged
merged 4 commits into from
Feb 27, 2025
Merged

Conversation

ckyrouac
Copy link
Collaborator

Just a couple of cleanup commits from review comments.

This provides stricter parsing (e.g. fails when the authorized_keys file
contains invalid content), and is groundwork for removing the command
field from keys.

Signed-off-by: ckyrouac <ckyrouac@redhat.com>
These options could contain a command which will cause the new bootc
system to be inaccessible.

Signed-off-by: ckyrouac <ckyrouac@redhat.com>
Signed-off-by: ckyrouac <ckyrouac@redhat.com>
Signed-off-by: ckyrouac <ckyrouac@redhat.com>
@github-actions github-actions bot added the area/system-reinstall-bootc Issues related to system-reinstall-botoc label Feb 27, 2025
@cgwalters
Copy link
Collaborator

Well since the integration testing is busted (which is the real expensive one that makes me sometimes hesitate to force push updates to existing PRs for nits), I guess we should maybe just close #1150 and merge this one?

@ckyrouac ckyrouac mentioned this pull request Feb 27, 2025
@ckyrouac ckyrouac merged commit 859bf9e into bootc-dev:main Feb 27, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/system-reinstall-bootc Issues related to system-reinstall-botoc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants