Commit 6c2e80d
[release-1.8] cherry-pick for v1.8.2 (#9376)
* fix rate limit validation on K8s 1.36 (#9166)
fixi: rate limit validation on K8s 1.36
Starting with K8s 1.36 the int range and maximum are validated
against each other, which breaks the BTP tests.
This can be fixed validating the field as uint32.
Signed-off-by: Clemens Beck <mail@beckcl.dev>
Co-authored-by: Clemens Beck <mail@beckcl.dev>
(cherry picked from commit a21fd29)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* fix(validation): allow IPv6 to be specific in loadBalancerSourceRanges (#9050)
* fix(validation): allow IPv6 to be specific in loadBalancerSourceRanges and loadBalancerIP
Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
(cherry picked from commit 3c1d7b9)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* fix: allow Backend TLS to disable ALPN with an empty list (#9162)
* Fixed Backend TLS to disable upstream ALPN instead of inheriting EnvoyProxy BackendTLS defaults.
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
* add test case for envoyproxy without tls
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
* Potential fix for pull request finding
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
* address comments
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
---------
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
(cherry picked from commit 21eb101)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* fix(gatewayapi): sort api key auth credentials (#9042)
* fix(gatewayapi): sort api key auth credentials
Signed-off-by: Alexej Disterhoft <alexej.disterhoft@redcare-pharmacy.com>
* docs(release-notes): add api key auth fix note
Signed-off-by: Alexej Disterhoft <alexej.disterhoft@redcare-pharmacy.com>
---------
Signed-off-by: Alexej Disterhoft <alexej.disterhoft@redcare-pharmacy.com>
Signed-off-by: Alexej Disterhoft <alexej@disterhoft.de>
Signed-off-by: zirain <zirain2009@gmail.com>
Co-authored-by: Arko Dasgupta <arkodg@users.noreply.github.com>
Co-authored-by: zirain <zirain2009@gmail.com>
(cherry picked from commit b96a2d3)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* fix(status): separate listener programmed state from HTTPRoute Accepted condition (#9129)
* fix(status): separate listener programmed state from HTTPRoute Accepted condition
Remove the NoReadyListeners block that set Accepted: False on routes when
no listeners were programmed. Route acceptance reflects binding validity;
listener programmed state is already tracked by the listener's own conditions.
Signed-off-by: apkatsikas <apkatsikas@gmail.com>
* fix(status): separate listener programmed state from TCPRoute/UDPRoute Accepted condition
Routes with a valid binding to an unprogrammed listener now correctly reach
Accepted=True regardless of listener readiness. Moves accepted=true and
IncrementAttachedRoutes before the IsReady check so the Accepted condition
reflects binding validity only, consistent with the HTTPRoute fix and the
Gateway API spec.
Signed-off-by: apkatsikas <apkatsikas@gmail.com>
(cherry picked from commit e9c3ac5)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* fix: reject ExternalName Service as a route backend (#9190)
fix: reject Service as a route backend
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
(cherry picked from commit ee16b3f)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* fix(status): fall back to service externalIPs for Gateway addresses (#9209)
When the Envoy service is of type LoadBalancer but no load balancer
controller assigns an ingress address (e.g. bare-metal clusters), the
Gateway status had no addresses and stayed Programmed=False with reason
AddressNotAssigned, even when reachable addresses were configured via
spec.externalIPs (e.g. through an EnvoyProxy service patch).
Use spec.externalIPs as a fallback when the load balancer ingress list
is empty. Ingress addresses keep priority when present, so behavior is
unchanged for clusters with a working load balancer controller.
Fixes #8987
Signed-off-by: Jules Dutel <jules.dutel@numberly.com>
(cherry picked from commit f2c0921)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* fix: don't duplicate ValidatingAdmissionPolicy in install.yaml (#9182)
* Fix duplicate VAP in install.yaml
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* Add regression check and release notes
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
---------
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
(cherry picked from commit 332081c)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* fix(listenerset): implement hostname conflict listener precedence (#9192)
* fix(listenerset): implement hostname conflict listener precedence
- In Gateway, multiple listeners with the same hostname all get marked
Conflicted (no winner). In ListenerSet, a Gateway-owned listener
wins over ListenerSet listeners; among ListenerSet listeners the first
in processing order wins.
- Conflicted ListenerSet listeners now get Accepted=False and
Programmed=False with the conflict reason (HostnameConflict or
ProtocolConflict), matching the Gateway API conformance expectation.
- AttachedListenerSets on the Gateway status is now incremented only for
ListenerSets that have at least one accepted listener, aligning with
the spec definition of "successfully attached".
Signed-off-by: zirain <zirain2009@gmail.com>
* ListenerSet conflict
Signed-off-by: zirain <zirain2009@gmail.com>
* fix and improve message
Signed-off-by: zirain <zirain2009@gmail.com>
* better display gateway listener
Signed-off-by: zirain <zirain2009@gmail.com>
---------
Signed-off-by: zirain <zirain2009@gmail.com>
(cherry picked from commit 9271654)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* fix: correct the config hot reload validation order (#9214)
Co-authored-by: Nimisha Mehta <nimisha_mehta@apple.com>
(cherry picked from commit 7e4492d)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* feat(helm): add CRDs dependency toggle (#8850)
* feat(helm): add CRDs dependency toggle
Allow skipping installation of the `crds` dependency on the gateway-helm
chart by providing a conditional `crds.enabled` variable. This boolean
defaults to `true` and toggles the inclusion of the dependency on the
parent chart.
ref: #8560
Signed-off-by: Gaston Festari <cilindrox@gmail.com>
* docs: update Helm installation steps
Use the `crds.enabled` variable in the examples instead of the
`--skip-crds` flag.
Update crds.enabled variable description.
Co-authored-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
Signed-off-by: Gaston Festari <cilindrox@gmail.com>
---------
Signed-off-by: Gaston Festari <cilindrox@gmail.com>
Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
Co-authored-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
(cherry picked from commit 62e01ce)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* fix: validate API Key auth ExtractFrom (#9250)
* fix: validate API Key auth ExtractFrom
Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
* add test
Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
* update
Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
* update
Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
* update
Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
---------
Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
(cherry picked from commit 50b1339)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* fix: XRateLimitHeadersOptionDisabled constant value must match CRD enum (#9224)
* fix: XRateLimitHeadersOptionDisabled constant value must match CRD enum
The constant held "Disabled" but the CRD enum specifies "Off", causing
xRateLimitHeaders: "Off" to silently fall through the translator switch
and always emit X-RateLimit headers.
Fixes #9223
Signed-off-by: gianniskt <gianniskt@gmail.com>
* fix: XRateLimitHeadersOptionDisabled constant value must match CRD enum
The constant held "Disabled" but the CRD enum specifies "Off", causing
xRateLimitHeaders: "Off" to silently fall through the translator switch
and always emit X-RateLimit headers.
Fixes #9223
Signed-off-by: gianniskt <gianniskt@gmail.com>
* fix: add release note for XRateLimitHeadersOptionDisabled fix
Signed-off-by: gianniskt <gianniskt@gmail.com>
---------
Signed-off-by: gianniskt <gianniskt@gmail.com>
Signed-off-by: Ioannis Koutroumpis <gianniskt@gmail.com>
(cherry picked from commit 43c9d8e)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* fix(ratelimit): shared global ratelimit with cost not working (#9245)
* fix(ratelimit): shared global ratelimit with cost not working
Signed-off-by: zirain <zirain2009@gmail.com>
* fix
Signed-off-by: zirain <zirain2009@gmail.com>
* fix mixed shared rule
Signed-off-by: zirain <zirain2009@gmail.com>
* add e2e
Signed-off-by: zirain <zirain2009@gmail.com>
---------
Signed-off-by: zirain <zirain2009@gmail.com>
(cherry picked from commit 7997791)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* fix(gatewayapi): http retry without backoff (#9238)
* fix http retry without backoff
Signed-off-by: zirain <zirain2009@gmail.com>
* release notes
Signed-off-by: zirain <zirain2009@gmail.com>
---------
Signed-off-by: zirain <zirain2009@gmail.com>
(cherry picked from commit 4872e53)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* fix: panic with empty deploy (#9365)
Signed-off-by: zirain <zirain2009@gmail.com>
(cherry picked from commit 42a0c2c)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* chore: restore current.yaml after cherry-picks
The cherry-picked bug-fix commits appended their release-note entries to
release-notes/current.yaml. Those notes are already finalized in
release-notes/v1.8.2.yaml, so restore current.yaml to its prior state to
avoid duplicating them in the next release cycle.
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* [release-1.8] bump ratelimit image to 1e50889b
Bump the Envoy Ratelimit image from ff287602 to 1e50889b in source and
the gateway-helm chart, and note the bump under security updates in the
v1.8.2 release notes.
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* testdata
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
* fix: use int64 format for ratelimit Requests field (#9377)
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
---------
Signed-off-by: Clemens Beck <mail@beckcl.dev>
Signed-off-by: jukie <10012479+jukie@users.noreply.github.com>
Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
Signed-off-by: Alexej Disterhoft <alexej.disterhoft@redcare-pharmacy.com>
Signed-off-by: Alexej Disterhoft <alexej@disterhoft.de>
Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: apkatsikas <apkatsikas@gmail.com>
Signed-off-by: Jules Dutel <jules.dutel@numberly.com>
Signed-off-by: Gaston Festari <cilindrox@gmail.com>
Signed-off-by: gianniskt <gianniskt@gmail.com>
Signed-off-by: Ioannis Koutroumpis <gianniskt@gmail.com>
Co-authored-by: Clemens Beck <125774647+beckcl@users.noreply.github.com>
Co-authored-by: Clemens Beck <mail@beckcl.dev>
Co-authored-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
Co-authored-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Alexej Disterhoft <alexej@disterhoft.de>
Co-authored-by: Arko Dasgupta <arkodg@users.noreply.github.com>
Co-authored-by: zirain <zirain2009@gmail.com>
Co-authored-by: Andrew Katsikas <apkatsikas@gmail.com>
Co-authored-by: jvlxz <jules.dutel@gmail.com>
Co-authored-by: Mengjia Liang <88690648+MengjiaLiang@users.noreply.github.com>
Co-authored-by: Nimisha Mehta <nimisha_mehta@apple.com>
Co-authored-by: Gaston Festari <cilindrox@gmail.com>
Co-authored-by: Ioannis Koutroumpis <gianniskt@gmail.com>1 parent eb4ffbd commit 6c2e80d
160 files changed
Lines changed: 3559 additions & 523 deletions
File tree
- api/v1alpha1
- validation
- charts
- gateway-crds-helm/templates/generated
- gateway-helm
- charts/crds/crds/generated
- internal
- envoygateway/config/loader
- testdata
- gatewayapi
- status
- testdata
- infrastructure/kubernetes/ratelimit/testdata/deployments
- xds/translator
- testdata
- in/xds-ir
- out
- ratelimit-config
- xds-ir
- release-notes
- site/content/en/latest
- api
- install
- tasks/operations
- test
- cel-validation
- e2e
- testdata
- tests
- helm
- gateway-crds-helm
- gateway-helm
- tools
- make
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| 24 | + | |
| 25 | + | |
24 | 26 | | |
25 | 27 | | |
26 | 28 | | |
| |||
36 | 38 | | |
37 | 39 | | |
38 | 40 | | |
39 | | - | |
| 41 | + | |
| 42 | + | |
40 | 43 | | |
41 | 44 | | |
42 | 45 | | |
43 | | - | |
| 46 | + | |
44 | 47 | | |
45 | 48 | | |
| 49 | + | |
| 50 | + | |
46 | 51 | | |
47 | 52 | | |
48 | 53 | | |
49 | | - | |
| 54 | + | |
50 | 55 | | |
51 | 56 | | |
| 57 | + | |
| 58 | + | |
52 | 59 | | |
53 | 60 | | |
54 | 61 | | |
55 | | - | |
| 62 | + | |
56 | 63 | | |
57 | 64 | | |
| 65 | + | |
| 66 | + | |
58 | 67 | | |
59 | 68 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
110 | 110 | | |
111 | 111 | | |
112 | 112 | | |
| 113 | + | |
113 | 114 | | |
114 | 115 | | |
115 | 116 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
81 | 81 | | |
82 | 82 | | |
83 | 83 | | |
84 | | - | |
| 84 | + | |
85 | 85 | | |
86 | 86 | | |
87 | 87 | | |
| |||
437 | 437 | | |
438 | 438 | | |
439 | 439 | | |
| 440 | + | |
440 | 441 | | |
441 | 442 | | |
442 | 443 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
34 | 34 | | |
35 | 35 | | |
36 | 36 | | |
37 | | - | |
| 37 | + | |
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
166 | 166 | | |
167 | 167 | | |
168 | 168 | | |
169 | | - | |
| 169 | + | |
170 | 170 | | |
171 | 171 | | |
172 | 172 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
251 | 251 | | |
252 | 252 | | |
253 | 253 | | |
| 254 | + | |
| 255 | + | |
| 256 | + | |
| 257 | + | |
| 258 | + | |
| 259 | + | |
| 260 | + | |
| 261 | + | |
| 262 | + | |
| 263 | + | |
| 264 | + | |
| 265 | + | |
| 266 | + | |
| 267 | + | |
| 268 | + | |
| 269 | + | |
| 270 | + | |
| 271 | + | |
| 272 | + | |
| 273 | + | |
| 274 | + | |
254 | 275 | | |
255 | 276 | | |
256 | 277 | | |
| |||
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1762 | 1762 | | |
1763 | 1763 | | |
1764 | 1764 | | |
1765 | | - | |
| 1765 | + | |
1766 | 1766 | | |
1767 | 1767 | | |
1768 | 1768 | | |
| |||
2146 | 2146 | | |
2147 | 2147 | | |
2148 | 2148 | | |
2149 | | - | |
| 2149 | + | |
2150 | 2150 | | |
2151 | 2151 | | |
2152 | 2152 | | |
| |||
Lines changed: 16 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
118 | 118 | | |
119 | 119 | | |
120 | 120 | | |
121 | | - | |
| 121 | + | |
122 | 122 | | |
123 | 123 | | |
124 | 124 | | |
125 | 125 | | |
126 | 126 | | |
127 | | - | |
| 127 | + | |
128 | 128 | | |
| 129 | + | |
129 | 130 | | |
| 131 | + | |
130 | 132 | | |
131 | 133 | | |
132 | 134 | | |
133 | 135 | | |
134 | 136 | | |
135 | | - | |
| 137 | + | |
136 | 138 | | |
| 139 | + | |
137 | 140 | | |
| 141 | + | |
138 | 142 | | |
139 | 143 | | |
140 | 144 | | |
141 | 145 | | |
142 | 146 | | |
143 | | - | |
| 147 | + | |
144 | 148 | | |
| 149 | + | |
145 | 150 | | |
| 151 | + | |
146 | 152 | | |
147 | 153 | | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
148 | 160 | | |
149 | 161 | | |
150 | 162 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
6 | | - | |
| 5 | + | |
| 6 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
27 | 27 | | |
28 | 28 | | |
29 | 29 | | |
| 30 | + | |
0 commit comments