Skip to content
This repository was archived by the owner on Aug 29, 2023. It is now read-only.

Use user's local ssh config#145

Open
xiaochuanyu wants to merge 1 commit into
YelpArchive:masterfrom
xiaochuanyu:master
Open

Use user's local ssh config#145
xiaochuanyu wants to merge 1 commit into
YelpArchive:masterfrom
xiaochuanyu:master

Conversation

@xiaochuanyu

@xiaochuanyu xiaochuanyu commented Nov 10, 2017

Copy link
Copy Markdown

I use a different users and keys for SSH depending on which host(s).
kafka-check doesn't allow this right now.
This PR attempts to parse user's local ssh config and make use of it.
Sample ssh config where this applies:

Host 1.2.3.4
   User ec2-user
   IdentityFile ~/.ssh/key.pem
   StrictHostKeyChecking no
   UserKnownHostsFile /dev/null

@simplesteph

Copy link
Copy Markdown
Contributor

@xiaochuanyu I did something similar here: https://github.com/Yelp/kafka-utils/pull/138/files
It may be worth refactoring your code and isolate the ssh client construction method for all the other components to benefit from it

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants