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

Unneeded health check created for kube-system:default-http-backend service #385

Closed
tomoe opened this issue Jul 3, 2018 · 6 comments
Closed
Assignees

Comments

@tomoe
Copy link

tomoe commented Jul 3, 2018

When I create an Ingress that has default backends, the controller still creates a GCE healthcheck pointing to kube-system:default-http-backend service (default 404 server) even thought there's no chance of hitting the pod.

Usually, that doesn't do any harm, but possibly duplicated healthcheck probes may overload the l7-default-backend pod and causes the pod to restart with error messages because the resource setting for the container is very small (cpu: 10m, memory: 20M).

Now I think:

  • it would be nice if HC gets created if it really is needed
  • (not an issue of the controller, but) l7-default-backend resource setting may need to be increased, autoscalable, and/or configurable.
@rramkumar1
Copy link
Contributor

rramkumar1 commented Jul 6, 2018

@tomoe This issue should be fixed in GLBC v1.2.0. We will only create a backend service + health check for the system default backend if the user does not specify a default backend in their Ingress.

@rramkumar1
Copy link
Contributor

/close

@ghost
Copy link

ghost commented Jun 17, 2019

I still get that default BE. I even remove it manually and it forces the creation / link again.

@rramkumar1
Copy link
Contributor

rramkumar1 commented Jun 18, 2019

@AlbertoDominguezS What GKE cluster version are you using?

I would suggest filing a separate issue. It's much easier to track that way.

@ghost
Copy link

ghost commented Jun 18, 2019

@rramkumar1 Cluster Version: 1.13.6-gke.5

@rramkumar1
Copy link
Contributor

@AlbertoDominguezS Ok, would be useful to see your Ingress YAML so please file a separate issue. Thanks!

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

No branches or pull requests

3 participants