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

Readiness Probe does not get reflected for NEG enabled ingress #541

Closed
freehan opened this issue Nov 8, 2018 · 5 comments · Fixed by #582
Closed

Readiness Probe does not get reflected for NEG enabled ingress #541

freehan opened this issue Nov 8, 2018 · 5 comments · Fixed by #582
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@freehan
Copy link
Contributor

freehan commented Nov 8, 2018

When the following conditions apply:

  • NEG is enabled on the ingress referenced service
  • Service is ClusterIP type

Then LB health check is created with default setting.

@freehan freehan added the kind/bug Categorizes issue or PR as related to a bug. label Nov 8, 2018
@manigandham
Copy link

manigandham commented Nov 13, 2018

We see this happen to on the latest version 1.11.2-gke.18 GKE clusters. NEG health checks are not reflecting any of the settings in the deployment YAML and remain unhealthy.

@hrdymchl
Copy link

hrdymchl commented Nov 25, 2018

+1

Posted about my issue on SO here, and this sounds like the same problem. Am also on version 1.11.2-gke.18.

@uschen
Copy link

uschen commented Dec 11, 2018

same issue here 1.11.3 and 1.11.4

the NEG will retain the old pod IPs for at least 5min, in the meantime, requests will be responded with 502.

@freehan
Copy link
Contributor Author

freehan commented Dec 14, 2018

same issue here 1.11.3 and 1.11.4

the NEG will retain the old pod IPs for at least 5min, in the meantime, requests will be responded with 502.

@uschen Based on you description, it might be another issue. I created an umbrella issue for container native load balancing #583. Can you take a look and see if it matches any known issues?

If not, would you mind creating an issue with more detail? It would be great if it could include repro steps.

@freehan
Copy link
Contributor Author

freehan commented Dec 14, 2018

PR to fix the health check bug is in flight: #582

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
4 participants