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

mon/OSDMonitor: skip prime_pg_temp if mapping is prior to osdmap #14826

Merged
merged 1 commit into from Apr 28, 2017

Conversation

Projects
None yet
5 participants
@tchaikov
Contributor

tchaikov commented Apr 27, 2017

Signed-off-by: Kefu Chai kchai@redhat.com

mon/OSDMonitor: skip prime_pg_temp if mapping is prior to osdmap
Signed-off-by: Kefu Chai <kchai@redhat.com>
@xiexingguo

This comment has been minimized.

Member

xiexingguo commented Apr 27, 2017

lgtm

@xiexingguo

This comment has been minimized.

Member

xiexingguo commented Apr 27, 2017

We(@runsisi together with I) spent whole day to figure it out, sigh...

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Apr 27, 2017

@xiexingguo yeah. but the upside is, we nailed it down before going to production!

@runsisi

This comment has been minimized.

Contributor

runsisi commented Apr 27, 2017

LGTM

@tchaikov tchaikov added the needs-qa label Apr 27, 2017

@liewegas liewegas merged commit d0b2e60 into ceph:master Apr 28, 2017

3 of 4 checks passed

default Build finished.
Details
Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
arm build successfully built on arm
Details

@tchaikov tchaikov deleted the tchaikov:wip-prime-pg-temp branch Apr 29, 2017

Abhishekvrshny pushed a commit to Abhishekvrshny/ceph that referenced this pull request Jul 13, 2017

rgw: reset return code in when iterating over the bucket the objects
Fixes: ceph#14826

Signed-off-by: Orit Wasserman <owasserm@redhat.com>
(cherry picked from commit e23fdcb)
(cherry picked from commit a3003f6)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment