orchestrator icon indicating copy to clipboard operation
orchestrator copied to clipboard
verified publisher icon percona

Handle failures in GracefulMasterTakeover

Open o-fedorov opened this issue 1 year ago • 7 comments

Related issue: https://github.com/percona/orchestrator/issues/43

Description

  1. If GracefulMasterTakeover fails, reset read_only for master.
  2. Introduce PostUnsuccessfulGracefulTakeoverProcesses to allow running more sophisticated recovery.
  3. Add a related system test.

o-fedorov avatar Jul 03 '24 20:07 o-fedorov

@kamil-holubicki, could I please ask you to take a look at this PR?

o-fedorov avatar Jul 04 '24 15:07 o-fedorov

Hello @kamil-holubicki. Hope you are doing well. Could you please share any feedback regarding this PR?

o-fedorov avatar Jul 11 '24 16:07 o-fedorov

@egegunes, @fabio-silva, @kamil-holubicki, @igroene, could I please ask you for a feedback regarding this PR?

o-fedorov avatar Jul 16 '24 15:07 o-fedorov

Unfortunately, we do not prioritize feature requests or code submissions with our limited bandwidth for Orchestrator as our focus is on more widely impacting issues that affect the majority of the Percona user base. I understand there are discussions underway between our sales team and your leadership that might allow us to one-day take this on.

hulyav avatar Jul 17 '24 14:07 hulyav

Hi @hulyav. Thank you for your response. I’d like to highlight that my team has already fixed this particular bug on our side, and merging the fix to the upstream would benefit the community, since there can be other users that face this issue. Also, it would be nice to merge this pull request sooner rather than later to prevent the codebases divergence.

o-fedorov avatar Jul 17 '24 15:07 o-fedorov

Hello @hulyav and @kamil-holubicki . It is already 4 month since this PR was created. I have a feeling that it eventually will not be accepted. To get rid of the uncertainty about this change, should I close this PR and the corresponding issue as rejected?

o-fedorov avatar Nov 01 '24 15:11 o-fedorov

Thanks for submitting the patch — we appreciate the effort. While we do maintain a fork of Orchestrator for use in our own products, we aren’t the upstream maintainers and generally don’t make changes unless it addresses an issue we’re actively working on or a customer has commissioned the fix. That said, you're welcome to keep the PR open or close it at your discretion — totally your call.

hulyav avatar May 13 '25 05:05 hulyav