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: ceph-disk --dmcrypt create must not require admin key #12033

Merged
1 commit merged into from Dec 6, 2016
Merged

jewel: ceph-disk --dmcrypt create must not require admin key #12033

1 commit merged into from Dec 6, 2016

Conversation

ghost
Copy link

@ghost ghost commented Nov 16, 2016

ceph-disk --dmcrypt needs to put a config-key and authorize
the OSD to get it back. The corresponding permissions are
added to the bootstrap-osd profile in the monitor.

When preparing the OSD lockbox, use the bootstrap-osd profile instead of
implicitly requiring admin permissions to perform the initial config-key
and auth get-or-create operations.

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

Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit c1c6224)
@ghost ghost self-assigned this Nov 16, 2016
@ghost ghost added this to the jewel milestone Nov 16, 2016
@ghost ghost added bug-fix core labels Nov 16, 2016
@ghost ghost changed the base branch from jewel to jewel-next November 16, 2016 23:52
ghost pushed a commit that referenced this pull request Nov 23, 2016
… require admin key

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost
Copy link
Author

ghost commented Dec 5, 2016

It passed the rados (http://tracker.ceph.com/issues/17851#note-12) suite. It also passed the upgrade/jewel-x and upgrade/hammer-x (http://tracker.ceph.com/issues/17851#note-15) suites. Note that it won't be included in 10.2.4, reason why it targets jewel-next.

ghost pushed a commit that referenced this pull request Dec 5, 2016
… require admin key

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost merged commit afda23a into ceph:jewel-next Dec 6, 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
1 participant