Skip to content

Conversation

@kvoli
Copy link
Contributor

@kvoli kvoli commented Sep 4, 2024

When server.decommissioning.periodic_nudge_interval is set to a non-zero value, each node will periodically nudge the ranges with a decommissioning replica the node holds a lease for.

When there are no ranges remaining on the decommissioning node, which are leased on a node, the nudger will stop.

Resolves: #130085
Release note: None

@blathers-crl
Copy link

blathers-crl bot commented Sep 4, 2024

It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR?

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@kvoli kvoli force-pushed the 240904.enqueue-decommissioning-replicas branch 2 times, most recently from 5750d1a to 3b4de5e Compare September 4, 2024 15:34
@kvoli kvoli self-assigned this Sep 4, 2024
@kvoli kvoli force-pushed the 240904.enqueue-decommissioning-replicas branch 2 times, most recently from a8ea6e2 to a19a7f9 Compare September 4, 2024 17:59
When `server.decommissioning.periodic_nudge_interval` is set to a
non-zero value, each node will periodically nudge the ranges with a
decommissioning replica the node holds a lease for.

When there are no ranges remaining on the decommissioning node, which
are leased on a node, the nudger will stop.

The setting is disabled by default (=0).

Resolves: cockroachdb#130085
Release note: None
@kvoli kvoli force-pushed the 240904.enqueue-decommissioning-replicas branch from a19a7f9 to 37ba3fd Compare September 4, 2024 18:45
@kvoli
Copy link
Contributor Author

kvoli commented Sep 6, 2024

Closing in favor of #130117.

@kvoli kvoli closed this Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

server: add configurable decommission nudger

2 participants