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

Ingress resources not working after update from v1.8.1 to v1.9.4 (chart version v4.7.1 to v4.8.3) #10973

Closed
marino-u opened this issue Feb 6, 2024 · 8 comments
Labels
kind/support Categorizes issue or PR as a support question. needs-priority needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. triage/needs-information Indicates an issue needs more information in order to work on it.

Comments

@marino-u
Copy link

marino-u commented Feb 6, 2024

Hi, I have an ingress resource with path "/" and several ingresses for apis with path (for example "/api1", "/api2") etc.
The cluster is on EKS and after an upgrade of EKS cluster and nginx-controller from v1.8.1 to v1.9.4 (chart version v4.7.1 to v4.8.3), all the requests to "/api1", "/api2", ... are handled by the ingress at "/".
I attach the ingress resources configuration as a screenshot (sorry but I can't copy/paste text):

apigateway-ingress
authz-private-ingress

Do you have any idea on what the problem could be? Maybe there were some changes between the versions that i didn't found.
Thanks in advance for your help.

@k8s-ci-robot k8s-ci-robot added the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Feb 6, 2024
@k8s-ci-robot
Copy link
Contributor

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. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@k8s-ci-robot k8s-ci-robot added needs-kind Indicates a PR lacks a `kind/foo` label and requires one. needs-priority labels Feb 6, 2024
@strongjz
Copy link
Member

strongjz commented Feb 6, 2024

Read some of the considerations for 1.9.0 https://github.com/kubernetes/ingress-nginx/releases/tag/controller-v1.9.0

There were some changes that might effect your configurations.

@longwuyuan
Copy link
Contributor

/kind support

@k8s-ci-robot k8s-ci-robot added kind/support Categorizes issue or PR as a support question. and removed needs-kind Indicates a PR lacks a `kind/foo` label and requires one. labels Feb 8, 2024
@strongjz
Copy link
Member

/triage more-information

There are only two ingresses shown, / and /wonder-src-auth , so /api would go to the /

Do you post all the ingresses?

Did you check the links https://github.com/kubernetes/ingress-nginx/releases/tag/controller-v1.9.0

@k8s-ci-robot
Copy link
Contributor

@strongjz: The label(s) triage/more-information cannot be applied, because the repository doesn't have them.

In response to this:

/triage more-information

There are only two ingresses shown, / and /wonder-src-auth , so /api would go to the /

Do you post all the ingresses?

Did you check the links https://github.com/kubernetes/ingress-nginx/releases/tag/controller-v1.9.0

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@longwuyuan
Copy link
Contributor

/triage needs-information

@k8s-ci-robot k8s-ci-robot added the triage/needs-information Indicates an issue needs more information in order to work on it. label Feb 15, 2024
@marino-u
Copy link
Author

@strongjz "/api" was just an example. I set up an environment to replicate the behaviour.
My idea is that ssl-redirect removes the trailing slash and the path is no more matched; in the following example default-app1-svc-80 is the service associated with /app1($|(/.*)) prefix, while default-app-svc-80 is the one associated with "/" prefix.
After 308 Permanent redirect, the request is sent to the wrong service:
immagine
Do you have any idea about that?

BRs,
Marino

@marino-u
Copy link
Author

It seems that now, with the annotations validation, rewrite-prefix should always start with "/" .
I got the following error and I solved adding the slash before $1:
validation error on ingress wonder/wonder-src-authz-private-ingress: annotation rewrite-target contains invalid value $1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/support Categorizes issue or PR as a support question. needs-priority needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. triage/needs-information Indicates an issue needs more information in order to work on it.
Projects
Development

No branches or pull requests

4 participants