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

[v1.14] Backports 2023-11-28 #29442

Merged
merged 2 commits into from
Nov 29, 2023
Merged

[v1.14] Backports 2023-11-28 #29442

merged 2 commits into from
Nov 29, 2023

Conversation

sayboras
Copy link
Member

@sayboras sayboras commented Nov 28, 2023

Once this PR is merged, a GitHub action will update the labels of these PRs:

 29115

@sayboras sayboras requested a review from a team as a code owner November 28, 2023 12:05
@maintainer-s-little-helper maintainer-s-little-helper bot added backport/1.14 This PR represents a backport for Cilium 1.14.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. labels Nov 28, 2023
@sayboras
Copy link
Member Author

/test-backport-1.14

@sayboras sayboras marked this pull request as draft November 28, 2023 12:33
Signed-off-by: Tam Mach <tam.mach@cilium.io>
This is to fix the issue in which the wrong list of HTTP routes is used
to construct virtual host for insecure and secure routes. Ideally, only
related HTTP routes for the same listener ports should be considered.

Normally, this should not have any side effect, except using weightage
clusters for the multiple, but the same, backends. However, in case of
redirect filter, this will cause redirect loop as mentioned in the below
issue.

Fixes: 299648f
Fixes: #28186
Signed-off-by: Tam Mach <tam.mach@cilium.io>
@sayboras
Copy link
Member Author

/test-backport-1.14

1 similar comment
@sayboras
Copy link
Member Author

/test-backport-1.14

@sayboras sayboras marked this pull request as ready for review November 28, 2023 12:54
Copy link
Member

@meyskens meyskens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for me

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Nov 28, 2023
@sayboras sayboras merged commit 5eaa5ed into v1.14 Nov 29, 2023
236 of 291 checks passed
@sayboras sayboras deleted the tam/backport-29115 branch November 29, 2023 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.14 This PR represents a backport for Cilium 1.14.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. ready-to-merge This PR has passed all tests and received consensus from code owners to merge.
Projects
No open projects
Status: Released
Development

Successfully merging this pull request may close these issues.

None yet

3 participants