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
mgr: misc minor changes #11386
mgr: misc minor changes #11386
Conversation
ea80ce6
to
6b01eef
Compare
@jcsp Can you take a look?(Just some cleanups, simple and easy) Thanks! |
retest this please |
6b01eef
to
20f6615
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All looks good, please could you just add a Fixes line in the commit where I commented.
@@ -112,8 +112,7 @@ void MgrStandby::send_beacon() | |||
available); | |||
|
|||
monc->send_mon_message(m); | |||
// TODO configure period | |||
timer.add_event_after(5, new C_StdFunction( | |||
timer.add_event_after(g_conf->mgr_beacon_period, new C_StdFunction( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please could you add "Fixes: http://tracker.ceph.com/issues/17455" to the commit message
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jcsp Done. Thanks for the review.
Fixes: http://tracker.ceph.com/issues/17455 Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Which can fail due to missing keyring. Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
20f6615
to
ce6b190
Compare
@jcsp Ping |
No description provided.