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: meta sync thread crash at RGWMetaSyncShardCR #16711

Merged
merged 2 commits into from Sep 10, 2017

Conversation

Projects
None yet
5 participants
@cbodley
Contributor

cbodley commented Jul 31, 2017

fangyuxiangGL and others added some commits Jun 13, 2017

rgw: meta sync thread crash at RGWMetaSyncShardCR
Fixes: http://tracker.ceph.com/issues/20251

Signed-off-by: fang yuxiang fang.yuxiang@eisoo.com
(cherry picked from commit 45877d3)
rgw: lease_stack: use reset method instead of assignment
It seems that the intent of 45877d3 was to
change all instances of "lease_stack = ..." to "lease_stack.reset(...)", but
this one was missed.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
(cherry picked from commit fc425af)

@cbodley cbodley added this to the jewel milestone Jul 31, 2017

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Aug 1, 2017

Contributor

Jenkins re-test this please

Contributor

smithfarm commented Aug 1, 2017

Jenkins re-test this please

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Aug 2, 2017

Contributor

The job died or timeout on unittest_throttle.log

Jenkins re-test this please

Contributor

smithfarm commented Aug 2, 2017

The job died or timeout on unittest_throttle.log

Jenkins re-test this please

@amitkumar50

changes exactly same as backport

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Sep 7, 2017

Contributor

@cbodley This passed an rgw suite at http://tracker.ceph.com/issues/20613#note-53

OK to merge as is, or do we need someone else to review?

Contributor

smithfarm commented Sep 7, 2017

@cbodley This passed an rgw suite at http://tracker.ceph.com/issues/20613#note-53

OK to merge as is, or do we need someone else to review?

@cbodley

This comment has been minimized.

Show comment
Hide comment
@cbodley

cbodley Sep 7, 2017

Contributor

@yehudasa @oritwas can you please review?

Contributor

cbodley commented Sep 7, 2017

@yehudasa @oritwas can you please review?

@cbodley cbodley requested review from yehudasa and oritwas Sep 7, 2017

@smithfarm smithfarm merged commit bcc85d1 into ceph:jewel Sep 10, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details

@cbodley cbodley deleted the cbodley:wip-20346 branch Sep 11, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment