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: cleanups #15272

Merged
merged 2 commits into from May 26, 2017

Conversation

Projects
None yet
2 participants
@tchaikov
Contributor

tchaikov commented May 24, 2017

@liewegas i am trying to move some wip-bigbang changes into master sooner. so we (actually you =) can maintain less patches in another branch.

tchaikov added some commits May 20, 2017

mon,mgr: std::move(osd_stat) when possible
osd_stat_t could be a relatively large struct, so use std::move() when
possible

Signed-off-by: Kefu Chai <kchai@redhat.com>
mon/PGMonitor: assert(format_version == 1)
we have been PGMonitor's formart_version has been 1 since v0.67
(dumpling). so let's drop it.

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

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

@liewegas liewegas merged commit 94c834a into ceph:master May 26, 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

@tchaikov tchaikov deleted the tchaikov:wip-mon-cleanups branch May 26, 2017

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