Commit dab16cd1 authored by Kubernetes Submit Queue's avatar Kubernetes Submit Queue Committed by GitHub

Merge pull request #49266 from MrHohn/e2e-proxy-healthz

Automatic merge from submit-queue (batch tested with PRs 49409, 49352, 49266, 48418) [e2e] Also verify content returned by kube-proxy healthz url **What this PR does / why we need it**: Enhance kube-proxy url test. This helps to detect the port collision case --- node-problem-detector also serves /healthz to return 200 ok. Verify the content to confirm /healthz is served by kube-proxy. **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: From #49263 **Special notes for your reviewer**: /assign @freehan @nicksardo **Release note**: ```release-note NONE ```
parents f5709147 5a2f2261
...@@ -85,6 +85,8 @@ var _ = framework.KubeDescribe("Networking", func() { ...@@ -85,6 +85,8 @@ var _ = framework.KubeDescribe("Networking", func() {
By("checking kube-proxy URLs") By("checking kube-proxy URLs")
config.GetSelfURL(ports.ProxyHealthzPort, "/healthz", "200 OK") config.GetSelfURL(ports.ProxyHealthzPort, "/healthz", "200 OK")
// Verify /healthz returns the proper content.
config.GetSelfURL(ports.ProxyHealthzPort, "/healthz", "lastUpdated")
config.GetSelfURL(ports.ProxyStatusPort, "/proxyMode", "iptables") // the default config.GetSelfURL(ports.ProxyStatusPort, "/proxyMode", "iptables") // the default
}) })
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment