Skip to content

mgr/cephadm: make orch status fail, if ssh key is not set#35678

Merged
sebastian-philipp merged 1 commit intoceph:masterfrom
sebastian-philipp:cephadm-availabe-ssh-set
Jun 23, 2020
Merged

mgr/cephadm: make orch status fail, if ssh key is not set#35678
sebastian-philipp merged 1 commit intoceph:masterfrom
sebastian-philipp:cephadm-availabe-ssh-set

Conversation

@sebastian-philipp
Copy link
Contributor

@sebastian-philipp sebastian-philipp commented Jun 19, 2020

ceph-salt relies on the output of orch status to determine the
end of the bootstrap process.

See ceph/ceph-salt#270

Signed-off-by: Sebastian Wagner sebastian.wagner@suse.com

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard backend
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

ceph-salt relies on the output of `orch status` to determine the
end of the bootstrap process.

Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
Copy link
Contributor

@ricardoasmarques ricardoasmarques left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working as expected, LGTM:

master:~ # ceph cephadm get-pub-key
ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQCWCuV+r5MrtY9K5vmigbvII62MpvT8rW6Jq2iIiBsugILSb7gal//avxJNM3jdJAQE8/wUnjVm507x3Yvbzbso8WE+RZnRzroPrnwXS02v/dgove2xsKKX0reJUKJj6m0Ut8kVjb6eKiy2LEd9LHuu+xrJulEgGx25tCbSD/WHY0wuZXuaF3dREdAYIR27kPJIWAs8r6C3DUKJgcA53IMkYFHcI26KysEg80XPw3dCeezy6i6yXZ+8nuYcBSQjZRx5KZfEedbyBQFjSvFLh4k2damLqeR4jJOaCyWc649pFHo8cfc17ylJzwBcY8QO0qusiy0S97o1aL2iK3Wf9Z8R

master:~ # ceph orch status
Backend: cephadm
Available: True

master:~ # ceph cephadm clear-key

master:~ # ceph orch status
Backend: cephadm
Available: False (SSH keys not set. Use `ceph cephadm set-priv-key` and `ceph cephadm set-pub-key` or `ceph cephadm generate-key`)

master:~ # ceph cephadm generate-key

master:~ # ceph orch status
Backend: cephadm
Available: True

@sebastian-philipp
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants