cilium / cilium / 40117

39%

push

travis-ci-com

<a href="https://github.com/cilium/cilium/commit/<a class=hub.com/cilium/cilium/commit/<a class="double-link" href="https://git"><a class=hub.com/cilium/cilium/commit/<a class="double-link" href="https://git"><a class=hub.com/cilium/cilium/commit/365c788bb69eae6523ca22a3fa2b595069529c0a">365c788bb">&lt;a href=&quot;https://github.com/cilium/cilium/commit/</a><a class="double-link" href="https://github.com/cilium/cilium/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/cilium/cilium/commit/365c788bb69eae6523ca22a3fa2b595069529c0a">365c788bb</a><a href="https://github.com/cilium/cilium/commit/365c788bb69eae6523ca22a3fa2b595069529c0a">&lt;a href=&quot;https://github.com/cilium/cilium/commit/365c788bb69eae6523ca22a3fa2b595069529c0a&quot;&gt;&amp;quot;&amp;gt;k8s: Use kubelet&amp;amp;#39;s logic to close all idle connections

Following up from https://github.com/cilium/cilium/pull/19259, this
commit implements the same logic as kubelet [1] to close all idle
connections. We can rely on HTTP/2 to determine which connections are
idle (HTTP/2 pings), instead of closing all connections indiscriminately
(when the heartbeat fails) via the custom dialer in setDialer().

We can assume HTTP/2 support as it has been supported since K8s 1.20
[2]. Users can disable HTTP/2 by setting the DISABLE_HTTP2 environment
variable, however this only really meant from very advanced users, hence
no documentation. Kubernetes has been running with HTTP/2 as the default
for a long time (~since Dec 2020) without major issues.

CloseIdleConnectionsFor() was introduced in K8s 1.23 [3], hence this commit
cannot be backported to any earlier version of Cilium which doesn&amp;amp;#39;t
support K8s 1.23.

[1]: https://github.com/kubernetes/kubernetes/pull/104844
[2]: https://github.com/kubernetes/kubernetes/pull/95981/
[3]: https://github</a>.com/kubernetes/kubernetes/commit/&lt;/a&gt;&lt;a class=&quot;double-link&quot; href=&quot;https://github.com/cilium/cilium/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://github.com/cilium/cilium/commit/b9d865a81&quot;&gt;b9d865a81&lt;/a&gt;&quot;&gt;b9d865a81&lt;/a&gt;&lt;... (continued)

10 of 10 new or added lines in 1 file covered. (100.0%)

45567 of 129982 relevant lines covered (35.06%)

6808.69 hits per line