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

mon/AuthMonitor: generate bootstrap-mgr key on upgrade #16395

Merged
merged 1 commit into from Jul 19, 2017

Conversation

Projects
None yet
5 participants
@jecluis
Member

jecluis commented Jul 18, 2017

We were adding the entity, but no key. By generating a key, we avoid
making things really weird.

Signed-off-by: Joao Eduardo Luis joao@suse.de

@jecluis jecluis added this to the luminous milestone Jul 18, 2017

@jecluis jecluis requested review from tchaikov and jcsp Jul 18, 2017

@liewegas liewegas added the needs-qa label Jul 18, 2017

mon/AuthMonitor: generate bootstrap-mgr key on upgrade
We were adding the entity, but no key. By generating a key, we avoid
making things really weird.

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

Signed-off-by: Joao Eduardo Luis <joao@suse.de>
@yuriw

This comment has been minimized.

Show comment
Hide comment
@yuriw

yuriw Jul 18, 2017

Contributor

test this please

Contributor

yuriw commented Jul 18, 2017

test this please

@liewegas liewegas merged commit 798e46b into ceph:master Jul 19, 2017

3 of 4 checks passed

make check (arm64) make check failed
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
@vasukulkarni

This comment has been minimized.

Show comment
Hide comment
@vasukulkarni

vasukulkarni Aug 2, 2017

Member

Was this backported to recent Luminous 12.1.2 released today? I dont see the tracker has backport label?

Member

vasukulkarni commented Aug 2, 2017

Was this backported to recent Luminous 12.1.2 released today? I dont see the tracker has backport label?

@vasukulkarni

This comment has been minimized.

Show comment
Hide comment
@vasukulkarni
Member

vasukulkarni commented Aug 2, 2017

@theanalyst

This comment has been minimized.

Show comment
Hide comment
@theanalyst

theanalyst Aug 2, 2017

Member

@vasukulkarni this is a part of 12.1.2, we'll start backports only after 12.2.0 has been released

Member

theanalyst commented Aug 2, 2017

@vasukulkarni this is a part of 12.1.2, we'll start backports only after 12.2.0 has been released

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