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/balancer: pool-specific optimization support and bug fixes #20154
Conversation
This currently only works for upmap. Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
b4668b0
to
bce4fbf
Compare
src/pybind/mgr/balancer/module.py
Outdated
@@ -675,8 +691,8 @@ def do_crush_compat(self, plan): | |||
overlap[osd] = 1 | |||
visited[osd] = 1 | |||
if len(overlap) > 0: | |||
self.log.err('error: some osds belong to multiple subtrees: %s' % | |||
overlap) | |||
self.log.error('error: some osds belong to multiple subtrees: %s' % |
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.
this has been fixed by #20130 :)
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.
looks good, aside for the dup commit! Thanks!
bce4fbf
to
52e336e
Compare
…Eval Otherwise there might be obsolete pool/root infos etc. Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@xiexingguo shall we backport 52e336e to luminous? IMO, it's a bug fix of balancer mgr plugin. the same applies to #20180. |
@xiexingguo thanks in advance. |
thanks @xiexingguo ! |
No description provided.