Skip to content

pacific: Revert "osd: Try other PGs when reservation failures occur"#40195

Merged
liewegas merged 2 commits intoceph:pacificfrom
neha-ojha:wip-revert-40077-pacific
Mar 18, 2021
Merged

pacific: Revert "osd: Try other PGs when reservation failures occur"#40195
liewegas merged 2 commits intoceph:pacificfrom
neha-ojha:wip-revert-40077-pacific

Conversation

@neha-ojha
Copy link
Member

Revert #40077 due to https://tracker.ceph.com/issues/49868

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

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 api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

This reverts commit 6f65539.

Signed-off-by: Neha Ojha <nojha@redhat.com>
This reverts commit e0ed012.

Signed-off-by: Neha Ojha <nojha@redhat.com>
@github-actions github-actions bot added the core label Mar 17, 2021
@github-actions github-actions bot added this to the pacific milestone Mar 17, 2021
@neha-ojha neha-ojha changed the title pacific: Revert "osd: Try other PGs when reservation failures occur pacific: Revert "osd: Try other PGs when reservation failures occur" Mar 17, 2021
@neha-ojha neha-ojha requested review from dzafman and jdurgin March 17, 2021 18:31
Copy link
Member

@jdurgin jdurgin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sounds good since pacific is so close to release. we can backport later once we work out the bugs.

@liewegas liewegas merged commit 21b88e4 into ceph:pacific Mar 18, 2021
@neha-ojha
Copy link
Member Author

@neha-ojha neha-ojha deleted the wip-revert-40077-pacific branch March 18, 2021 15:35
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.

3 participants