Skip to content
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

add a list of host fingerprints #24

Merged
merged 1 commit into from Dec 1, 2014

Conversation

sa2ajj
Copy link

@sa2ajj sa2ajj commented Nov 26, 2014

No description provided.

@seankelly
Copy link
Member

Having these in a format that can be copied to known_hosts would be nice. I am also unsure if the RSA1 key is necessary. I've seen DSA used rarely, but never RSA1 over RSA.

@sa2ajj
Copy link
Author

sa2ajj commented Nov 27, 2014

Good points. I will fix that.

@sa2ajj
Copy link
Author

sa2ajj commented Nov 29, 2014

On a related note: maybe we should only enable protocol 2?

@djmitche
Copy link
Member

As for ssh2 only -- I had assumed that was the default, but yes, good idea.

I'm not opposed to listing host keys in the docs, as that will give new admins a secure way to set up their known_hosts file. But after that, SSH itself will verify the key on connect, so I don't see why Ansible needs to be involved. If we had inter-host SSH connections going on, then using Ansible to get all host keys and share them to all other hosts in /etc/ssh/known_hosts would be useful. But that's not useful right now.

@seankelly
Copy link
Member

Agreed. Having the host keys in the docs is useful, both for new admins and for new hosts. Having a playbook to get new keys does not seem as useful due to not needing it very often. When new hosts are brought up, the keys can be either fetched by the person logging in the first time or even better getting them immediately after OS installation.

@sa2ajj
Copy link
Author

sa2ajj commented Dec 1, 2014

Removed the playbook.

djmitche added a commit to djmitche/buildbot-infra that referenced this pull request Dec 1, 2014
@djmitche djmitche merged commit 9604a3c into buildbot:master Dec 1, 2014
@sa2ajj sa2ajj deleted the add-fingerprints branch December 1, 2014 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants