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
mon: PG Monitor should report waiting for backfill #7398
Conversation
@@ -2061,7 +2061,7 @@ void PGMonitor::get_health(list<pair<health_status_t,string> >& summary, | |||
if (p->first & PG_STATE_INCOMPLETE) | |||
note["incomplete"] += p->second; | |||
if (p->first & PG_STATE_BACKFILL_WAIT) | |||
note["backfill"] += p->second; | |||
note["waiting for backfill"] += p->second; |
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.
should be "backfill-wait" instead? to be consistent with the document.
or, we should be consistent with the naming of "foo_wait", just like "recovery_wait". so we should update both the doc and the code:
- s/backfill-wait/backfill_wait/ in doc
- s/"backfill"/"backfill_wait"/ in the line above
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.
I guess I'll change to backfill_wait here and the doc as well..
@tchaikov updated |
lgtm modulo the typo in commit message. |
Instead of backfill as it is actually waiting for backfill instead of actually backfilling Fixes: ceph#12744 Reported-by: Dan Mick <dmick@redhat.com> Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
Since the state reported is `backfilling` and `backfill_wait` Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
Updated again. Thanks for the patient reviews |
lgtm |
mon: PG Monitor should report waiting for backfill Reviewed-by: Kefu Chai <kchai@redhat.com>
Instead of backfill (as the state is
wait_backfill
)Fixes: #12744