Skip to content

quincy: osd/scrub: do not start scrubbing if the PG is snap-trimming#46498

Merged
yuriw merged 1 commit intoceph:quincyfrom
ronen-fr:wip-rf-46440-quincy
Jul 20, 2022
Merged

quincy: osd/scrub: do not start scrubbing if the PG is snap-trimming#46498
yuriw merged 1 commit intoceph:quincyfrom
ronen-fr:wip-rf-46440-quincy

Conversation

@ronen-fr
Copy link
Contributor

@ronen-fr ronen-fr commented Jun 2, 2022

Both 'snap-trim' and 'snaptrim-wait' PG states now prevent
scrub from starting.

Background:
A PG should not be scrubbed and trimmed concurrently. Unlike
write operations, snap trimming does not verify that a targeted
object is not in the executing scrub's chunk.
The trimmer always checked for active scrubs before starting. The
scrubber - did not. This PR fixes that omission.

Fixes: https://tracker.ceph.com/issues/49525

Signed-off-by: Ronen Friedman rfriedma@redhat.com
(cherry picked from commit 51816cd)

Both 'snap-trim' and 'snaptrim-wait' PG states now prevent
scrub from starting.

Background:
A PG should not be scrubbed and trimmed concurrently. Unlike
write operations, snap trimming does not verify that a targeted
object is not in the executing scrub's chunk.
The trimmer always checked for active scrubs before starting. The
scrubber - did not. This PR fixes that omission.

Fixes: https://tracker.ceph.com/issues/49525

Signed-off-by: Ronen Friedman <rfriedma@redhat.com>
(cherry picked from commit 51816cd)
@github-actions github-actions bot added the core label Jun 2, 2022
@github-actions github-actions bot added this to the quincy milestone Jun 2, 2022
@ronen-fr ronen-fr requested review from ljflores and neha-ojha June 2, 2022 11:43
@ronen-fr ronen-fr added the DNM label Jun 2, 2022
@ronen-fr
Copy link
Contributor Author

ronen-fr commented Jun 2, 2022

I marked it DNM, as I would rather have the 'main' version collect some track record before merging the backport.

@ronen-fr ronen-fr added needs-qa and removed DNM labels Jun 20, 2022
@neha-ojha
Copy link
Member

jenkins test make check

@rzarzynski
Copy link
Contributor

Main PR: #46440.

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.

4 participants