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

radosgw-admin: use zone id when creating a zone #14340

Merged
merged 1 commit into from Apr 6, 2017

Conversation

Projects
None yet
2 participants
@oritwas
Contributor

oritwas commented Apr 5, 2017

Fixes: http://tracker.ceph.com/issues/19498
Signed-off-by: Orit Wasserman owasserm@redhat.com

radosgw-admin: use zone id when creating a zone
Fixes: http://tracker.ceph.com/issues/19498
Signed-off-by: Orit Wasserman <owasserm@redhat.com>
@cbodley

cbodley approved these changes Apr 5, 2017

looks good. i see that RGWZoneParams::create() defaults to exclusive = true, so we'll fail with EEXIST if there's another zone with the same id 👍

@cbodley cbodley merged commit bf46157 into ceph:master Apr 6, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment