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

[release-1.12] Cherry-pick Set KEEPALIVE on istio-ingress-gateway listeners #3105

Conversation

mvladev
Copy link

@mvladev mvladev commented Nov 4, 2020

How to categorize this PR?

/area networking
/kind bug
/priority normal

What this PR does / why we need it:

Some LoadBalancers (AWS NLB) do not set KEEPALIVE when they open a TCP connection to the Istio Ingress Gateway. For long living connections it can cause silent timeouts when idle timeouts are hit.

Therefore envoy must be configured to send KEEPALIVE to downstream (LB).

See envoyproxy/envoy#3634

Which issue(s) this PR fixes:

n/a

Special notes for your reviewer:

Cherry-pick of #3104

Release note:

`istio-ingressgateway` now uses KEEPALIVE to downstream LoadBalancers to prevent idle timeout issues.

Some LoadBalancers do not set KEEPALIVE when they open a TCP connection
to the Istio Ingress Gateway. For long living connections it can cause
silent timeouts.
Therefore envoy must be configured to send KEEPALIVE to downstream (LB).

See envoyproxy/envoy#3634
@mvladev mvladev requested a review from a team as a code owner November 4, 2020 08:17
@gardener-robot gardener-robot added area/networking Networking related kind/bug Bug priority/normal size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Nov 4, 2020
Copy link
Member

@rfranzke rfranzke left a comment

Choose a reason for hiding this comment

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

/lgtm

@rfranzke rfranzke merged commit 77d1744 into gardener:release-v1.12 Nov 4, 2020
@mvladev mvladev deleted the istio-ingress-gateway-downstream-keep-alive-release-v1.12 branch November 6, 2020 11:47
@gardener-robot gardener-robot added priority/3 Priority (lower number equals higher priority) and removed priority/3 Priority (lower number equals higher priority) labels Mar 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/networking Networking related kind/bug Bug size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants