Skip to content

exec/util/passwd: rename authorized_keys.d fragment to "ignition"#749

Merged
ajeddeloh merged 1 commit intocoreos:masterfrom
rfairley:rfairley-rename-authorized_keys_d
Feb 28, 2019
Merged

exec/util/passwd: rename authorized_keys.d fragment to "ignition"#749
ajeddeloh merged 1 commit intocoreos:masterfrom
rfairley:rfairley-rename-authorized_keys_d

Conversation

@rfairley
Copy link
Copy Markdown
Contributor

@rfairley rfairley commented Feb 26, 2019

Renames the written fragment in ~/.ssh/authorized_keys.d/ from
"coreos-ignition" to "ignition". This is to have a more
distro-independent identifier.

Fixes: #710


Tested this change in FCOS, and confirmed the ignition fragment gets written with the keys given in the Ignition config "sshAuthorizedKeys".

$ ls ~/.ssh/authorized_keys.d/ignition -l
-rw-------. 1 core core 424 Feb 26 19:48 /var/home/core/.ssh/authorized_keys.d/ignition

@coreosbot
Copy link
Copy Markdown
Contributor

Can one of the admins verify this patch?

@arithx
Copy link
Copy Markdown
Contributor

arithx commented Feb 26, 2019

ok to test

Renames the written fragment in ~/.ssh/authorized_keys.d/ from
"coreos-ignition" to "ignition". This is to have a more
distro-independent identifier.

Fixes: coreos#710
@rfairley rfairley force-pushed the rfairley-rename-authorized_keys_d branch from 6c3cc44 to 37e5465 Compare February 26, 2019 21:05
@rfairley
Copy link
Copy Markdown
Contributor Author

^ re-pushed for a minor edit in the commit message

Copy link
Copy Markdown
Contributor

@ajeddeloh ajeddeloh left a comment

Choose a reason for hiding this comment

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

LGTM

@rfairley
Copy link
Copy Markdown
Contributor Author

Good to merge from my end

@ajeddeloh ajeddeloh merged commit 3ede525 into coreos:master Feb 28, 2019
@rfairley rfairley deleted the rfairley-rename-authorized_keys_d branch February 28, 2019 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants