-
Notifications
You must be signed in to change notification settings - Fork 78
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
osconfig: install root ssh keys to /var/roothome #407
Conversation
3171be6
to
6b24170
Compare
6b24170
to
bd69633
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! This looks great and works well in my test. There is a tiny tweak in the tests needed it seems though.
bd69633
to
247cbdc
Compare
The current location via tmpfiles.d to install the root ssh keys seems to be not working. There is an error that `/root/.ssh` does not exist from `systemd-tmpfiles-setup` and indeed the authorized_keys file is on in `/root/.ssh`. It seems like everything else in the firstboot in tmpfiles.d is refering to `/var/roothome` instead of `/root` and switching to this for the location of the ssh keys solves the issue. Co-authored-by: Colin Walters <[email protected]> Signed-off-by: Colin Walters <[email protected]>
247cbdc
to
119dbb4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much! 👍 for all the change that I did not make (which are all relevant ones ;)
The current location via tmpfiles.d to install the root ssh keys seems to be not working on firstboot. There is an error that
/root/.ssh
does not exist fromsystemd-tmpfiles-setup
and indeed the authorized_keys file is on in/root/.ssh
.It seems like everything else in the firstboot in
tmpfiles.d is refering to
/var/roothome
instead of/root
and switching to this for the location of the ssh keys solves the issue.I tested this via bib and a local container using the new
bootc install to-filesystem
pipeline :) So hopefully it should finally work.