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 forward both 80 and 443 ports to tenant clusters #235

Merged
merged 1 commit into from
Jul 26, 2024
Merged

Conversation

kvaps
Copy link
Member

@kvaps kvaps commented Jul 25, 2024

We need to separate HTTP and HTTPS traffic and send them into tenant clusters.
Currently traffic was sending only on HTTPS port, this PR enables HTTP traffic forwarding.

Nginx ingress does not support setting correct upstream according to type of traffic (http or https)
There are set of issues in upstream.

Good to know that we found reliable workaround

fixes: #209 (comment)

@kvaps kvaps requested a review from gecube July 25, 2024 19:08
@kvaps kvaps merged commit 32f22ad into main Jul 26, 2024
@kvaps kvaps deleted the ingress-80-443 branch July 26, 2024 10:01
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

Successfully merging this pull request may close these issues.

Ingress addon node pods are not labeled with their role
1 participant