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

[FEATURE] Add more comprehensive support for ssh_config, specifically bastion hosts #1058

Open
memetb opened this issue Dec 20, 2023 · 2 comments

Comments

@memetb
Copy link

memetb commented Dec 20, 2023

System Information

Linux distribution debian

Terraform version v1.1.3

Provider and libvirt versions 9adc2521e37b0c8938330bf81111d3c633825c76-dirty


Hello @dmacvicar, I've coded a new feature for the project and am opening an issue prior to submitting a WIP PR.

I have updated the code for ssh.go to add more thorough support for making use of ssh_config parameters.

This code change resolves the following specific and important issues:

  • inability to use a identity file other than ~/.ssh/id_rsa
  • inability to use bastion hosts as they are defined in the ssh_config file

This feature requires upgrading ssh_config module to v1.2.0 which is the as of now current module (is currently at v0.0.0 in the master branch). Other than this, all changes are confined to ssh.go.

@SimonWoidig
Copy link

This is something that we would love to see aswell - especially the proxies.

@memetb
Copy link
Author

memetb commented Feb 29, 2024

@SimonWoidig : fyi, this feature is completed in #1059, and merged to production branch on memetb/terraform-provider-libvirt fork project. You will have to build on your own though.

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