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: fix failed to create bucket if a non-master zonegroup has a single zone #14766

Merged
merged 1 commit into from Jun 9, 2017

Conversation

smithfarm
Copy link
Contributor

…le zone

If a non-master zonegroup has a single zone, the metadata sync thread not running and
the non-master zonegroup can't sync user from master zonegroup,
so we can't create bucket(or other metadata update) in it
because the authenticated user not found in the zone of non-master zonegroup.

Signed-off-by: weiqiaomiao <wei.qiaomiao@zte.com.cn>
(cherry picked from commit 949af79)

Conflicts:
        src/rgw/rgw_rados.cc - retain d32654b
            which was merged after this commit
@smithfarm smithfarm self-assigned this Apr 25, 2017
@smithfarm smithfarm added this to the jewel milestone Apr 25, 2017
@smithfarm smithfarm changed the title jewel: fix failed to create bucket if a non-master zonegroup has a single zone jewel: rgw: fix failed to create bucket if a non-master zonegroup has a single zone Apr 25, 2017
asheplyakov pushed a commit to asheplyakov/ceph that referenced this pull request May 18, 2017
…t if a non-master zonegroup has a single zone

Reviewed-by: Nathan Cutler <ncutler@suse.com>
@theanalyst
Copy link
Member

@yehudasa @cbodley this passed an integration run http://tracker.ceph.com/issues/19538#note-61 do you think this is ready to merge

@cbodley
Copy link
Contributor

cbodley commented Jun 9, 2017

@theanalyst yes please

@smithfarm smithfarm assigned theanalyst and unassigned smithfarm Jun 9, 2017
@theanalyst theanalyst merged commit fd9256b into ceph:jewel Jun 9, 2017
@smithfarm smithfarm deleted the wip-19757-jewel branch June 9, 2017 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants