kubernetes / ingress-nginx

Ingress NGINX Controller for Kubernetes
https://kubernetes.github.io/ingress-nginx/
Apache License 2.0
17.33k stars 8.22k forks source link

Problem ingress deployment creation, on AKS, has been deprecated controller.podLabels.aadpodidbinding=mi-devxxx and this will be replaced with Azure AD Workload Identity #9256

Closed silandrew closed 3 weeks ago

silandrew commented 1 year ago

Problem ingress deployment creation, on AKS, has been deprecated controller.podLabels.aadpodidbinding=mi-devxxx and replaced with Azure AD Workload Identity :

depicted pod identity on AKS this is replaced by Azure AD Workload Identity:

NGINX Ingress controller version ingress-nginx- chart ver 4.2.5 app ver 1.3.1 (exec into the pod and run nginx-ingress-controller --version.):

Kubernetes version 124.3 (use kubectl version):

Environment Azurte AKS:

As minimally and precisely as possible. Keep in mind we do not have access to your cluster or application. Help up us (if possible) reproducing the issue using minikube or kind.

Install minikube/kind

Install the ingress controller

kubectl apply -f https://raw.githubusercontent.com/kubernetes/ingress-nginx/main/deploy/static/provider/baremetal/deploy.yaml

Install an application that will act as default backend (is just an echo app)

kubectl apply -f https://raw.githubusercontent.com/kubernetes/ingress-nginx/main/docs/examples/http-svc.yaml

Create an ingress (please add any additional annotation required)

echo " apiVersion: networking.k8s.io/v1 kind: Ingress metadata: name: xxxxxx namespace: xxxx annotations:

kubernetes.io/ingress.allow-http: "false"

# defines controller implementing this ingress resource: https://docs.microsoft.com/en-us/azure/dev-spaces/how-to/ingress-https-traefik
# ingress.class annotation is being deprecated in Kubernetes 1.18: https://kubernetes.io/docs/concepts/services-networking/ingress/#deprecated-annotation
# For backwards compatibility, when this annotation is set, precedence is given over the new field ingressClassName under spec.
kubernetes.io/ingress.class: nginx
#kubernetes.io/ingress.allow-http: "false"
#nginx.ingress.kubernetes.io/force-ssl-redirect: "true"
nginx.ingress.kubernetes.io/use-forwarded-headers: "true"
nginx.ingress.kubernetes.io/enable-real-ip: "true"
nginx.ingress.kubernetes.io/compute-full-forwarded-for: "true"
nginx.ingress.kubernetes.io/proxy-add-original-uri-header: "true"    
nginx.ingress.kubernetes.io/affinity: cookie
nginx.ingress.kubernetes.io/affinity-mode: persistent
nginx.ingress.kubernetes.io/proxy-body-size: 1024m
#traefik.ingress.kubernetes.io/router.entrypoints: websecure
# traefik.ingress.kubernetes.io/router.tls: "true"
# traefik.ingress.kubernetes.io/router.tls.options: default
# traefik.ingress.kubernetes.io/router.middlewares: gzip-compress@file

spec:

ingressClassName: "traefik-internal"

tls:

make a request

--->

Anything else we need to know:

this replace pod identity az aks update -g rg-aks-xxx -n aks-xxx2 --enable-managed-identity --assign-identity xxxxxxxxxx

k8s-ci-robot commented 1 year ago

@silandrew: This issue is currently awaiting triage.

If Ingress contributors determines this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
longwuyuan commented 1 year ago

/remove-kind bug /help /kind support

k8s-ci-robot commented 1 year ago

@longwuyuan: This request has been marked as needing help from a contributor.

Guidelines

Please ensure that the issue body includes answers to the following questions:

For more details on the requirements of such an issue, please see here and ensure that they are met.

If this request no longer meets these requirements, the label can be removed by commenting with the /remove-help command.

In response to [this](https://github.com/kubernetes/ingress-nginx/issues/9256): >/remove-kind bug >/help >/kind support Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
longwuyuan commented 3 weeks ago

This is 2 years old now and there have been changes to AKS install as well as K8S specs since then.

I don't know what this "ontroller.podLabels.aadpodidbinding=mi-aks-dev-01" but whatever it is is heavily relying on Azure and AKS. So its less likely to be any code in the controller itself.

In any case to take any action now, the install has to be attempted again on AKS and the precise detailed elaborate information on the problem to install needs to be posted here.

There are several Azure users currently as per issues posted in the last 2 years and so its not likely that there is a problem in the controller per se for installing on AKS.

If there is a problem, please post the data as requested so that someone can analyze and comment. Hence closing this issue now as there is no current action item being tracked here and this is just adding to the tally of open issues, without tracking any action item.

/close

k8s-ci-robot commented 3 weeks ago

@longwuyuan: Closing this issue.

In response to [this](https://github.com/kubernetes/ingress-nginx/issues/9256#issuecomment-2336624290): >This is 2 years old now and there have been changes to AKS install as well as K8S specs since then. > >I don't know what this `"ontroller.podLabels.aadpodidbinding=mi-aks-dev-01"` but whatever it is is heavily relying on Azure and AKS. So its less likely to be any code in the controller itself. > >In any case to take any action now, the install has to be attempted again on AKS and the precise detailed elaborate information on the problem to install needs to be posted here. > >There are several Azure users currently as per issues posted in the last 2 years and so its not likely that there is a problem in the controller per se for installing on AKS. > >If there is a problem, please post the data as requested so that someone can analyze and comment. Hence closing this issue now as there is no current action item being tracked here and this is just adding to the tally of open issues, without tracking any action item. > >/close Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository.