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

Use user's local ssh config #145

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

xiaochuanyu
Copy link

@xiaochuanyu xiaochuanyu commented Nov 10, 2017

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
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