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: rgw: add_zone only clears master_zone if --master=false #9327

Merged
1 commit merged into from Jun 29, 2016

Conversation

cbodley
Copy link
Contributor

@cbodley cbodley commented May 25, 2016

@cbodley cbodley added this to the jewel milestone May 25, 2016
@ghost ghost changed the title [jewel] rgw: add_zone only clears master_zone if --master=false jewel: rgw: add_zone only clears master_zone if --master=false May 31, 2016
@ghost ghost changed the title jewel: rgw: add_zone only clears master_zone if --master=false DNM: jewel: rgw: add_zone only clears master_zone if --master=false May 31, 2016
@ghost
Copy link

ghost commented May 31, 2016

@cbodley could you please repush with cherry-pick -x so that the commit has a reference to the corresponding master commit ?

@ghost ghost self-assigned this May 31, 2016
Fixes: http://tracker.ceph.com/issues/15901

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 40c4d15)
@cbodley cbodley force-pushed the wip-rgw-zone-modify-master-jewel branch from f9d9b67 to 933fdef Compare May 31, 2016 13:20
@cbodley
Copy link
Contributor Author

cbodley commented May 31, 2016

@dachary done, thank you

@ghost ghost changed the title DNM: jewel: rgw: add_zone only clears master_zone if --master=false jewel: rgw: add_zone only clears master_zone if --master=false May 31, 2016
@ghost
Copy link

ghost commented May 31, 2016

@cbodley thanks !

@Abhishekvrshny
Copy link

test this please

ghost pushed a commit that referenced this pull request Jun 16, 2016
@ghost
Copy link

ghost commented Jun 29, 2016

It passed the rgw suite http://tracker.ceph.com/issues/16344#note-6 (except for one failure which I believe to be unrelated).

@ghost ghost merged commit 980abcf into ceph:jewel Jun 29, 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
2 participants