Skip to content

octopus: Revert "rocksdb: do not use non-zero recycle_log_file_num setting"#47053

Merged
ljflores merged 1 commit intoceph:octopusfrom
ljflores:wip-fix-55636
Jul 12, 2022
Merged

octopus: Revert "rocksdb: do not use non-zero recycle_log_file_num setting"#47053
ljflores merged 1 commit intoceph:octopusfrom
ljflores:wip-fix-55636

Conversation

@ljflores
Copy link
Member

@ljflores ljflores commented Jul 11, 2022

This reverts commit 6e225bf. It
causes a consistently-reproducible BlueFS bug. See the tracker
for details.

Fixes: https://tracker.ceph.com/issues/55636
Signed-off-by: Laura Flores lflores@redhat.com

Contribution Guidelines

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox
  • jenkins test windows

This reverts commit 6e225bf. It
causes a consistently-reproducible BlueFS bug. See the tracker
for details.

Fixes: https://tracker.ceph.com/issues/55636
Signed-off-by: Laura Flores <lflores@redhat.com>
@ljflores ljflores requested review from a team and ifed01 July 11, 2022 19:16
@ljflores ljflores changed the title Revert "rocksdb: do not use non-zero recycle_log_file_num setting" octopus: Revert "rocksdb: do not use non-zero recycle_log_file_num setting" Jul 11, 2022
@ljflores ljflores added this to the octopus milestone Jul 11, 2022
@ljflores
Copy link
Member Author

ljflores commented Jul 11, 2022

@ifed01 I have created this PR to fix https://tracker.ceph.com/issues/55636 since Octopus is going into the final stages of the point release. If you see this before the point release is finalized, let me know if you have a fix you want to try, and I can close this revert.

@ljflores ljflores requested a review from neha-ojha July 11, 2022 19:23
@aclamk
Copy link
Contributor

aclamk commented Jul 12, 2022

I believe that there is an actual bug in BlueFS. But as intermediary action revert is a best solution.

@ljflores
Copy link
Member Author

Merging this, as it has been tested here: https://pulpito.ceph.com/?branch=wip-yuri7-testing-2022-07-08-1007-octopus

@ljflores ljflores merged commit b9484e1 into ceph:octopus Jul 12, 2022
@ljflores ljflores deleted the wip-fix-55636 branch July 12, 2022 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants