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

osd/PG: make non-empty PastIntervals non-fatal #15639

Merged
merged 1 commit into from Jun 15, 2017

Conversation

Projects
None yet
3 participants
@liewegas
Member

liewegas commented Jun 12, 2017

Yes, we should not have past intervals in this case. However, we don't
need to crash the OSD; the ERR in clog is sufficient to fail the teuthology
run, and users in production don't actually care.

See: http://tracker.ceph.com/issues/20167
Signed-off-by: Sage Weil sage@redhat.com

osd/PG: make non-empty PastIntervals non-fatal
Yes, we should not have past intervals in this case.  However, we don't
need to crash the OSD; the ERR in clog is sufficient to fail the teuthology
run, and users in production don't actually care.

See: http://tracker.ceph.com/issues/20167
Signed-off-by: Sage Weil <sage@redhat.com>

@liewegas liewegas requested a review from tchaikov Jun 12, 2017

@tchaikov tchaikov added the needs-qa label Jun 13, 2017

@liewegas liewegas merged commit d17dbaf into ceph:master Jun 15, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details

@liewegas liewegas deleted the liewegas:wip-20167 branch Jun 15, 2017

zealoussnow added a commit to zealoussnow/ceph that referenced this pull request Oct 23, 2017

osd/pg: make non-empty PastIntervals non-fatal
see details ceph#15639

Signed-off-by: Leo Zhang <nguzcf@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment