-
Notifications
You must be signed in to change notification settings - Fork 395
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
pkg/daemon: ensure /home/core/.ssh is there, not just /home/core #448
pkg/daemon: ensure /home/core/.ssh is there, not just /home/core #448
Conversation
90766f3
to
1a2437c
Compare
1a2437c
to
6f23585
Compare
Since we later write inside /home/core/.ssh. Signed-off-by: Antonio Murdaca <runcom@linux.com>
6f23585
to
13bed7d
Compare
@@ -608,9 +608,8 @@ func (dn *Daemon) updateSSHKeys(newUsers []ignv2_2types.PasswdUser) error { | |||
// Keys should only be written to "/home/core/.ssh" | |||
// Once Users are supported fully this should be writing to PasswdUser.HomeDir | |||
glog.Infof("Writing SSHKeys at %q", coreUserSSHPath) | |||
|
|||
if err := dn.fileSystemClient.MkdirAll(filepath.Dir(coreUserSSHPath), os.FileMode(0600)); err != nil { |
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.
for clarity:
filepath.Dir("/home/core/.ssh") == "/home/core"
filepath.Dir("/home/core/.ssh/") == "/home/core/.ssh"
just a leading slash... but we don't need that anyway
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.
gah i lost that on the refactor from the .join
to the const
. good catch!
unit flake from #417 /retest |
Haproxy e2e-aws flake /retest |
failure looks like the bug fixed in #442 /retest |
/lgtm |
Cluster operator network error /retest |
"Cluster operator network has not yet reported success" -> then timed out. |
/retest |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ashcrow, kikisdeliveryservice, runcom The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
FYI: that is correctly reported already and probably a container runtime bug /retest |
Signed-off-by: Antonio Murdaca runcom@linux.com
- What I did
The code was just checking
/home/core
but we later try to write to/home/core/.ssh
(that folder is surely there but this needs to be fixed anyway).I've centralized file writes in #401 as well to reduce the chances of doing this again.
- How to verify it
- Description for the changelog