giantswarm / ingress-operator

The ingress-operator connects control plane ingress controllers with tenant cluster ingress controllers on a Giant Swarm Kubernetes installation.
2 stars 1 forks source link

ingress operator fails to update service #28

Closed r7vme closed 6 years ago

r7vme commented 7 years ago

After chaging TPO lbPort values, ingress operator can not update service.

 24   protocolPorts:
 25   - ingressPort: 30010
 26     lbPort: 30035
 27     protocol: http
 28   - ingressPort: 30011
 29     lbPort: 30036
 30     protocol: https
{"caller":"github.com/giantswarm/ingress-operator/vendor/github.com/giantswarm/operatorkit/framework/retryresource/resource.go:225","time":"17-09-29 17:27:46.051","underlyingResource":"service","warning":"retrying 'ProcessCreateState' due to error (Service \"nginx-ingress-controller\" is invalid: [spec.ports[12].name: Duplicate value: \"http-30010-lrzi1\", spec.ports[13].name: Duplicate value: \"https-30011-lrzi1\"])"}
{"caller":"github.com/giantswarm/ingress-operator/service/resource/service/resource.go:235","cluster":"lrzi1","debug":"creating the service data in the Kubernetes API","time":"17-09-29 17:27:56.247"}
{"caller":"github.com/giantswarm/ingress-operator/vendor/github.com/giantswarm/operatorkit/framework/retryresource/resource.go:225","time":"17-09-29 17:27:56.251","underlyingResource":"service","warning":"retrying 'ProcessCreateState' due to error (Service \"nginx-ingress-controller\" is invalid: [spec.ports[12].name: Duplicate value: \"http-30010-lrzi1\", spec.ports[13].name: Duplicate value: \"https-30011-lrzi1\"])"}

svc/nginx-ingress-controller has

 53   - name: http-30010-lrzi1
 54     nodePort: 30043
 55     port: 30043
 56     protocol: TCP
 57     targetPort: 30043
 58   - name: https-30011-lrzi1
 59     nodePort: 30044
 60     port: 30044
 61     protocol: TCP
 62     targetPort: 30044

Also it added new entry to configmap, but i expect it to update existing entry.

installations|master⚡ ⇒ kubectl get cm/ingress-nginx-tcp-services -n kube-system -o yaml. Duplicates for `lrzi1`
apiVersion: v1
data:
  "30023": pv1ig/worker:30010
  "30024": pv1ig/worker:30011
  "30031": t4rap/worker:30010
  "30032": t4rap/worker:30011
  "30035": lrzi1/worker:30010
  "30036": lrzi1/worker:30011
  "30041": 9hr00/worker:30010
  "30042": 9hr00/worker:30011
  "30043": lrzi1/worker:30010
  "30044": lrzi1/worker:30011
  "30045": bb6e0/worker:30010
  "30046": bb6e0/worker:30011
r7vme commented 6 years ago

I think this is outdated. Closing.