Skip to content
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.

Sign up for GitHub

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

Jump to bottom

crush: ceph osd crush reweight-subtree does not reweight parent node #5374

Merged
6 commits merged into from Aug 30, 2015
Merged

crush: ceph osd crush reweight-subtree does not reweight parent node #5374

6 commits merged into from Aug 30, 2015

Conversation

ghost
Copy link

@ghost ghost commented Jul 28, 2015

@liewegas @ldachary
unittest_crush_wrapper: parse env
1e73753 
Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit 14c60b2)
@liewegas @ldachary
unittest_crush_wrapper: attach buckets to root in adjust_item_weight …
0ccdf34 
…test

Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit a8351eb)
@liewegas @ldachary
unittest_crush_wrapper: test adjust_subtree_weight
d2f31ad 
Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit b461dc2)
@liewegas @ldachary
crush/CrushWrapper: adjust subtree base in adjust_subtree_weight
05fc59b 
We were correctly adjusting all the children, but the subtree root was
not being updated.

Fixes: #11855
Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit 1da96ab)
@liewegas @ldachary
crush/CrushWrapper: return changed from adjust_subtree_weight
0bd4c81 
Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit 551ab2d)
@liewegas @ldachary
crush/CrushWrapper: fix adjust_subtree_weight debug
695f782 
Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit 26be86f)
@ghost ghost self-assigned this Jul 28, 2015
@ghost ghost added this to the hammer milestone Jul 28, 2015
@ghost ghost added bug-fix core labels Jul 28, 2015
@loic-bot
Copy link

SUCCESS: http://jenkins.ceph.dachary.org/job/ceph/6695/

ghost pushed a commit that referenced this pull request Jul 29, 2015
@ldachary
Merge pull request #5374: ceph osd crush reweight-subtree does not re…
34feb84 
…weight parent node

Reviewed-by:
@ghost
Copy link
Author

ghost commented Aug 30, 2015

@liewegas does this backport look good to merge ? It passed a run of the hammer rados suite ( see http://tracker.ceph.com/issues/12701#rados for details ).

@liewegas
Copy link
Member

👍

ghost pushed a commit that referenced this pull request Aug 30, 2015
@ldachary
Merge pull request #5374: ceph osd crush reweight-subtree does not re…
5044bb4 
…weight parent node

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 30, 2015
Merge pull request #5374 from dachary/wip-12487-hammer
d3b160e 
ceph osd crush reweight-subtree does not reweight parent node

Reviewed-by: Sage Weil <sage@redhat.com>
@ghost ghost merged commit d3b160e into ceph:hammer Aug 30, 2015
@ghost ghost changed the title ceph osd crush reweight-subtree does not reweight parent node crush: ceph osd crush reweight-subtree does not reweight parent node Sep 13, 2015
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug-fix core
Projects
None yet
Milestone
hammer
2 participants
@loic-bot @liewegas