Skip to content

squid: osd/scrub: register for 'osd_max_scrubs' config changes#61185

Merged
ronen-fr merged 1 commit intoceph:squidfrom
ronen-fr:wip-rf-61184-squid
Jan 7, 2025
Merged

squid: osd/scrub: register for 'osd_max_scrubs' config changes#61185
ronen-fr merged 1 commit intoceph:squidfrom
ronen-fr:wip-rf-61184-squid

Conversation

@ronen-fr
Copy link
Contributor

@ronen-fr ronen-fr commented Dec 26, 2024

Since #55340, the osd_max_scrubs (also) affects the parameters of the async scrub reserver used by the replicas. Thus,
the code must notice and acknowledge changes to this config.

Backport of #61184.

Original tracker: https://tracker.ceph.com/issues/69362
Backport tracker: https://tracker.ceph.com/issues/69374

(cherry picked from commit 31e6bac)

Since ceph#55340, the
osd_max_scrubs (also) affects the parameters of the
async scrub reserver used by the replicas. Thus,
the code must notice and acknowledge changes to this config.

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

Signed-off-by: Ronen Friedman <[email protected]>
(cherry picked from commit 31e6bac)
@github-actions github-actions bot added the core label Dec 26, 2024
@github-actions github-actions bot added this to the squid milestone Dec 26, 2024
@ronen-fr ronen-fr marked this pull request as ready for review December 27, 2024 14:39
@ronen-fr ronen-fr requested a review from a team as a code owner December 27, 2024 14:39
@ronen-fr
Copy link
Contributor Author

ronen-fr commented Jan 7, 2025

@ronen-fr ronen-fr merged commit 1d46039 into ceph:squid Jan 7, 2025
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