Skip to content

quincy: Revert PR 47901#48104

Merged
yuriw merged 2 commits intoceph:quincyfrom
ljflores:wip-quincy-revert-pr-47901
Sep 15, 2022
Merged

quincy: Revert PR 47901#48104
yuriw merged 2 commits intoceph:quincyfrom
ljflores:wip-quincy-revert-pr-47901

Conversation

@ljflores
Copy link
Member

@ljflores ljflores commented Sep 14, 2022

When testing the Quincy RC for 17.2.4, we discovered this failure:

Description: rados/thrash-erasure-code/{ceph clusters/{fixed-2 openstack} fast/fast mon_election/classic msgr-failures/osd-delay objectstore/bluestore-comp-zstd rados recovery-overrides/{more-async-partial-recovery} supported-random-distro$/{rhel_8} thrashers/default thrashosds-health workloads/ec-rados-plugin=clay-k=4-m=2}

Dead job: http://pulpito.front.sepia.ceph.com/yuriw-2022-09-10_14:05:53-rados-quincy-release-distro-default-smithi/7024541/

The failure is intermittent, which explains why it did not show up in #47901 (comment), nor in the rerun of @yuriw's validation run.

We found that running the affected test 50 times is a reliable reproducer, as seen by:
http://pulpito.front.sepia.ceph.com/lflores-2022-09-14_15:11:39-rados-quincy-release-distro-default-smithi/

We suspected #47901 to be the culprit, so we reran the reproducer with those two commits reverted:
http://pulpito.front.sepia.ceph.com/lflores-2022-09-14_17:02:02-rados-wip-lflores-testing-quincy-release-distro-default-smithi/

All 50 tests passed with the commits from #47901 reverted.

So far, this bug has only been observed in Quincy (and once on Pacific, but the affected test branch included #47899, which hasn't been merged yet). It's possible that this bug exists in main since #44499 has merged.

Fixes: https://tracker.ceph.com/issues/57546
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

@ljflores ljflores requested a review from a team as a code owner September 14, 2022 21:15
@github-actions github-actions bot added the core label Sep 14, 2022
@github-actions github-actions bot added this to the quincy milestone Sep 14, 2022
@ljflores ljflores changed the title Wip quincy revert pr 47901 quincy: Revert PR 47901 Sep 14, 2022
@ljflores ljflores requested review from neha-ojha and yuriw September 14, 2022 21:18
@ljflores
Copy link
Member Author

@ljflores
Copy link
Member Author

ljflores commented Sep 15, 2022

@yuriw all green! RADOS approved.

@yuriw yuriw merged commit f1c897c into ceph:quincy Sep 15, 2022
@ljflores ljflores deleted the wip-quincy-revert-pr-47901 branch September 15, 2022 14:34
rzarzynski added a commit to rzarzynski/ceph that referenced this pull request Dec 21, 2022
This reverts commit 9337fa6.

It's a `main`'s counterpart of quincy's PRs ceph#48104 and ceph#49304.

Fixes: https://tracker.ceph.com/issues/57546
Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
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