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 #4492

Conversation

tnqn
Copy link
Member

@tnqn tnqn commented Dec 18, 2022

Cherry pick of #4491 on release-1.7.

#4491: Make memberlist cluster rejoin dead nodes periodically

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

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>
@tnqn tnqn 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 Dec 18, 2022
@tnqn
Copy link
Member Author

tnqn commented Dec 18, 2022

/test-whole-conformance
/skip-conformance
/test-networkpolicy
/test-e2e
/test-flexible-ipam-e2e
/test-multicast-e2e
/test-windows-all
/test-ipv6-all
/test-ipv6-only-all

@codecov
Copy link

codecov bot commented Dec 18, 2022

Codecov Report

Merging #4492 (c4c5375) into release-1.7 (b4c36c5) will increase coverage by 0.44%.
The diff coverage is 84.28%.

Impacted file tree graph

@@               Coverage Diff               @@
##           release-1.7    #4492      +/-   ##
===============================================
+ Coverage        62.74%   63.18%   +0.44%     
===============================================
  Files              298      298              
  Lines            44408    44439      +31     
===============================================
+ Hits             27862    28078     +216     
+ Misses           14316    14134     -182     
+ Partials          2230     2227       -3     
Flag Coverage Δ
e2e-tests 44.77% <76.81%> (?)
kind-e2e-tests 50.41% <76.81%> (-0.04%) ⬇️
unit-tests 44.08% <50.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
cmd/antrea-agent/agent.go 0.00% <0.00%> (ø)
pkg/agent/memberlist/cluster.go 78.39% <85.50%> (+1.94%) ⬆️
pkg/controller/networkpolicy/tier.go 50.00% <0.00%> (-5.00%) ⬇️
pkg/controller/networkpolicy/status_controller.go 68.54% <0.00%> (-3.23%) ⬇️
...agent/flowexporter/connections/deny_connections.go 76.34% <0.00%> (-3.23%) ⬇️
.../certificatesigningrequest/approving_controller.go 73.40% <0.00%> (-3.20%) ⬇️
pkg/agent/cniserver/ipam/antrea_ipam.go 74.02% <0.00%> (-1.74%) ⬇️
pkg/controller/ipam/antrea_ipam_controller.go 75.10% <0.00%> (-1.32%) ⬇️
pkg/agent/openflow/pipeline.go 72.94% <0.00%> (-0.61%) ⬇️
pkg/agent/controller/trafficcontrol/controller.go 81.79% <0.00%> (-0.43%) ⬇️
... and 23 more

@xliuxu
Copy link
Contributor

xliuxu commented Dec 19, 2022

E2e failed with #4477, which should not be related with this PR.

@tnqn
Copy link
Member Author

tnqn commented Dec 19, 2022

/skip-e2e

@tnqn tnqn merged commit 0fda448 into antrea-io:release-1.7 Dec 19, 2022
@tnqn tnqn deleted the automated-cherry-pick-of-#4491-upstream-release-1.7 branch December 19, 2022 02:23
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