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 #4176: Bugfix: Remove multicast group from cache when group is #4202

Conversation

wenyingd
Copy link
Contributor

@wenyingd wenyingd commented Sep 6, 2022

Cherry pick of #4176 on release-1.8.

#4176: Bugfix: Remove multicast group from cache when group is

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

An issue exists in the code when calling UninstallGroups in multicast
feature, that the group entry in the cache is not removed. This is
because the existing code only remove group in featureService cache.

To resolve the issue, rename the existing API UninstallGroup as
UninstallServiceGroup, and add a new API UninstallMulticastGroup to
delete multicast related groups. The new API also removes the group
entry in the cache.

Signed-off-by: wenyingd <[email protected]>
@wenyingd wenyingd 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 Sep 6, 2022
@codecov
Copy link

codecov bot commented Sep 6, 2022

Codecov Report

Merging #4202 (07b8728) into release-1.8 (33e1b53) will increase coverage by 0.35%.
The diff coverage is 69.23%.

Impacted file tree graph

@@               Coverage Diff               @@
##           release-1.8    #4202      +/-   ##
===============================================
+ Coverage        65.91%   66.27%   +0.35%     
===============================================
  Files              304      304              
  Lines            46593    46601       +8     
===============================================
+ Hits             30712    30884     +172     
+ Misses           13465    13325     -140     
+ Partials          2416     2392      -24     
Flag Coverage Δ
e2e-tests 39.34% <7.69%> (?)
integration-tests 35.00% <7.69%> (+0.04%) ⬆️
kind-e2e-tests 49.02% <7.69%> (+0.16%) ⬆️
unit-tests 44.99% <69.23%> (+0.28%) ⬆️
Impacted Files Coverage Δ
pkg/agent/multicast/mcast_controller.go 53.47% <0.00%> (ø)
pkg/agent/proxy/proxier.go 66.11% <0.00%> (+0.59%) ⬆️
pkg/agent/openflow/client.go 72.63% <100.00%> (+2.90%) ⬆️
pkg/agent/cniserver/ipam/antrea_ipam.go 52.81% <0.00%> (-21.22%) ⬇️
pkg/agent/cniserver/ipam/ipam_service.go 53.93% <0.00%> (-10.12%) ⬇️
pkg/controller/externalippool/controller.go 84.82% <0.00%> (-6.25%) ⬇️
pkg/ovs/openflow/ofctrl_packetin.go 63.33% <0.00%> (-3.34%) ⬇️
pkg/controller/traceflow/controller.go 74.72% <0.00%> (-2.53%) ⬇️
pkg/controller/ipam/antrea_ipam_controller.go 76.25% <0.00%> (-2.01%) ⬇️
.../flowexporter/connections/conntrack_connections.go 79.52% <0.00%> (-1.91%) ⬇️
... and 29 more

@wenyingd wenyingd requested a review from tnqn September 6, 2022 08:50
@wenyingd
Copy link
Contributor Author

wenyingd commented Sep 6, 2022

/test-all

@tnqn tnqn merged commit 4ce5bbd into antrea-io:release-1.8 Sep 6, 2022
@wenyingd wenyingd deleted the automated-cherry-pick-of-#4176-upstream-release-1.8 branch September 13, 2022 03: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