-
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
Use a less-common port for e2e test http pod. #101381
Conversation
8080 is a common port that is more likely to cause conflicts when used in hostNetwork mode for tests.
/sig network |
whoah, seems like nodelocalDNS should really not do that. we should file a seperate issue imho. |
/retest |
/triage accepted Thanks @aojea, the tests have passed now. |
I agree, I should have picked a different port for the NodeLocalDNS healthcheck. Since this config has been used for a while now, I am wary of changing it, in case it causes some new conflict on user clusters. |
/lgtm |
is it at least configurable? |
Yes, the port can be specified in the configmap -
|
thanks, sorry for distracting from merging the immediate e2e test |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aojea, BenTheElder, prameshj 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 |
True.. was hoping to merge early next week. Didn't add a hold label in time. What dashboards should we refer to? This one? |
the ci-signal group monitors that and open issues. |
|
#96453 (comment) Can you revert the last pr too? |
Thanks! I had forgotten we did this.. reverting the PR at kubernetes/test-infra#21936 |
What type of PR is this?
What this PR does / why we need it:
8080 is a common port that is more likely to cause conflicts when used
in hostNetwork mode for tests. For example - enabling these tests to run along with NodeLocalDNS will fail, since NodeLocalDNS uses 8080 port for healthcheck and runs in hostNetwork mode. The test pods trying to bind to 8080 will fail to schedule in this case.
The nodelocaldns testgrid does not currently run these tests though - https://testgrid.k8s.io/sig-network-gce#gci-gce-coredns-nodecache
Related to #100889 (does not fix it though)
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.: