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

Automated cherry pick of #4491: Make memberlist cluster rejoin dead nodes periodically #4528

Conversation

xliuxu
Copy link
Contributor

@xliuxu xliuxu commented Jan 3, 2023

Cherry pick of #4491 on release-1.8.

#4491: Make memberlist cluster rejoin dead nodes periodically

For details on the cherry pick process, see the cherry pick requests page.

@xliuxu xliuxu added the kind/cherry-pick Categorizes issue or PR as related to the cherry-pick of a bug fix from the main branch to a release label Jan 3, 2023
@xliuxu xliuxu force-pushed the automated-cherry-pick-of-#4491-upstream-release-1.8 branch from cb18b59 to d4a6ec8 Compare January 3, 2023 03:27
The patch periodically rejoins Nodes that were removed from the member
list by memberlist because they were unreachable for more than 15
seconds (the GossipToTheDeadTime we are using). Without it, once there
is a network downtime lasting more than 15 seconds, the agent wouldn't
try to reach any other Node and would think it's the only alive Node
until it's restarted.

Signed-off-by: Quan Tian <qtian@vmware.com>
@xliuxu xliuxu force-pushed the automated-cherry-pick-of-#4491-upstream-release-1.8 branch from d4a6ec8 to 39f5c97 Compare January 3, 2023 03:28
@codecov
Copy link

codecov bot commented Jan 3, 2023

Codecov Report

Merging #4528 (39f5c97) into release-1.8 (45b2090) will increase coverage by 2.92%.
The diff coverage is 85.50%.

Impacted file tree graph

@@               Coverage Diff               @@
##           release-1.8    #4528      +/-   ##
===============================================
+ Coverage        63.16%   66.08%   +2.92%     
===============================================
  Files              304      304              
  Lines            46650    46729      +79     
===============================================
+ Hits             29466    30881    +1415     
+ Misses           14829    13469    -1360     
- Partials          2355     2379      +24     
Flag Coverage Δ
integration-tests 35.15% <ø> (+0.05%) ⬆️
kind-e2e-tests 48.83% <76.81%> (+6.54%) ⬆️
unit-tests 45.20% <50.72%> (+0.01%) ⬆️
Impacted Files Coverage Δ
pkg/agent/memberlist/cluster.go 75.61% <85.50%> (+2.23%) ⬆️

... and 60 files with indirect coverage changes

@tnqn
Copy link
Member

tnqn commented Mar 27, 2023

/skip-all

@tnqn tnqn closed this Mar 27, 2023
@tnqn tnqn reopened this Mar 27, 2023
@tnqn tnqn merged commit aa1d356 into antrea-io:release-1.8 Mar 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cherry-pick Categorizes issue or PR as related to the cherry-pick of a bug fix from the main branch to a release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants