Skip to content

Comments

sync: Deploy admins' SSH keys for root#12

Merged
KellerFuchs merged 1 commit intohashbang:masterfrom
KellerFuchs:root
Feb 20, 2016
Merged

sync: Deploy admins' SSH keys for root#12
KellerFuchs merged 1 commit intohashbang:masterfrom
KellerFuchs:root

Conversation

@KellerFuchs
Copy link
Member

@KellerFuchs KellerFuchs commented Feb 19, 2016

This is only useful if we let root connect by SSH, which is happening in the next shell-etc PR.

KellerFuchs added a commit to KellerFuchs/shell-etc that referenced this pull request Feb 19, 2016
@KellerFuchs
ssh: Permit root login by pubkey only
562b02a 
This is useful in the context of hashbang/admin-tools#12
  as it gives admins access to the root account in case
  of emergency.
@KellerFuchs KellerFuchs mentioned this pull request Feb 19, 2016
Merged
@KellerFuchs
Copy link
Member Author

KellerFuchs commented Feb 19, 2016

PS: This has been already tested on the shell servers.

This was referenced Feb 19, 2016
Merged
Closed
@KellerFuchs
Copy link
Member Author

KellerFuchs commented Feb 20, 2016 

      +liam │ KellerFuchs: why are you working on a temp file in #12 and then copying it into place?
KellerFuchs │ liam: For two reasons
KellerFuchs │ 1) It automatically means that keys that are not (anymore) in ansible are removed
KellerFuchs │ 2) If something fials, either a complete file was generated and moved in place (mv is guaranteed atomic) or nothing was changed
KellerFuchs │ You cannot fall into some intermediate state where the keys are not in there anymore

KellerFuchs added a commit that referenced this pull request Feb 20, 2016
@KellerFuchs
Merge pull request #12 from KellerFuchs/root
5ae50b0 
sync: Deploy admins' SSH keys for root

Approved-by: @ArchimedesPi and @KellerFuchs
@KellerFuchs KellerFuchs merged commit 5ae50b0 into hashbang:master Feb 20, 2016
@KellerFuchs KellerFuchs deleted the root branch February 20, 2016 02:54
lrvick pushed a commit to hashbang/shell-etc that referenced this pull request Jun 15, 2017
@KellerFuchs
ssh: Permit root login by pubkey only
b14aee0 
This is useful in the context of hashbang/admin-tools#12
  as it gives admins access to the root account in case
  of emergency.
KellerFuchs added a commit to hashbang/shell-etc that referenced this pull request Sep 20, 2017
@KellerFuchs
ssh: Permit root login by pubkey only
5bee870 
This is useful in the context of hashbang/admin-tools#12
  as it gives admins access to the root account in case
  of emergency.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@KellerFuchs