Skip to content

[release/v1.3] release v1.3.1 cherry pick#5405

Merged
guydc merged 11 commits intoenvoyproxy:release/v1.3from
guydc:rel-v1.3.1-cp
Mar 4, 2025
Merged

[release/v1.3] release v1.3.1 cherry pick#5405
guydc merged 11 commits intoenvoyproxy:release/v1.3from
guydc:rel-v1.3.1-cp

Conversation

@guydc
Copy link
Copy Markdown
Contributor

@guydc guydc commented Mar 4, 2025

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Release Notes: Yes/No

shawnh2 and others added 9 commits March 4, 2025 12:47
@shawnh2 @zirain @guydc
feat: add defaulter for gateway-api resources loading from file (envo…
940638f 
…yproxy#5232)

* add defaulter that can set default values for our crd

Signed-off-by: shawnh2 <shawnhxh@outlook.com>

* resolve conflicts

Signed-off-by: shawnh2 <shawnhxh@outlook.com>

* add gateway schema defaulter while loading resources and fix all existing tests

Signed-off-by: shawnh2 <shawnhxh@outlook.com>

* add load test for all supported kind resources

Signed-off-by: shawnh2 <shawnhxh@outlook.com>

* fix lint

Signed-off-by: shawnh2 <shawnhxh@outlook.com>

---------

Signed-off-by: shawnh2 <shawnhxh@outlook.com>
Co-authored-by: zirain <zirain2009@gmail.com>
(cherry picked from commit 7146fe8)
Signed-off-by: Guy Daich <guy.daich@sap.com>
@tobrien-nydig @zirain
@dependabot @guydc
fix: Allow weights to be zero on endpoints (envoyproxy#5278)
6a90852 
* fix: Allow weights to be zero on endpoints

Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* chore: bump go to 1.24 (envoyproxy#5287)

* chore: bump go to 1.24

Signed-off-by: zirain <zirain2009@gmail.com>

* fix

Signed-off-by: zirain <zirain2009@gmail.com>

---------

Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* build(deps): bump sigs.k8s.io/kind from 0.26.0 to 0.27.0 in /tools/src/kind (envoyproxy#5295)

build(deps): bump sigs.k8s.io/kind in /tools/src/kind

Bumps [sigs.k8s.io/kind](https://github.com/kubernetes-sigs/kind) from 0.26.0 to 0.27.0.
- [Release notes](https://github.com/kubernetes-sigs/kind/releases)
- [Commits](kubernetes-sigs/kind@v0.26.0...v0.27.0)

---
updated-dependencies:
- dependency-name: sigs.k8s.io/kind
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* build(deps): bump helm.sh/helm/v3 from 3.17.0 to 3.17.1 (envoyproxy#5291)

Bumps [helm.sh/helm/v3](https://github.com/helm/helm) from 3.17.0 to 3.17.1.
- [Release notes](https://github.com/helm/helm/releases)
- [Commits](helm/helm@v3.17.0...v3.17.1)

---
updated-dependencies:
- dependency-name: helm.sh/helm/v3
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* build(deps): bump the golang-org group across 2 directories with 1 update (envoyproxy#5290)

Bumps the golang-org group with 1 update in the / directory: [golang.org/x/net](https://github.com/golang/net).
Bumps the golang-org group with 1 update in the /examples/extension-server directory: [golang.org/x/net](https://github.com/golang/net).

Updates `golang.org/x/net` from 0.34.0 to 0.35.0
- [Commits](golang/net@v0.34.0...v0.35.0)

Updates `golang.org/x/net` from 0.34.0 to 0.35.0
- [Commits](golang/net@v0.34.0...v0.35.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: golang-org
- dependency-name: golang.org/x/net
  dependency-type: indirect
  update-type: version-update:semver-minor
  dependency-group: golang-org
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* build(deps): bump github.com/golangci/golangci-lint from 1.63.4 to 1.64.5 in /tools/src/golangci-lint (envoyproxy#5294)

build(deps): bump github.com/golangci/golangci-lint

Bumps [github.com/golangci/golangci-lint](https://github.com/golangci/golangci-lint) from 1.63.4 to 1.64.5.
- [Release notes](https://github.com/golangci/golangci-lint/releases)
- [Changelog](https://github.com/golangci/golangci-lint/blob/master/CHANGELOG.md)
- [Commits](golangci/golangci-lint@v1.63.4...v1.64.5)

---
updated-dependencies:
- dependency-name: github.com/golangci/golangci-lint
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* Add test case

Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* Add test case

Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* one more test

Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* one more test

Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* remove

Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* Found the bug for real

Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* Focus on gatewayapi

Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* cleanup

Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* cleanup

Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* build(deps): bump github.com/evanphx/json-patch/v5 from 5.9.0 to 5.9.11 (envoyproxy#5293)

Bumps [github.com/evanphx/json-patch/v5](https://github.com/evanphx/json-patch) from 5.9.0 to 5.9.11.
- [Release notes](https://github.com/evanphx/json-patch/releases)
- [Commits](evanphx/json-patch@v5.9.0...v5.9.11)

---
updated-dependencies:
- dependency-name: github.com/evanphx/json-patch/v5
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* api: BackendTrafficPolicy DNSLookupFamily (envoyproxy#5249)

* api: BackendTrafficPolicy DNSLookupFamily

Signed-off-by: Guy Daich <guy.daich@sap.com>

* fix api

Signed-off-by: Guy Daich <guy.daich@sap.com>

* fix enum

Signed-off-by: Guy Daich <guy.daich@sap.com>

---------

Signed-off-by: Guy Daich <guy.daich@sap.com>
Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* clean up logic on if

Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* Update logic and add udp tests

Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* fix udp tests

Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* chore: ignore `sched.co` (envoyproxy#5305)

chore: ignore

Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

* newline

Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>

---------

Signed-off-by: Tim OBrien <timothy.obrien@nydig.com>
Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Guy Daich <guy.daich@sap.com>
Signed-off-by: tobrien-nydig <100732440+tobrien-nydig@users.noreply.github.com>
Co-authored-by: zirain <zirain2009@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Guy Daich <guy.daich@sap.com>
(cherry picked from commit 1f9a1bd)
Signed-off-by: Guy Daich <guy.daich@sap.com>
@bstasz-bonrepublic @arkodg @guydc
fix: Standalone mode - Secret and ConfigMap parsing (envoyproxy#5329)
dc6e382 
* Added Secret and ConfigMap parsing

Signed-off-by: Balazs Stasz <balazs.stasz@bonrepublic.com>

* Updated release notes

Signed-off-by: Balazs Stasz <balazs.stasz@bonrepublic.com>

* Fixed lint issues

Signed-off-by: Balazs Stasz <balazs.stasz@bonrepublic.com>

* Fixed gen-check issues

Signed-off-by: Balazs Stasz <balazs.stasz@bonrepublic.com>

* Added new test cases for Secret

Signed-off-by: Balazs Stasz <balazs.stasz@bonrepublic.com>

---------

Signed-off-by: Balazs Stasz <balazs.stasz@bonrepublic.com>
Signed-off-by: Arko Dasgupta <arkodg@users.noreply.github.com>
Co-authored-by: Arko Dasgupta <arkodg@users.noreply.github.com>
(cherry picked from commit a41e992)
Signed-off-by: Guy Daich <guy.daich@sap.com>
@arkodg @guydc
fix: backendSettings for extAuth (envoyproxy#5372)
099cd94 
* fix: backendSettings for extAuth

Fixes: envoyproxy#5371

Signed-off-by: Arko Dasgupta <arko@tetrate.io>

* release note

Signed-off-by: Arko Dasgupta <arko@tetrate.io>

---------

Signed-off-by: Arko Dasgupta <arko@tetrate.io>
(cherry picked from commit 1c0eca6)
Signed-off-by: Guy Daich <guy.daich@sap.com>
@arkodg @guydc
conformance: skip, skip-test features (envoyproxy#5399)
9860896 
So they dont show up in the supportedFeatures section
of the conformance report

Relates to kubernetes-sigs/gateway-api#3651 (comment)

Signed-off-by: Arko Dasgupta <arko@tetrate.io>
(cherry picked from commit bd53727)
Signed-off-by: Guy Daich <guy.daich@sap.com>
@zhaohuabing @guydc
validate all xds resources before returning the translation result (e…
fcc0753 
…nvoyproxy#5148)

* validate all xds resources before returning the translation result

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>

* fix lint

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>

* Log errors when envoy rejects xDS resources

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>

* remove duplicated line

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>

---------

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
(cherry picked from commit b1bf609)
Signed-off-by: Guy Daich <guy.daich@sap.com>
@zirain @guydc
chore: bump go 1.23.6 (envoyproxy#5204)
a2dd4db 
(cherry picked from commit 48459d4)
Signed-off-by: Guy Daich <guy.daich@sap.com>
@zirain @dependabot @guydc
build(deps): bump the k8s-io group (envoyproxy#5309)
7bb9acf 
* build(deps): bump the k8s-io group across 2 directories with 7 updates

Bumps the k8s-io group with 5 updates in the / directory:

| Package | From | To |
| --- | --- | --- |
| [k8s.io/api](https://github.com/kubernetes/api) | `0.32.1` | `0.32.2` |
| [k8s.io/apiextensions-apiserver](https://github.com/kubernetes/apiextensions-apiserver) | `0.32.1` | `0.32.2` |
| [k8s.io/cli-runtime](https://github.com/kubernetes/cli-runtime) | `0.32.1` | `0.32.2` |
| [k8s.io/kubectl](https://github.com/kubernetes/kubectl) | `0.32.1` | `0.32.2` |
| [sigs.k8s.io/controller-runtime](https://github.com/kubernetes-sigs/controller-runtime) | `0.20.1` | `0.20.2` |

Bumps the k8s-io group with 2 updates in the /examples/extension-server directory: [k8s.io/apimachinery](https://github.com/kubernetes/apimachinery) and [sigs.k8s.io/controller-runtime](https://github.com/kubernetes-sigs/controller-runtime).

Updates `k8s.io/api` from 0.32.1 to 0.32.2
- [Commits](kubernetes/api@v0.32.1...v0.32.2)

Updates `k8s.io/apiextensions-apiserver` from 0.32.1 to 0.32.2
- [Release notes](https://github.com/kubernetes/apiextensions-apiserver/releases)
- [Commits](kubernetes/apiextensions-apiserver@v0.32.1...v0.32.2)

Updates `k8s.io/apimachinery` from 0.32.1 to 0.32.2
- [Commits](kubernetes/apimachinery@v0.32.1...v0.32.2)

Updates `k8s.io/cli-runtime` from 0.32.1 to 0.32.2
- [Commits](kubernetes/cli-runtime@v0.32.1...v0.32.2)

Updates `k8s.io/client-go` from 0.32.1 to 0.32.2
- [Changelog](https://github.com/kubernetes/client-go/blob/master/CHANGELOG.md)
- [Commits](kubernetes/client-go@v0.32.1...v0.32.2)

Updates `k8s.io/kubectl` from 0.32.1 to 0.32.2
- [Commits](kubernetes/kubectl@v0.32.1...v0.32.2)

Updates `sigs.k8s.io/controller-runtime` from 0.20.1 to 0.20.2
- [Release notes](https://github.com/kubernetes-sigs/controller-runtime/releases)
- [Changelog](https://github.com/kubernetes-sigs/controller-runtime/blob/main/RELEASE.md)
- [Commits](kubernetes-sigs/controller-runtime@v0.20.1...v0.20.2)

Updates `k8s.io/apimachinery` from 0.32.1 to 0.32.2
- [Commits](kubernetes/apimachinery@v0.32.1...v0.32.2)

Updates `sigs.k8s.io/controller-runtime` from 0.20.1 to 0.20.2
- [Release notes](https://github.com/kubernetes-sigs/controller-runtime/releases)
- [Changelog](https://github.com/kubernetes-sigs/controller-runtime/blob/main/RELEASE.md)
- [Commits](kubernetes-sigs/controller-runtime@v0.20.1...v0.20.2)

---
updated-dependencies:
- dependency-name: k8s.io/api
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: k8s-io
- dependency-name: k8s.io/apiextensions-apiserver
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: k8s-io
- dependency-name: k8s.io/apimachinery
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: k8s-io
- dependency-name: k8s.io/cli-runtime
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: k8s-io
- dependency-name: k8s.io/client-go
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: k8s-io
- dependency-name: k8s.io/kubectl
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: k8s-io
- dependency-name: sigs.k8s.io/controller-runtime
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: k8s-io
- dependency-name: k8s.io/apimachinery
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: k8s-io
- dependency-name: sigs.k8s.io/controller-runtime
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: k8s-io
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump k8s.io

Signed-off-by: zirain <zirain2009@gmail.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: zirain <zirain2009@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
(cherry picked from commit 3e08722)
Signed-off-by: Guy Daich <guy.daich@sap.com>
@zirain @guydc
chore: fix CVE (envoyproxy#5360)
ff7b485 
* chore: fix CVE

Signed-off-by: zirain <zirain2009@gmail.com>

* lint

Signed-off-by: zirain <zirain2009@gmail.com>

---------

Signed-off-by: zirain <zirain2009@gmail.com>
(cherry picked from commit c8f518b)
Signed-off-by: Guy Daich <guy.daich@sap.com>
@guydc guydc requested a review from a team as a code owner March 4, 2025 20:58
@zirain @guydc
chore: bump go 1.23.5 (envoyproxy#5190)
024f041 
* chore: bump go

Signed-off-by: zirain <zirain2009@gmail.com>

* bump controller-gen

Signed-off-by: zirain <zirain2009@gmail.com>

* 1.23.5

Signed-off-by: zirain <zirain2009@gmail.com>

* revert

Signed-off-by: zirain <zirain2009@gmail.com>

---------

Signed-off-by: zirain <zirain2009@gmail.com>
(cherry picked from commit d50be42)
Signed-off-by: Guy Daich <guy.daich@sap.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 4, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 66.66667% with 102 lines in your changes missing coverage. Please review.

Project coverage is 67.91%. Comparing base (358bed5) to head (0b65cab).
Report is 1 commits behind head on release/v1.3.

Files with missing lines Patch % Lines
internal/gatewayapi/resource/defaulter.go 62.16% 42 Missing and 14 partials ⚠️
internal/xds/types/resourceversiontable.go 25.00% 9 Missing ⚠️
internal/xds/translator/authorization.go 20.00% 0 Missing and 8 partials ⚠️
internal/utils/proto/proto.go 71.42% 4 Missing and 2 partials ⚠️
internal/xds/cache/snapshotcache.go 0.00% 6 Missing ⚠️
internal/xds/translator/translator.go 16.66% 2 Missing and 3 partials ⚠️
internal/xds/translator/custom_response.go 33.33% 0 Missing and 4 partials ⚠️
internal/gatewayapi/resource/load.go 91.66% 2 Missing and 1 partial ⚠️
internal/xds/translator/compressor.go 25.00% 0 Missing and 3 partials ⚠️
internal/xds/translator/basicauth.go 33.33% 0 Missing and 2 partials ⚠️
Additional details and impacted files 
@@               Coverage Diff                @@
##           release/v1.3    #5405      +/-   ##
================================================
+ Coverage         66.91%   67.91%   +0.99%     
================================================
  Files               210      210              
  Lines             32992    33125     +133     
================================================
+ Hits              22076    22496     +420     
+ Misses             9580     9272     -308     
- Partials           1336     1357      +21

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@guydc
fix dependencies
0b65cab 
Signed-off-by: Guy Daich <guy.daich@sap.com>
@guydc guydc merged commit b6f69de into envoyproxy:release/v1.3 Mar 4, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zirain zirain zirain approved these changes

@arkodg arkodg arkodg approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@guydc @zirain @arkodg @shawnh2 @tobrien-nydig @bstasz-bonrepublic @zhaohuabing