-
Notifications
You must be signed in to change notification settings - Fork 39k
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
test/e2e/network: add test for Service internalTrafficPolicy #103409
test/e2e/network: add test for Service internalTrafficPolicy #103409
Conversation
62f00df
to
03db130
Compare
03db130
to
8a1cd1e
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andrewsykim The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
8a1cd1e
to
98c069d
Compare
/priority important-soon |
Host network better separate , per example, openshift imposes limitations on host network pods for security reasons, this way they people can filter then |
d81a394
to
9fd7ef2
Compare
/retest |
@aojea updated with new test cases using host network pods, PTAL |
/retest |
Oddly the network policy tests are running the new e2e tests in this PR despite the Feature tag being used. Their failing of course because the feature is not enabled yet. |
yep, the policy job run all sig-network tests, I asked to do this because twe can detect if the network policy tests "behave" and play well running in parallel with other tests, and doesn't block other traffic ... I need to find out if ginkgo can suppress all new features except the NetworkPlicy ones |
Sounds good, in the meantime I think we can just merge this after #103462 |
Signed-off-by: Andrew Sy Kim <kim.andrewsy@gmail.com>
… network) -> pod Signed-off-by: Andrew Sy Kim <kim.andrewsy@gmail.com>
…od (w/ host network) Signed-off-by: Andrew Sy Kim <kim.andrewsy@gmail.com>
…host network) -> pod (w/ host network) Signed-off-by: Andrew Sy Kim <kim.andrewsy@gmail.com>
9fd7ef2
to
04d59ff
Compare
NetworkPolicy failures might be legit, but will re-run one more time to be sure https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/103409/pull-kubernetes-e2e-ubuntu-gce-network-policies/1412401677541052416 /retest |
/lgtm unhold when you are happy Andrew, the test added here are passing 👍 |
/retest |
/hold cancel |
/test pull-kubernetes-integration
|
@andrewsykim: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Signed-off-by: Andrew Sy Kim kim.andrewsy@gmail.com
What type of PR is this?
/kind feature
What this PR does / why we need it:
Adds an e2e test for internalTrafficPolicy field.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: