-
Notifications
You must be signed in to change notification settings - Fork 240
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
ssh: Use ed25519 algorithm instead ECDSA #4304
Conversation
d26876c
to
85a36a2
Compare
Key generated using ecdsa algorithm is causing issue for podman remote connection on podman desktop side because the library they consume doesn't have support for this algorithm. This PR is switching the ecdsa to ed25519 with openssh type which is supported by the library consumed in podman desktop. [0] podman-desktop/podman-desktop#8351 [1] mscdex/ssh2#1375
…mpatibility We were using RSA Algo long time ago and this function is used to provide a backward compatibility for that version of crc but since in previous commit we have updated the ssh algo from ecdsa to ed25519, to have the backward compatibility for some released version we are renaming this function.
85a36a2
to
8794f8a
Compare
/retest |
@praveenkumar: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Can you list the testing steps in the PR description? |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: anjannath The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
podman-desktop/podman-desktop#8351 have steps around how it can be tested, I also added the steps here. |
What i did to test was, ensure running then verified i can ssh into the crc VM using the new
|
Thanks @praveenkumar and @anjannath . I tested and it works for me too via terminal but not through PD (I can't see any id_ed25519 file in the crc directory). Also, the step to add podman system connection is only required for Windows, right? I am on MacBook. Logs:
Podman Desktop version (v1.12.0) |
there's a typo in this command:
the path to the ssh key should use the |
Key generated using ecdsa algorithm is causing issue for podman remote
connection on podman desktop side because the library they consume
doesn't have support for this algorithm. This PR is switching the ecdsa
to ed25519 which is supported by the library consumed in podman desktop.
[0] podman-desktop/podman-desktop#8351
[1] mscdex/ssh2#1375
how to test
Setting => preference => Extension: podman => Enable the Remote