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

luminous: Revert "PG: add custom_reaction Backfilled and release reservations after backfill #24902

Merged
merged 1 commit into from Nov 5, 2018

Conversation

Projects
None yet
4 participants
@neha-ojha
Member

neha-ojha commented Nov 2, 2018

https://tracker.ceph.com/issues/36678


This is due to recent crashes we have seen here: https://tracker.ceph.com/issues/22902#note-7
We suspect that this PR could have caused them. Reverting until we get to the root cause.

This reverts commit f512f90.

Signed-off-by: Neha Ojha nojha@redhat.com

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug
Revert "PG: add custom_reaction Backfilled and release reservations a…
…fter backfill"

This reverts commit f512f90.

Signed-off-by: Neha Ojha <nojha@redhat.com>

@neha-ojha neha-ojha requested a review from jdurgin Nov 2, 2018

@jdurgin

jdurgin approved these changes Nov 2, 2018

@jdurgin jdurgin added this to the luminous milestone Nov 2, 2018

@jdurgin jdurgin added the needs-qa label Nov 2, 2018

@jdurgin jdurgin changed the title from Revert "PG: add custom_reaction Backfilled and release reservations after backfill to luminous: Revert "PG: add custom_reaction Backfilled and release reservations after backfill Nov 3, 2018

@yuriw

This comment has been minimized.

Contributor

yuriw commented Nov 3, 2018

@dzafman

This comment has been minimized.

Member

dzafman commented Nov 3, 2018

I did a test run with the same revert and a back port for #24889

@yuriw yuriw merged commit 8157642 into ceph:luminous Nov 5, 2018

4 checks passed

Docs: build check OK - docs built
Details
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment