-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Changing a Service from type: LoadBalancer to type: ClusterIP doesn't remove status.loadBalancer.ingress[] #9972
Comments
Hey @dsharp-pivotal thanks for opening this issue. Just to clarify, it seems like your expected outcome depends on changing the functionality of I'm not very familiar with |
I don't care terribly much about whether or not the tunnel still works (although indeed ideally the tunnel would stop working). I want to be able to observe, via Please take a look at LoadBalancerEmulator: https://github.com/kubernetes/minikube/blob/master/pkg/minikube/tunnel/loadbalancer_patcher.go Due to how it's implemented, it will add |
/assign |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
Steps to reproduce the issue:
type: LoadBalancer
kubectl get service my-svc -o yaml
showsstatus.loadBalancer.ingress[0].ip
with the service's clusterIPActual:
status.loadBalancer.ingress[0].ip
remains.Expected:
status.loadBalancer.ingress
should be removed.This would help automated tests that need to work with both minikube and cloud k8s clusters like GKE, that test adding and removing external access on Service.
We took a look at LoadBalancerEmulator, and saw that it has a method
cleanupService()
. However, it is only called on Services with type: LoadBalancer, and the cleanup methods are only called during shutdown.Ideally, it seems like PatchServices should reconcile all Services to add or remove status.loadBalancer.ingress as appropriate.
The text was updated successfully, but these errors were encountered: