-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
promote ProxyTerminatingEndpoints to GA #117718
promote ProxyTerminatingEndpoints to GA #117718
Conversation
This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The 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. |
/assign @thockin @andrewsykim The next step is to promote some test to conformance, but it has to demonstrate they are stable for more than 1 week |
/sig network |
Change-Id: Ife524c831d905acbc606aa7631e1194f91199938
f332c7e
to
791573d
Compare
The existing termination period of 600 seconds for pods on the e2e test causes that those pods are kept running after the test has finished. 100 seconds is a good compromise to avoid leaving pods lingering and more than enought for the test to finish. Change-Id: I993162a77125345df1829044dc2514e03b13a407
unrelated
/test pull-kubernetes-unit run it again /test pull-kubernetes-e2e-gce |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
/lgtm
/approve
LGTM label has been added. Git tree hash: 33a65a4313bf5eca361229ee19a434227a8a8197
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aojea, thockin 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 |
unrelated, but why is this test running here? |
Is this related to #117824 ? |
it just removed the feature tag from the test name so the test started to be exercised on all jobs |
/kind feature
There are also e2e test with loadbalancers and deployments performing rolling updates
https://testgrid.k8s.io/google-gce#gci-gce-ingress&include-filter-by-regex=should%20not%20have%20connectivity%20disruption%20during%20rolling%20update%20with%20externalTrafficPolicy