Skip to content

squid: qa/standalone/scrub: remove TEST_recovery_scrub_2#60287

Merged
ronen-fr merged 1 commit intoceph:squidfrom
ronen-fr:wip-rf-60198-squid
Oct 15, 2024
Merged

squid: qa/standalone/scrub: remove TEST_recovery_scrub_2#60287
ronen-fr merged 1 commit intoceph:squidfrom
ronen-fr:wip-rf-60198-squid

Conversation

@ronen-fr
Copy link
Contributor

That test does no longer match the actual requirements and implementation of scrubbing.

Backport of #60198.

Original tracker: https://tracker.ceph.com/issues/50245
Backport tracker: https://tracker.ceph.com/issues/68403

(cherry picked from commit 0c4028a)

That test does no longer match the actual requirements and
implementation of scrubbing.
It was already deactivated in
ceph#59590. Here - it is
fully removed, mainly for the sake of backporting.

Fixes (original): https://tracker.ceph.com/issues/50245
Fixes (Squid backport): https://tracker.ceph.com/issues/68403

(cherry picked from commit 0c4028a)
Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
@ronen-fr ronen-fr requested a review from a team as a code owner October 13, 2024 16:12
@github-actions github-actions bot added this to the squid milestone Oct 13, 2024
@ronen-fr
Copy link
Contributor Author

jenkins test api

@ronen-fr ronen-fr added the tests label Oct 14, 2024
@ronen-fr ronen-fr requested a review from rzarzynski October 15, 2024 08:34
@ronen-fr
Copy link
Contributor Author

As this changes only one file - only one specific Teuthology test - there is no point in running Teuthology. I plan to merge as is. The only risk is that that one test would continue to fail.

@ronen-fr ronen-fr merged commit f6b098d into ceph:squid Oct 15, 2024
@ronen-fr ronen-fr deleted the wip-rf-60198-squid branch October 15, 2024 11:26
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.

2 participants