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 cluster must default to ceph #16870

Merged
merged 1 commit into from Aug 24, 2017

Conversation

Projects
None yet
3 participants
@liewegas
Member

liewegas commented Aug 7, 2017

ceph-disk: dmcrypt cluster must default to ceph
If ceph_fsid is not found, which is the case for legacy dmcrypted OSD,
the cluster must default to ceph, as it was before support for non
standard cluster names was introduced.

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

Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit ed22159)
@liewegas

This comment has been minimized.

Show comment
Hide comment
@liewegas

liewegas Aug 7, 2017

Member

#16859 commit was based on kraken, not jewel, and targeted kraken branch

Member

liewegas commented Aug 7, 2017

#16859 commit was based on kraken, not jewel, and targeted kraken branch

@ghost ghost changed the title from ceph-disk: dmcrypt cluster must default to ceph to jewel: ceph-disk: dmcrypt cluster must default to ceph Aug 7, 2017

@ghost

ghost approved these changes Aug 7, 2017

normalized for backport conventions

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Aug 9, 2017

pushed to ceph-ci for testing

ghost commented Aug 9, 2017

pushed to ceph-ci for testing

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Aug 9, 2017

teuthology-suite -k distro --verbose --suite ceph-disk --ceph wip-20929-jewel --machine-type vp

ghost commented Aug 9, 2017

teuthology-suite -k distro --verbose --suite ceph-disk --ceph wip-20929-jewel --machine-type vp

@liewegas liewegas merged commit 55cd3d9 into ceph:jewel Aug 24, 2017

3 checks passed

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

@liewegas liewegas deleted the liewegas:wip-20929-jewel branch Aug 24, 2017

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