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

Ceph disk lit 2 #791

Merged
merged 2 commits into from
Sep 28, 2017
Merged

Ceph disk lit 2 #791

merged 2 commits into from
Sep 28, 2017

Conversation

leseb
Copy link
Member

@leseb leseb commented Sep 25, 2017

@leseb leseb force-pushed the ceph-disk-lit-2 branch 3 times, most recently from 3760062 to 314569f Compare September 27, 2017 22:33
@leseb
Copy link
Member Author

leseb commented Sep 27, 2017

needs ceph/ceph-ansible#1956

@leseb leseb changed the title [skip ci] DNM: Ceph disk lit 2 Ceph disk lit 2 Sep 27, 2017
@leseb
Copy link
Member Author

leseb commented Sep 27, 2017

jenkins test this please

@leseb
Copy link
Member Author

leseb commented Sep 28, 2017

jenkins test ceph_ansible-luminous-centos7-dmcrypt_journal_collocation

@leseb leseb force-pushed the ceph-disk-lit-2 branch 2 times, most recently from 7c99d4c to 5466f0b Compare September 28, 2017 19:50
@leseb
Copy link
Member Author

leseb commented Sep 28, 2017

jenkins test ceph_ansible-luminous-xenial-dmcrypt_journal_collocation

ceph-disk list output is really hazardous, so we now have a simpler
scenario to detect journal, block.wal, block.db. It's meant to be used
by ceph-ansible (or other tools) that needs to run the osd activation
stage. For this, it needs to know some variables about the OSD itself.

Signed-off-by: Sébastien Han <seb@redhat.com>
We now use the tool from inside the code. Avoiding painful/unclear
awk/sed/pipe expressions.

Signed-off-by: Sébastien Han <seb@redhat.com>
@leseb leseb merged commit 2832914 into master Sep 28, 2017
@leseb leseb deleted the ceph-disk-lit-2 branch September 28, 2017 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant