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: ensure UP is not set for newly-created OSDs #16534

Merged
merged 1 commit into from Jul 26, 2017

Conversation

Projects
None yet
2 participants
@liewegas
Member

liewegas commented Jul 24, 2017

Due to http://tracker.ceph.com/issues/20751 it is possible for the UP
state bit to be set for OSDs without EXISTS. Make sure it is cleared as
needed when the OSD id is reused.

Signed-off-by: Sage Weil sage@redhat.com

mon/OSDMonitor: ensure UP is not set for newly-created OSDs
Due to http://tracker.ceph.com/issues/20751 it is possible for the UP
state bit to be set for OSDs without EXISTS.  Make sure it is cleared as
needed when the OSD id is reused.

Signed-off-by: Sage Weil <sage@redhat.com>

@liewegas liewegas added this to the luminous milestone Jul 24, 2017

@liewegas liewegas requested a review from tchaikov Jul 24, 2017

@liewegas

This comment has been minimized.

Member

liewegas commented Jul 26, 2017

http://pulpito.ceph.com/sage-2017-07-25_20:13:42-rados-wip-sage-testing-distro-basic-smithi/

noisy run due to config regression, but i think this patch is good to go.

@tchaikov tchaikov added needs-qa and removed needs-review labels Jul 26, 2017

@liewegas liewegas merged commit a3b4f08 into ceph:master Jul 26, 2017

3 of 4 checks passed

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

@liewegas liewegas deleted the liewegas:wip-20751 branch Jul 26, 2017

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