Skip to content

Trigger the election as soon as possible when doing a forced manual failover#1067

Merged
enjoy-binbin merged 1 commit intovalkey-io:unstablefrom
enjoy-binbin:speedup_failover_force
Sep 25, 2024
Merged

Trigger the election as soon as possible when doing a forced manual failover#1067
enjoy-binbin merged 1 commit intovalkey-io:unstablefrom
enjoy-binbin:speedup_failover_force

Conversation

@enjoy-binbin
Copy link
Member

In CLUSTER FAILOVER FORCE case, we will set mf_can_start to
1 and wait for a cron to trigger the election. We can also set
a CLUSTER_TODO_HANDLE_MANUALFAILOVER flag so that we can start
the election as soon as possible instead of waiting for the cron,
so that we won't have a 100ms delay (clusterCron).

…ailover

In CLUSTER FAILOVER FORCE case, we will set mf_can_start to
1 and wait for a cron to trigger the election. We can also set
a CLUSTER_TODO_HANDLE_MANUALFAILOVER flag so that we can start
the election as soon as possible instead of waiting for the cron,
so that we won't have a 100ms delay (clusterCron).

Signed-off-by: Binbin <[email protected]>
@codecov
Copy link

codecov bot commented Sep 24, 2024

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.64%. Comparing base (6ce75cd) to head (8ba4331).
⚠️ Report is 735 commits behind head on unstable.

Additional details and impacted files
@@             Coverage Diff              @@
##           unstable    #1067      +/-   ##
============================================
+ Coverage     70.60%   70.64%   +0.03%     
============================================
  Files           114      114              
  Lines         61677    61678       +1     
============================================
+ Hits          43550    43571      +21     
+ Misses        18127    18107      -20     
Files with missing lines Coverage Δ
src/cluster_legacy.c 86.22% <100.00%> (+0.06%) ⬆️

... and 10 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@PingXie PingXie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@enjoy-binbin enjoy-binbin merged commit 6e02164 into valkey-io:unstable Sep 25, 2024
@enjoy-binbin enjoy-binbin deleted the speedup_failover_force branch September 25, 2024 04:08
@enjoy-binbin enjoy-binbin added the release-notes This issue should get a line item in the release notes label Sep 28, 2024
naglera pushed a commit to naglera/placeholderkv that referenced this pull request Oct 10, 2024
…ailover (valkey-io#1067)

In CLUSTER FAILOVER FORCE case, we will set mf_can_start to
1 and wait for a cron to trigger the election. We can also set a
CLUSTER_TODO_HANDLE_MANUALFAILOVER flag so that we
can start the election as soon as possible instead of waiting for
the cron, so that we won't have a 100ms delay (clusterCron).

Signed-off-by: Binbin <[email protected]>
Signed-off-by: naglera <[email protected]>
@enjoy-binbin enjoy-binbin moved this to 8.0.4 in Valkey 8.0 Jun 17, 2025
enjoy-binbin added a commit to vitarb/valkey that referenced this pull request Jun 17, 2025
…ailover (valkey-io#1067)

In CLUSTER FAILOVER FORCE case, we will set mf_can_start to
1 and wait for a cron to trigger the election. We can also set a
CLUSTER_TODO_HANDLE_MANUALFAILOVER flag so that we
can start the election as soon as possible instead of waiting for
the cron, so that we won't have a 100ms delay (clusterCron).

Signed-off-by: Binbin <[email protected]>
zuiderkwast pushed a commit to vitarb/valkey that referenced this pull request Aug 15, 2025
…ailover (valkey-io#1067)

In CLUSTER FAILOVER FORCE case, we will set mf_can_start to
1 and wait for a cron to trigger the election. We can also set a
CLUSTER_TODO_HANDLE_MANUALFAILOVER flag so that we
can start the election as soon as possible instead of waiting for
the cron, so that we won't have a 100ms delay (clusterCron).

Signed-off-by: Binbin <[email protected]>
zuiderkwast pushed a commit to vitarb/valkey that referenced this pull request Aug 15, 2025
…ailover (valkey-io#1067)

In CLUSTER FAILOVER FORCE case, we will set mf_can_start to
1 and wait for a cron to trigger the election. We can also set a
CLUSTER_TODO_HANDLE_MANUALFAILOVER flag so that we
can start the election as soon as possible instead of waiting for
the cron, so that we won't have a 100ms delay (clusterCron).

Signed-off-by: Binbin <[email protected]>
Signed-off-by: Viktor Söderqvist <[email protected]>
@zuiderkwast zuiderkwast moved this from 8.0.4 to 8.0.5 in Valkey 8.0 Aug 18, 2025
zuiderkwast pushed a commit to vitarb/valkey that referenced this pull request Aug 21, 2025
…ailover (valkey-io#1067)

In CLUSTER FAILOVER FORCE case, we will set mf_can_start to
1 and wait for a cron to trigger the election. We can also set a
CLUSTER_TODO_HANDLE_MANUALFAILOVER flag so that we
can start the election as soon as possible instead of waiting for
the cron, so that we won't have a 100ms delay (clusterCron).

Signed-off-by: Binbin <[email protected]>
Signed-off-by: Viktor Söderqvist <[email protected]>
zuiderkwast pushed a commit that referenced this pull request Aug 22, 2025
…ailover (#1067)

In CLUSTER FAILOVER FORCE case, we will set mf_can_start to
1 and wait for a cron to trigger the election. We can also set a
CLUSTER_TODO_HANDLE_MANUALFAILOVER flag so that we
can start the election as soon as possible instead of waiting for
the cron, so that we won't have a 100ms delay (clusterCron).

Signed-off-by: Binbin <[email protected]>
Signed-off-by: Viktor Söderqvist <[email protected]>
sarthakaggarwal97 pushed a commit to sarthakaggarwal97/valkey that referenced this pull request Sep 16, 2025
…ailover (valkey-io#1067)

In CLUSTER FAILOVER FORCE case, we will set mf_can_start to
1 and wait for a cron to trigger the election. We can also set a
CLUSTER_TODO_HANDLE_MANUALFAILOVER flag so that we
can start the election as soon as possible instead of waiting for
the cron, so that we won't have a 100ms delay (clusterCron).

Signed-off-by: Binbin <[email protected]>
Signed-off-by: Viktor Söderqvist <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cluster release-notes This issue should get a line item in the release notes

Projects

Status: 8.0.5
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants