-
Notifications
You must be signed in to change notification settings - Fork 456
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
expanded ssh_config parameters for qemu+ssh uri option #1059
base: main
Are you sure you want to change the base?
Conversation
this allows different hosts (jump hosts) to have different identity files specified
as per commit from MaxMatti: dmacvicar@1152bdd
the hostKeyCallback makes use of the SSH port and fails if a custom ssh port is being used by the host
this value was chosen as the lowest RSA available by default on a debian build running OpenSSH_9.2 and works out of the box for most hosts tested by authority. Any older systems can specifically set their key algorithms in .ssh/config
082f086
to
dec8f23
Compare
will this ever be merged? |
@tuxillo fyi, I'm using my forked project in production. I haven't published that project on terraform specifically because I want to keep the noise ratio down, but if you so wish, you can use the forked project's production branch which has this merged as well as a couple of other small features merged in. |
@dmacvicar This looks like good improvement regarding the ssh connection. May you have a look regarding compatibility, etc. if this can be merged into upstream? |
Hello @dmacvicar,
this PR is in relation to the issue #1058 I recently opened.
This is a preliminary PR and work in progress. I have listed the known issues with this PR at this point. I would also like to get feedback from you before putting more effort in.This feature is a must-have for my use case since I require to access my bare metal servers through a bastion host.
Known issues:
3. little to no effort on code style and documentation: this was a quick and dirty feature feasibility test to see if the desired workflow worked correctly