Project

General

Profile

Bug #12210

register_new_pgs() should check ruleno instead of its index

Added by Kefu Chai over 3 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Category:
-
Target version:
-
Start date:
07/03/2015
Due date:
% Done:

0%

Source:
Community (dev)
Tags:
Backport:
hammer,firefly
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:

Description

after a rule is removed, and pg num is increased, "ceph -s" fails to update the pg num accordingly. in other words, the monitor is not updated.


Related issues

Copied to Ceph - Backport #12395: register_new_pgs() should check ruleno instead of its index Resolved 07/03/2015
Copied to Ceph - Backport #12396: register_new_pgs() should check ruleno instead of its index Resolved 07/03/2015

Associated revisions

Revision 49879339 (diff)
Added by Xinze Chi over 3 years ago

mon/PGMonitor: bug fix pg monitor get crush rule

when some rules have been deleted before, the index in array of crush->rules
is not always equals to crush_ruleset of pool.

Fixes: #12210
Reported-by: Ning Yao <>
Signed-off-by: Xinze Chi <>

Revision 43a72e47 (diff)
Added by Xinze Chi over 3 years ago

mon/PGMonitor: bug fix pg monitor get crush rule

when some rules have been deleted before, the index in array of crush->rules
is not always equals to crush_ruleset of pool.

Fixes: #12210
Reported-by: Ning Yao <>
Signed-off-by: Xinze Chi <>
(cherry picked from commit 498793393c81c0a8e37911237969fba495a3a183)

Revision 5445461b (diff)
Added by Xinze Chi over 3 years ago

mon/PGMonitor: bug fix pg monitor get crush rule

when some rules have been deleted before, the index in array of crush->rules
is not always equals to crush_ruleset of pool.

Fixes: #12210
Reported-by: Ning Yao <>
Signed-off-by: Xinze Chi <>
(cherry picked from commit 498793393c81c0a8e37911237969fba495a3a183)

History

#2 Updated by Kefu Chai over 3 years ago

  • Priority changed from Normal to Urgent

#3 Updated by Kefu Chai over 3 years ago

  • Description updated (diff)

#4 Updated by Sage Weil over 3 years ago

  • Status changed from Need Review to Pending Backport

#5 Updated by Nathan Cutler over 3 years ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF