jewel: mon: MonmapMonitor: return success when monitor will be removed #12267

Merged
merged 2 commits into from Dec 2, 2016

Projects

None yet

2 participants

jecluis added some commits Nov 2, 2016
@jecluis @ldachary jecluis mon: MonmapMonitor: return success when monitor will be removed
Fixes: http://tracker.ceph.com/issues/17725

Signed-off-by: Joao Eduardo Luis <joao@suse.de>
(cherry picked from commit c9d46cf)
25f1b39
@jecluis @ldachary jecluis mon: MonmapMonitor: drop unnecessary 'goto' statements
Signed-off-by: Joao Eduardo Luis <joao@suse.de>
(cherry picked from commit 20dcb59)
00de014
@dachary dachary added this to the jewel milestone Dec 1, 2016
@dachary dachary self-assigned this Dec 1, 2016
@dachary
Member
dachary commented Dec 1, 2016 edited

This is intentionally a duplicate of #11999

@dachary
Member
dachary commented Dec 2, 2016

@smithfarm rewrote this PR message history to clear the confusion. http://tracker.ceph.com/issues/17784 was restored to its previous state and http://tracker.ceph.com/issues/17904 updated to reflect the duplicate. Thanks a lot for noticing the mistake.

@dachary
Member
dachary commented Dec 2, 2016

The only make check error is ERROR: flake8: commands failed which is expected. Merging on behalf of @athanatos

@dachary dachary merged commit b168c26 into ceph:jewel Dec 2, 2016

2 of 3 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
@theanalyst theanalyst changed the title from jewel: Error EINVAL: removing mon.a at 172.21.15.16:6789/0, there will be 1 monitors to jewel: mon: MonmapMonitor: return success when monitor will be removed Dec 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment