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

jewel: tools: ceph-post-file: use new ssh key #11746

Merged
2 commits merged into from Nov 3, 2016

Conversation

liewegas
Copy link
Member

@liewegas liewegas commented Nov 2, 2016

liewegas and others added 2 commits November 2, 2016 09:37
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>
(cherry picked from commit ecd02bf)

# Conflicts:
#	src/CMakeLists.txt
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>
(cherry picked from commit a61fcb2)
@liewegas liewegas added this to the jewel milestone Nov 2, 2016
@ghost ghost self-assigned this Nov 2, 2016
@ghost ghost added the bug-fix label Nov 2, 2016
@ghost
Copy link

ghost commented Nov 2, 2016

pushed as wip-post-file-key-jewel for gitbuilders

@ghost
Copy link

ghost commented Nov 2, 2016

tests to validate the packaging changes are ok

teuthology-suite -k distro --verbose --suite upgrade/jewel-x --suite-branch jewel --ceph wip-post-file-key-jewel --machine-type vps --priority 101 machine_types/vps.yaml

@badone
Copy link
Contributor

badone commented Nov 2, 2016

lgtm

@ghost ghost changed the title ceph-post-file: use new ssh key (jewel) jewel: ceph-post-file: use new ssh key Nov 3, 2016
@ghost ghost merged commit 5e079cf into ceph:jewel Nov 3, 2016
@theanalyst theanalyst changed the title jewel: ceph-post-file: use new ssh key "jewel: tools: ceph-post-file: use new ssh key" Nov 17, 2016
@theanalyst theanalyst changed the title "jewel: tools: ceph-post-file: use new ssh key" jewel: tools: ceph-post-file: use new ssh key Nov 17, 2016
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants