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

rgw: fix period update crash #13054

Merged
merged 3 commits into from Jan 30, 2017

Conversation

Projects
None yet
3 participants
@oritwas
Contributor

oritwas commented Jan 22, 2017

@oritwas oritwas requested a review from cbodley Jan 22, 2017

@oritwas oritwas requested a review from yehudasa Jan 22, 2017

@yehudasa

This comment has been minimized.

Member

yehudasa commented Jan 25, 2017

@oritwas can you verify that failure here is unrelevant?

@oritwas

This comment has been minimized.

Contributor

oritwas commented Jan 26, 2017

looks unrelated.
test_mock fails with "unkown file" error

@yehudasa

lgtm

oritwas added some commits Jan 22, 2017

rgw: fix init_zg_from_period when default zone is not set as default
Signed-off-by: Orit Wasserman <owasserm@redhat.com>
rgw: we need to reinit the zonegroup after assignment to avoid invali…
…d cct and store

Fixes: http://tracker.ceph.com/issues/18631
Signed-off-by: Orit Wasserman <owasserm@redhat.com>
rgw: add check for update return value
Signed-off-by: Orit Wasserman <owasserm@redhat.com>

@cbodley cbodley changed the title from Wip rgw period update crash to rgw: fix period update crash Jan 30, 2017

@cbodley cbodley merged commit b4446ac into ceph:master Jan 30, 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