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

Add dual-stack service support #2207

Merged
merged 1 commit into from
May 27, 2021
Merged

Conversation

xliuxu
Copy link
Contributor

@xliuxu xliuxu commented May 25, 2021

This is the following PR separated from #1973.

Kubernetes 1.21 add support for dual-stack services. To support this we need to update third_party/proxy pkg to reflect the recent changes in the upstream kubernetes repo. Unit tests are added accordingly.

@codecov-commenter
Copy link

codecov-commenter commented May 25, 2021

Codecov Report

Merging #2207 (3cbe27b) into main (29f4deb) will increase coverage by 3.51%.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2207      +/-   ##
==========================================
+ Coverage   61.63%   65.14%   +3.51%     
==========================================
  Files         273      274       +1     
  Lines       20673    20830     +157     
==========================================
+ Hits        12742    13570     +828     
+ Misses       6594     5880     -714     
- Partials     1337     1380      +43     
Flag Coverage Δ
e2e-tests 54.06% <66.66%> (?)
kind-e2e-tests 52.58% <66.66%> (-0.04%) ⬇️
unit-tests 41.23% <37.50%> (+0.02%) ⬆️

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

Impacted Files Coverage Δ
pkg/agent/proxy/types/types.go 46.66% <0.00%> (ø)
pkg/agent/proxy/proxier.go 63.89% <66.66%> (+0.29%) ⬆️
pkg/agent/proxy/service.go 100.00% <100.00%> (ø)
...g/controller/networkpolicy/clusternetworkpolicy.go 72.79% <0.00%> (-14.35%) ⬇️
pkg/agent/nodeportlocal/rules/iptable_rule.go 55.93% <0.00%> (-7.71%) ⬇️
pkg/apiserver/storage/ram/watch.go 90.38% <0.00%> (-3.85%) ⬇️
...gent/controller/networkpolicy/status_controller.go 72.60% <0.00%> (-2.74%) ⬇️
...kg/controller/networkpolicy/store/networkpolicy.go 83.58% <0.00%> (-1.42%) ⬇️
pkg/controller/types/networkpolicy.go 100.00% <0.00%> (ø)
pkg/k8s/client.go
... and 52 more

Copy link
Contributor

@antoninbas antoninbas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one small comment on testing, otherwise LGTM (I didn't really review the files under third_party in details since they are copied from upstream).

@jianjuns @tnqn do you think we should try to merge this before the Antrea v1.1 release since dual-stack services are Beta (enabled by default) in K8s v1.21?

pkg/agent/proxy/proxier_test.go Outdated Show resolved Hide resolved
third_party/proxy/service.go Show resolved Hide resolved
@jianjuns
Copy link
Contributor

Yes, agreed would be nice to include this one in 1.1. And the PR looks good to me.

@antoninbas antoninbas added this to the Antrea v1.1 release milestone May 27, 2021
Kubernetes 1.21 supports dual-stack services. This is a following
up PR of antrea-io#1973. This PR updates third_party/proxy package based on
the recent changes of kubernetes v1.21.0. Unit test for dual-stack
service is added.

Signed-off-by: Xu Liu <[email protected]>
@xliuxu
Copy link
Contributor Author

xliuxu commented May 27, 2021

/test-all

Copy link
Contributor

@antoninbas antoninbas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated unit test looks good to me

@xliuxu
Copy link
Contributor Author

xliuxu commented May 27, 2021

/test-conformance
/test-e2e

@xliuxu xliuxu merged commit 084a817 into antrea-io:main May 27, 2021
@xliuxu xliuxu deleted the dual_stack_svc branch May 27, 2021 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants