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

External IP set on NodePort ingress service is ignored, #4861

Closed
underyx opened this issue Dec 25, 2019 · 3 comments
Closed

External IP set on NodePort ingress service is ignored, #4861

underyx opened this issue Dec 25, 2019 · 3 comments

Comments

@underyx
Copy link

underyx commented Dec 25, 2019

NGINX Ingress controller version: 0.26.0 - 0.26.2 (working correctly on 0.25.1)

What happened: My ingress-nginx service is type: NodePort and lists externalIPs. After upgrading to 0.26, the ingress resources started being given the cluster-internal IP instead of the external IP of the service:

NAMESPACE   NAME                       HOSTS                       ADDRESS       PORTS     AGE
public      underyx-me-ingress         underyx.me,www.underyx.me   10.96.68.61   80, 443   7d18h

What you expected to happen: The ingress resources should report their external IP address.

How to reproduce it (as minimally and precisely as possible): Configure your service like this

apiVersion: v1
kind: Service
metadata:
  name: ingress-nginx
  namespace: ingress-nginx
  labels:
    app.kubernetes.io/name: ingress-nginx
    app.kubernetes.io/part-of: ingress-nginx
spec:
  type: NodePort
  ports:
    - name: http
      port: 80
      targetPort: 80
    - name: https
      port: 443
      targetPort: 443
  selector:
    app.kubernetes.io/name: ingress-nginx
    app.kubernetes.io/part-of: ingress-nginx
  externalIPs:
    - <add your external IP here>

Anything else we need to know: This was probably broken with #4490. It seems like #4700 attempted to fix it but it still doesn't work for me in 0.26.2.

@aledbf
Copy link
Member

aledbf commented Dec 25, 2019

@underyx there is no release with #4700 (is planned for 0.27.0)

You can use quay.io/kubernetes-ingress-controller/nginx-ingress-controller:dev in the meantime.

@aledbf aledbf closed this as completed Dec 25, 2019
@underyx
Copy link
Author

underyx commented Dec 25, 2019

Oh, I see, sorry then. I got confused by the tag marked on GitHub's UI on the commit page:

Screenshot 2019-12-25 at 14 09 52

Does this not mean that the commit is part of the 0.26.2 release?

@aledbf
Copy link
Member

aledbf commented Dec 25, 2019

Does this not mean that the commit is part of the 0.26.2 release?

Not sure why that got tagged in that release.

The release only contains one commit since 0.26.1 https://github.com/kubernetes/ingress-nginx/releases/tag/nginx-0.26.2

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

2 participants