Skip to content
This repository has been archived by the owner on Feb 18, 2025. It is now read-only.

Recovery processes ending with "&" are executed asynchronously #968

Merged
merged 3 commits into from
Aug 26, 2019

Conversation

shlomi-noach
Copy link
Collaborator

This applies to all recovery processes:

  • PreGracefulTakeoverProcesses
  • PreFailoverProcesses
  • PostMasterFailoverProcesses
  • PostIntermediateMasterFailoverProcesses
  • PostFailoverProcesses
  • PostUnsuccessfulFailoverProcesses
  • PostGracefulTakeoverProcesses

A failure of an async process has no effect; the error is logged, but the rest of the processes continue to run and the flow proceeds.

Also: PreGracefulTakeoverProcesses now gets {successorHost}, {successorPort} information, since with graceful takeover we pre-select designated instance.

@shlomi-noach shlomi-noach had a problem deploying to production/mysqlcluster=conductor August 26, 2019 09:47 Failure
@shlomi-noach shlomi-noach temporarily deployed to production/mysql_cluster=conductor August 26, 2019 09:48 Inactive
@shlomi-noach shlomi-noach had a problem deploying to production/mysqlcluster=conductor August 26, 2019 10:10 Failure
@shlomi-noach shlomi-noach had a problem deploying to production/mysql_cluster=conductor August 26, 2019 10:11 Failure
@shlomi-noach shlomi-noach had a problem deploying to production/mysql_cluster=conductor August 26, 2019 10:11 Failure
@shlomi-noach shlomi-noach temporarily deployed to production/mysql_cluster=conductor August 26, 2019 10:13 Inactive
@shlomi-noach shlomi-noach temporarily deployed to production/mysql_cluster=conductor August 26, 2019 10:21 Inactive
@shlomi-noach
Copy link
Collaborator Author

Tests well in production

@shlomi-noach shlomi-noach merged commit 420dad3 into master Aug 26, 2019
@shlomi-noach shlomi-noach deleted the async-recovery-processes branch August 26, 2019 11:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant