Skip to content

Number of degraded objects in EC pool is wrong when there is OSD down(in)#3726

Closed
guangyy wants to merge 1 commit intoceph:masterfrom
guangyy:wip-4254
Closed

Number of degraded objects in EC pool is wrong when there is OSD down(in)#3726
guangyy wants to merge 1 commit intoceph:masterfrom
guangyy:wip-4254

Conversation

@guangyy
Copy link
Copy Markdown
Contributor

@guangyy guangyy commented Feb 13, 2015

For EC pool, actingset should be used instead of acting to calculate the number of degraded objects.

Signed-off-by: Guang Yang yguang@yahoo-inc.com

@loic-bot
Copy link
Copy Markdown

SUCCESS: the output of run-make-check.sh on centos-centos7 for 89d5ca3 is http://paste2.org/_bYdcIH1d

:octocat: Sent from GH.

@ghost
Copy link
Copy Markdown

ghost commented Feb 13, 2015

@guangyy could you please update the commit message with a short explanation about why it is wrong ?

@ghost ghost added core bug-fix labels Feb 13, 2015
@tchaikov
Copy link
Copy Markdown
Contributor

yes, looks good to me. a short(er) description would help.

@liewegas
Copy link
Copy Markdown
Member

looks right to me!

@liewegas liewegas added this to the hammer milestone Feb 13, 2015
…(in)

With EC pool (crush rule choose indep), when there is an OSD down, the size of the 'acting' list does not change (CRUSH_ITEM_NONE is used to replace the down OSD), in this case, 'actingset' should be used to calculate the degraded objects.

Signed-off-by: Guang Yang <yguang@yahoo-inc.com>
@guangyy
Copy link
Copy Markdown
Contributor Author

guangyy commented Feb 13, 2015

@dachary , updated with a more descriptive commit message.

@ghost
Copy link
Copy Markdown

ghost commented Feb 13, 2015

@guangyy great, thanks. Would you mind opening a pull request targeted to hammer instead of master ? This is a bug fix that needs to go in hammer and it will later be merged in master. While you're at it, could you amend the commit message to start the subject with osd: ? Something like

osd: number of degraded objects in EC pool is wrong when there is OSD down(in)

@loic-bot
Copy link
Copy Markdown

SUCCESS: the output of run-make-check.sh on centos-centos7 for 2af596f is http://paste2.org/_EksgFcvG

:octocat: Sent from GH.

@guangyy
Copy link
Copy Markdown
Contributor Author

guangyy commented Feb 13, 2015

@dachary , I just created one target to hammer - #3729. Thanks!

@guangyy guangyy closed this Feb 13, 2015
@guangyy guangyy deleted the wip-4254 branch February 27, 2015 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants