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

ceph-post-file replace DSA with RSA ssh key #10800

Merged
merged 2 commits into from Aug 22, 2016
Merged

ceph-post-file replace DSA with RSA ssh key #10800

merged 2 commits into from Aug 22, 2016

Conversation

djgalloway
Copy link

@djgalloway djgalloway commented Aug 19, 2016

DSA keys are being deprecated: http://www.openssh.com/legacy.html

drop.ceph.com will continue to allow the old DSA key but eventually,
users submitting logs using ceph-post-file will run into issues when
OpenSSH completely drops support for the algorithm.

Fixes: http://tracker.ceph.com/issues/14267

@liewegas
Copy link
Member

rados suite has a ceph-post-file test

@tchaikov
Copy link
Contributor

tchaikov commented Aug 21, 2016

tested at http://pulpito.ceph.com/kchai-2016-08-20_18:41:17-rados-wip-kefu-testing---basic-mira/

failed ones are tracked by

@djgalloway could you put

Fixes: http://tracker.ceph.com/issues/14267

instead of

Fixes: #14267

because #<number> could collide with the pull request number. see also https://github.com/ceph/ceph/blob/master/SubmittingPatches.rst#preparing-and-sending-patches

otherwise, lgtm.

David Galloway added 2 commits August 22, 2016 10:27
DSA keys are being deprecated: http://www.openssh.com/legacy.html

drop.ceph.com will continue to allow the old DSA key but eventually,
users submitting logs using ceph-post-file will run into issues when
OpenSSH completely drops support for the algorithm.

Fixes: http://tracker.ceph.com/issues/14267

Signed-off-by: David Galloway <dgallowa@redhat.com>
In my case, I had multiple private keys in ssh-agent which resulted in
the sftp connection failing despite explicitly specifying the private
key to use

Signed-off-by: David Galloway <dgallowa@redhat.com>
@djgalloway
Copy link
Author

@tchaikov: Fixed commit message

@tchaikov tchaikov merged commit 1a23c2e into master Aug 22, 2016
@tchaikov tchaikov deleted the wip-postfile-rsa branch August 22, 2016 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants