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

Conversation

wenyingd
Copy link
Contributor

@wenyingd wenyingd commented Sep 6, 2022

Cherry pick of #4176 on release-1.7.

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

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

@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 #4203 (52c2eff) into release-1.7 (3b03389) will decrease coverage by 5.64%.
The diff coverage is 7.69%.

❗ Current head 52c2eff differs from pull request most recent head 79a95c8. Consider uploading reports for the commit 79a95c8 to get more accurate results

Impacted file tree graph

@@               Coverage Diff               @@
##           release-1.7    #4203      +/-   ##
===============================================
- Coverage        50.28%   44.63%   -5.65%     
===============================================
  Files              261      258       -3     
  Lines            39347    39196     -151     
===============================================
- Hits             19785    17497    -2288     
- Misses           17609    19933    +2324     
+ Partials          1953     1766     -187     
Flag Coverage Δ
e2e-tests 44.63% <7.69%> (?)
kind-e2e-tests ?

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

Impacted Files Coverage Δ
pkg/agent/multicast/mcast_controller.go 0.00% <0.00%> (ø)
pkg/agent/proxy/proxier.go 46.66% <0.00%> (-8.14%) ⬇️
pkg/agent/openflow/client.go 61.35% <11.11%> (-0.27%) ⬇️
pkg/ipfix/ipfix_process.go 0.00% <0.00%> (-100.00%) ⬇️
pkg/ipfix/ipfix_registry.go 0.00% <0.00%> (-100.00%) ⬇️
pkg/ipfix/ipfix_intermediate.go 0.00% <0.00%> (-90.91%) ⬇️
.../agent/flowexporter/priorityqueue/priorityqueue.go 0.00% <0.00%> (-90.00%) ⬇️
...lowaggregator/clickhouseclient/clickhouseclient.go 0.00% <0.00%> (-83.70%) ⬇️
pkg/ipfix/ipfix_collector.go 0.00% <0.00%> (-83.34%) ⬇️
pkg/flowaggregator/certificate.go 0.00% <0.00%> (-75.53%) ⬇️
... and 62 more

@wenyingd wenyingd force-pushed the automated-cherry-pick-of-#4176-upstream-release-1.7 branch from 97b8af2 to b54d532 Compare September 6, 2022 06:52
@wenyingd wenyingd requested a review from tnqn September 6, 2022 08:50
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 <wenyingd@vmware.com>
@wenyingd wenyingd force-pushed the automated-cherry-pick-of-#4176-upstream-release-1.7 branch from b54d532 to 79a95c8 Compare September 6, 2022 09:23
@tnqn
Copy link
Member

tnqn commented Sep 6, 2022

/test-all

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