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

Handle not full host definitions #3

Open
ivangretsky opened this issue Nov 17, 2020 · 1 comment
Open

Handle not full host definitions #3

ivangretsky opened this issue Nov 17, 2020 · 1 comment

Comments

@ivangretsky
Copy link

Good day, @aabouzaid !

Your script is a lifesaver! I got an issue though. My host definitions are not always "full". What I mean is that not all my hosts definitions have all Host, HostName, User and Port. Most of them have only Host, HostName and User records. Some have additional fields like IdentityFile. This brakes the script. If you have time to improve the script, please look into handing this edge case.

@aabouzaid
Copy link
Owner

@ivangretsky Thanks for reporting this issue 👍

As you see this project is 5 years old, and I don't use that style anymore where the SSH config works as a source of truth. (I used Netbox for that and used the Netbox dynamic inventory script to handle it).

But feel free to fork the project and do the changes you need.
You can mention me in the fork if you needed any help.

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

No branches or pull requests

2 participants