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 #3295: Fix NodePort/LoadBalancer issue when proxyAll is enabled #3329

Conversation

hongliangl
Copy link
Contributor

Cherry pick of #3295 on release-1.4.

#3295: Fix NodePort/LoadBalancer issue when proxyAll is enabled

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

When proxyAll is enabled, create a NodePort/LoadBalancer Service whose
externalTrafficPolicy is Cluster, then only an OVS group with all
Endpoints will be installed. If change externalTrafficPolicy of the
Service from Cluster to Local, an OVS group with only local Endpoints
should be also installed since externalTrafficPolicy is Local, but it
is not. This patch fixes the issue that OVS group with only local
Endpoints is not installed when externalTrafficPolicy of Service is
changed from Cluster to Local.

Signed-off-by: Hongliang Liu <[email protected]>
@hongliangl hongliangl 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 Feb 17, 2022
@hongliangl
Copy link
Contributor Author

/test-all-features-conformance
/test-conformance
/test-e2e
/test-flexible-ipam-e2e
/test-ipv6-conformance
/test-ipv6-e2e
/test-ipv6-networkpolicy
/test-ipv6-only-conformance
/test-ipv6-only-e2e
/test-ipv6-only-networkpolicy
/test-multicluster-e2e
/test-networkpolicy
/test-windows-conformance
/test-windows-e2e
/test-windows-networkpolicy
/test-windows-proxyall-e2e
/test-integration

@codecov-commenter
Copy link

Codecov Report

Merging #3329 (8b99d80) into release-1.4 (73639d1) will increase coverage by 1.11%.
The diff coverage is 61.11%.

Impacted file tree graph

@@               Coverage Diff               @@
##           release-1.4    #3329      +/-   ##
===============================================
+ Coverage        59.80%   60.91%   +1.11%     
===============================================
  Files              290      290              
  Lines            24543    24550       +7     
===============================================
+ Hits             14677    14955     +278     
+ Misses            8260     7971     -289     
- Partials          1606     1624      +18     
Flag Coverage Δ
kind-e2e-tests 47.99% <61.11%> (+1.24%) ⬆️
unit-tests 40.18% <55.55%> (-0.02%) ⬇️

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

Impacted Files Coverage Δ
pkg/agent/proxy/proxier.go 60.89% <61.11%> (+3.46%) ⬆️
pkg/apiserver/handlers/endpoint/handler.go 58.82% <0.00%> (-11.77%) ⬇️
pkg/agent/cniserver/ipam/antrea_ipam_controller.go 61.53% <0.00%> (-3.85%) ⬇️
pkg/controller/networkpolicy/store/addressgroup.go 83.01% <0.00%> (-3.78%) ⬇️
pkg/agent/controller/networkpolicy/reject.go 68.59% <0.00%> (-1.66%) ⬇️
pkg/agent/agent.go 51.66% <0.00%> (-0.20%) ⬇️
pkg/agent/openflow/client.go 57.11% <0.00%> (+0.21%) ⬆️
pkg/ovs/openflow/ofctrl_bridge.go 54.98% <0.00%> (+0.60%) ⬆️
...kg/agent/flowexporter/connections/conntrack_ovs.go 77.57% <0.00%> (+1.21%) ⬆️
pkg/ovs/openflow/ofctrl_action.go 69.58% <0.00%> (+1.66%) ⬆️
... and 9 more

@tnqn
Copy link
Member

tnqn commented Feb 18, 2022

/test-all

2 similar comments
@hongliangl
Copy link
Contributor Author

/test-all

@hongliangl
Copy link
Contributor Author

/test-all

@tnqn
Copy link
Member

tnqn commented Feb 23, 2022

/test-e2e
/test-integration

@tnqn
Copy link
Member

tnqn commented Feb 24, 2022

/skip-e2e which failed on unrelated test: TestFlowAggregator

@tnqn tnqn merged commit 29e019d into antrea-io:release-1.4 Feb 24, 2022
@hongliangl hongliangl deleted the automated-cherry-pick-of-#3295-upstream-release-1.4 branch April 21, 2022 01:00
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.

3 participants