Skip to content

osd/scrub: register for 'osd_max_scrubs' config changes#61184

Merged
ronen-fr merged 1 commit intoceph:mainfrom
ronen-fr:wip-rf-watch-scrubmax
Dec 27, 2024
Merged

osd/scrub: register for 'osd_max_scrubs' config changes#61184
ronen-fr merged 1 commit intoceph:mainfrom
ronen-fr:wip-rf-watch-scrubmax

Conversation

@ronen-fr
Copy link
Contributor

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.

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

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]>
@ronen-fr ronen-fr requested a review from a team as a code owner December 26, 2024 13:11
@github-actions github-actions bot added the core label Dec 26, 2024
@ronen-fr ronen-fr added bug-fix needs-squid-backport PR needs a squid backport and removed core labels Dec 26, 2024
@ronen-fr
Copy link
Contributor Author

jenkins test api

@ronen-fr
Copy link
Contributor Author

Merging based on my Teuthology tests (4 failures - all accounted for)

@ronen-fr ronen-fr merged commit 68d65ab into ceph:main Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants