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

Add ceph-defaults to galaxy role breakout script #1740

Merged
merged 1 commit into from
Aug 3, 2017

Conversation

logan2211
Copy link
Contributor

@logan2211 logan2211 commented Aug 3, 2017

The ceph/ansible-ceph-defaults repo will need to be created.

This is needed because downstream projects that consume ceph-ansible, such as openstack-ansible, are unable to do so due to #1737 until there is a ceph-defaults breakout role created.

@leseb
Copy link
Member

leseb commented Aug 3, 2017

@logan2211 good catch thanks!

@leseb leseb merged commit 0d5d2db into ceph:master Aug 3, 2017
@logan2211 logan2211 deleted the ceph-defaults-breakout branch August 3, 2017 14:16
@logan2211
Copy link
Contributor Author

@leseb don't forget https://github.com/ceph/ansible-ceph-defaults needs to be created and populated now ;) eager to get this done so we can get https://review.openstack.org/#/c/490476/ moving forward

openstack-gerrit pushed a commit to openstack/openstack-ansible that referenced this pull request Aug 3, 2017
A massive refactor of ceph-ansible merged about 18 hours ago in
ceph/ceph-ansible#1727, which breaks our
ceph gate. A patch already exists[1] which will adjust our ceph
deployment according to the upstream changes, however there is some
work to be done upstream[2] before our fix can merge. Until then,
to keep the OSA master gate unblocked, we should pin the ceph-ansible
roles immediately prior to the refactor commits.

[1] https://review.openstack.org/#/c/490476/
[2] ceph/ceph-ansible#1740

Change-Id: Ie96984d207163404bb7f98cc10aa02a3035a362c
@leseb
Copy link
Member

leseb commented Aug 4, 2017

@logan2211 done! https://github.com/ceph/ansible-ceph-defaults
And it's in the galaxy too!

@logan2211
Copy link
Contributor Author

Thank you!

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.

None yet

3 participants