Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[wip,dnm,dnr] migration: prototype long-running migrations orchestrator #54903

Closed

Conversation

irfansharif
Copy link
Contributor

Don't look. This will be the thing that will orchestrates all long running migrations.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@irfansharif irfansharif force-pushed the 200928.lrm-orchestrator branch from 0b1bf26 to 2c239a1 Compare September 29, 2020 04:25
craig bot pushed a commit that referenced this pull request Oct 14, 2020
54936: kvserver,rpc: set the stage for maintaining a local blocklist of permanently removed nodes r=irfansharif a=tbg

As part of long-running migrations (#54903) we need to make sure, to the best of our abilities, that decommissioned nodes can not return to the cluster. We will achieve this by persisting this knowledge locally (in an eventually consistent way, via gossip) and ading checks in the RPC layer.

This PR stops short of adding the actual storage but sets the stage for doing so.

First seven commits are from another PR and should be ignored here.

Co-authored-by: Tobias Grieger <[email protected]>
This callback will be called after validating a `SET CLUSTER SETTING
version` but before executing it. It will be used in future commits to
execute long-running migrations.

Release note: None
@irfansharif irfansharif force-pushed the 200928.lrm-orchestrator branch from 2c239a1 to 021054e Compare October 21, 2020 21:13
@irfansharif irfansharif requested a review from a team as a code owner October 21, 2020 21:13
@irfansharif irfansharif removed the request for review from a team October 21, 2020 21:13
@irfansharif irfansharif force-pushed the 200928.lrm-orchestrator branch from 021054e to ddd770b Compare October 22, 2020 03:24
We'll depend on it in a future commit.

Release note: None
This request will be fleshed out and used in future commits that
introduce the long running migration orchestrator process.

Release note: None
@irfansharif irfansharif force-pushed the 200928.lrm-orchestrator branch 5 times, most recently from 641b54f to 366d5d9 Compare October 28, 2020 20:52
@irfansharif irfansharif requested a review from a team as a code owner October 28, 2020 20:52
@irfansharif irfansharif force-pushed the 200928.lrm-orchestrator branch 3 times, most recently from b570253 to cd533d7 Compare October 29, 2020 14:13
@irfansharif
Copy link
Contributor Author

Subsumed by #56107.

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.

2 participants