Skip to content

[release/v1.0] cherry pick from main to release/v1.0#2911

Merged
Xunzhuo merged 68 commits intoenvoyproxy:release/v1.0from
Xunzhuo:cherry-pick-v1.0
Mar 13, 2024
Merged

[release/v1.0] cherry pick from main to release/v1.0#2911
Xunzhuo merged 68 commits intoenvoyproxy:release/v1.0from
Xunzhuo:cherry-pick-v1.0

Conversation

@Xunzhuo
Copy link
Copy Markdown
Member

@Xunzhuo Xunzhuo commented Mar 13, 2024

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

cherrypick d8ff060...b5f4a07

@Xunzhuo Xunzhuo requested a review from a team as a code owner March 13, 2024 08:55
Xunzhuo and others added 29 commits March 13, 2024 17:04
…#2785)

fix: complete missing release notes and re organize it

Signed-off-by: bitliu <[email protected]>
* chore: add testdata to passive health checks

Signed-off-by: yeedove <[email protected]>

* fix test

Signed-off-by: yeedove <[email protected]>

---------

Signed-off-by: yeedove <[email protected]>
* Delete unused status keys in gatewayapi-runner

Signed-off-by: Yuneui Jeong <[email protected]>

* Delete unused status keys in xds-translator runner

Signed-off-by: Yuneui Jeong <[email protected]>

* Add tests and fix code to pass all tests

Signed-off-by: Yuneui Jeong <[email protected]>

* Cover more

Signed-off-by: Yuneui <[email protected]>

* Change struct's name and other minor fixes

Signed-off-by: Yuneui Jeong <[email protected]>

---------

Signed-off-by: Yuneui Jeong <[email protected]>
Signed-off-by: Yuneui <[email protected]>
* Add support for using the system truststore with upstream TLS.

Signed-off-by: Lior Okman <[email protected]>

* Make the linter happy

Signed-off-by: Lior Okman <[email protected]>

---------

Signed-off-by: Lior Okman <[email protected]>
* docs: refactor user guides

Signed-off-by: bitliu <[email protected]>

* fix: relative paths

Signed-off-by: bitliu <[email protected]>

---------

Signed-off-by: bitliu <[email protected]>
* fix: gen-check error

Signed-off-by: bitliu <[email protected]>

* run lint for docs

Signed-off-by: bitliu <[email protected]>

---------

Signed-off-by: bitliu <[email protected]>
* refactor:set[T] instead of map[T]bool

Signed-off-by: Dennis Zhou <[email protected]>

* fix lint

Signed-off-by: Dennis Zhou <[email protected]>

---------

Signed-off-by: Dennis Zhou <[email protected]>
Co-authored-by: Xunzhuo <[email protected]>
…urces to IR (envoyproxy#2799)

* Added an option to translate to IR representation.

Signed-off-by: Lior Okman <[email protected]>

* Added a unit test, and made sure that existing services have an IP
address.

Signed-off-by: Lior Okman <[email protected]>

* Add omitempty where needed.

Signed-off-by: Lior Okman <[email protected]>

* Make gen-check happy

Signed-off-by: Lior Okman <[email protected]>

* Added some documentation.

Signed-off-by: Lior Okman <[email protected]>

---------

Signed-off-by: Lior Okman <[email protected]>
* docs: basic auth example use https

Signed-off-by: phantooom <[email protected]>

* docs: refactor user guides (envoyproxy#2797)

* docs: refactor user guides

Signed-off-by: bitliu <[email protected]>

* fix: relative paths

Signed-off-by: bitliu <[email protected]>

---------

Signed-off-by: bitliu <[email protected]>
Signed-off-by: phantooom <[email protected]>

* Fix gen check (envoyproxy#2814)

* fix: gen-check error

Signed-off-by: bitliu <[email protected]>

* run lint for docs

Signed-off-by: bitliu <[email protected]>

---------

Signed-off-by: bitliu <[email protected]>
Signed-off-by: phantooom <[email protected]>

* refactor: set instead of map for mergeGateways (envoyproxy#2803)

* refactor:set[T] instead of map[T]bool

Signed-off-by: Dennis Zhou <[email protected]>

* fix lint

Signed-off-by: Dennis Zhou <[email protected]>

---------

Signed-off-by: Dennis Zhou <[email protected]>
Co-authored-by: Xunzhuo <[email protected]>
Signed-off-by: phantooom <[email protected]>

* Update site/content/en/latest/user/security/basic-auth.md

Co-authored-by: Huabing Zhao <[email protected]>
Signed-off-by: zou rui <[email protected]>

---------

Signed-off-by: phantooom <[email protected]>
Signed-off-by: bitliu <[email protected]>
Signed-off-by: Dennis Zhou <[email protected]>
Signed-off-by: zou rui <[email protected]>
Co-authored-by: Xunzhuo <[email protected]>
Co-authored-by: Dennis Zhou <[email protected]>
Co-authored-by: Huabing Zhao <[email protected]>
)

* add referenced BackendRefs for ExtAuth to Resource Tree

Signed-off-by: huabing zhao <[email protected]>

* clean up the controller code

Signed-off-by: huabing zhao <[email protected]>

* minor changes

Signed-off-by: huabing zhao <[email protected]>

* return errors

Signed-off-by: huabing zhao <[email protected]>

* fix validate error

Signed-off-by: huabing zhao <[email protected]>

* fix gen

Signed-off-by: huabing zhao <[email protected]>

* Support BackendTLSPolicy for the Ext HTTP/GRPC auth service

Signed-off-by: huabing zhao <[email protected]>

* fix lint

Signed-off-by: huabing zhao <[email protected]>

* fix gen

Signed-off-by: huabing zhao <[email protected]>

* check cross-ns reference grant

Signed-off-by: huabing zhao <[email protected]>

* fix test

Signed-off-by: huabing zhao <[email protected]>

* fix test

Signed-off-by: huabing zhao <[email protected]>

---------

Signed-off-by: huabing zhao <[email protected]>
* fix bootstrap merge

Signed-off-by: zirain <[email protected]>

* refactor validateBootstrap

Signed-off-by: zirain <[email protected]>

* lint

Signed-off-by: zirain <[email protected]>

* update test

Signed-off-by: zirain <[email protected]>

---------

Signed-off-by: zirain <[email protected]>
* stop populating ReasonTargetNotFound for all the policies

Signed-off-by: shawnh2 <[email protected]>

* add test to ensure the status is expected

Signed-off-by: shawnh2 <[email protected]>

* fix gen-check and lint

Signed-off-by: shawnh2 <[email protected]>

---------

Signed-off-by: shawnh2 <[email protected]>
* docs: update EnvoyProxy logs

Signed-off-by: zirain <[email protected]>

* lint

Signed-off-by: zirain <[email protected]>

---------

Signed-off-by: zirain <[email protected]>
* fix: remove default replicas function

Signed-off-by: Ardika Bagus <[email protected]>

* chore: omit replicas because nil equal to 1 by default

Signed-off-by: Ardika Bagus <[email protected]>

* chore: add a note when a user is being explicit on deployment replicas

Signed-off-by: Ardika Bagus <[email protected]>

---------

Signed-off-by: Ardika Bagus <[email protected]>
* docs: Routing outside k8s

Fixes: envoyproxy#2482

Signed-off-by: Arko Dasgupta <[email protected]>

* updates

Signed-off-by: Arko Dasgupta <[email protected]>

---------

Signed-off-by: Arko Dasgupta <[email protected]>
…nvoyproxy#2825)

Bumps google.golang.org/protobuf from 1.32.0 to 1.33.0.

---
updated-dependencies:
- dependency-name: google.golang.org/protobuf
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…2826)

Bumps [golang.org/x/net](https://github.com/golang/net) from 0.21.0 to 0.22.0.
- [Commits](golang/net@v0.21.0...v0.22.0)

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

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…voyproxy#2827)

Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.8.4 to 1.9.0.
- [Release notes](https://github.com/stretchr/testify/releases)
- [Commits](stretchr/testify@v1.8.4...v1.9.0)

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

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…proxy#2835)

try to create every secret instead of returning eraly

Signed-off-by: huabing zhao <[email protected]>
…proxy#2829)

Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.62.0 to 1.62.1.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.62.0...v1.62.1)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* e2e tests for http ext auth

Signed-off-by: huabing zhao <[email protected]>

* export util methods to avoid unparam link issues

Signed-off-by: huabing zhao <[email protected]>

* fixt test

Signed-off-by: Huabing Zhao <[email protected]>

---------

Signed-off-by: huabing zhao <[email protected]>
Signed-off-by: Huabing Zhao <[email protected]>
fix existing secret

Signed-off-by: zirain <[email protected]>
dependabot bot and others added 23 commits March 13, 2024 17:04
…nvoyproxy#2871)

Bumps [github.com/prometheus/common](https://github.com/prometheus/common) from 0.49.0 to 0.50.0.
- [Release notes](https://github.com/prometheus/common/releases)
- [Commits](prometheus/common@v0.49.0...v0.50.0)

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

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…#2873)

Bumps [fortio.org/fortio](https://github.com/fortio/fortio) from 1.63.3 to 1.63.4.
- [Release notes](https://github.com/fortio/fortio/releases)
- [Commits](fortio/fortio@v1.63.3...v1.63.4)

---
updated-dependencies:
- dependency-name: fortio.org/fortio
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
add tetrate to adopters

Signed-off-by: huabing zhao <[email protected]>
…2876)

* Don't override the ALPN array if HTTP/3 is enabled.

Signed-off-by: Lior Okman <[email protected]>

* Removed the unneeded CEL validation for HTTP/3 and ALPN, as well as the
CEL tests.

Signed-off-by: Lior Okman <[email protected]>

* Also regenerate the CRD.

Signed-off-by: Lior Okman <[email protected]>

---------

Signed-off-by: Lior Okman <[email protected]>
* [e2e] eg release upgrade test

Signed-off-by: Alexander Volchok <[email protected]>

* fixing linit

Signed-off-by: Alexander Volchok <[email protected]>

* Update test/e2e/tests/eg_upgrade.go

Co-authored-by: zirain <[email protected]>
Signed-off-by: Alex Volchok <[email protected]>

* Update test/e2e/tests/eg_upgrade.go

Co-authored-by: zirain <[email protected]>
Signed-off-by: Alex Volchok <[email protected]>

* adding updated go mod

Signed-off-by: Alexander Volchok <[email protected]>

* fix tests

Signed-off-by: Alexander Volchok <[email protected]>

* move eg upgrade tests to a dedicated suit

Signed-off-by: Alexander Volchok <[email protected]>

* removing unused

Signed-off-by: Alexander Volchok <[email protected]>

* fix code review feedbacks and move e2e clean after the eg upgrades suit

Signed-off-by: Alexander Volchok <[email protected]>

* don't clean after this step yet

Signed-off-by: Alexander Volchok <[email protected]>

* increase helm install / upgrade default timeouts

Signed-off-by: Alexander Volchok <[email protected]>

* restructure test order add an option to execute a single test

Signed-off-by: Alexander Volchok <[email protected]>

* fix kube make single test exec

Signed-off-by: Alexander Volchok <[email protected]>

* change to rc version

Signed-off-by: Alexander Volchok <[email protected]>

* removing loadtest part, changing to simple requests

Signed-off-by: Alexander Volchok <[email protected]>

---------

Signed-off-by: Alexander Volchok <[email protected]>
Signed-off-by: Alex Volchok <[email protected]>
Co-authored-by: zirain <[email protected]>
* docs for ext auth

Signed-off-by: huabing zhao <[email protected]>

* fix lint

Signed-off-by: huabing zhao <[email protected]>

---------

Signed-off-by: huabing zhao <[email protected]>
remove the \

Signed-off-by: huabing zhao <[email protected]>
* backend tls docs

Signed-off-by: Guy Daich <[email protected]>

* fix somy copy-paste mistakes

Signed-off-by: Guy Daich <[email protected]>

* fix typo

Signed-off-by: Guy Daich <[email protected]>

---------

Signed-off-by: Guy Daich <[email protected]>
* add PolicyStatus for BTP

Signed-off-by: shawnh2 <[email protected]>

* fix gen-check

Signed-off-by: shawnh2 <[email protected]>

* fix ns problem, add more test and modify controller behavior

Signed-off-by: shawnh2 <[email protected]>

* fix lint

Signed-off-by: shawnh2 <[email protected]>

* make gateway as the ancestor of btp if it is targeting to the gateway

Signed-off-by: shawnh2 <[email protected]>

* fix linter

Signed-off-by: shawnh2 <[email protected]>

* fix go.mod

Signed-off-by: shawnh2 <[email protected]>

* do some polish

Signed-off-by: shawnh2 <[email protected]>

---------

Signed-off-by: shawnh2 <[email protected]>
)

* Change the Merge behavior to Replace for SecurityPolicy

Signed-off-by: huabing zhao <[email protected]>

* add another http route

Signed-off-by: huabing zhao <[email protected]>

---------

Signed-off-by: huabing zhao <[email protected]>
* e2e: add backend weighted

Signed-off-by: ShyunnY <[email protected]>

* fix: Fix weight calculation issue and use AlmostEqual func

Signed-off-by: ShyunnY <[email protected]>

* fix: add additional comments

Signed-off-by: ShyunnY <[email protected]>

---------

Signed-off-by: ShyunnY <[email protected]>
Co-authored-by: Xunzhuo <[email protected]>
…om conflicting (envoyproxy#2786)

* * Validate that multiple policies that affect listener configuration don't map to
  the same listener filter chain.
* Change the XDS listener generation so that instead of
  defaultFilterChain for non-TLS routes, a filterChain with a
  destinationPort matcher is used.
  This allows multiple policies attached to non-TLS listeners that
  differ on the destination port to provide different policies without
  conflicting.

Signed-off-by: Lior Okman <[email protected]>

* Make hostname based routing work again for non-TLS listeners

Signed-off-by: Lior Okman <[email protected]>

* Fixed testdata for egctl

Signed-off-by: Lior Okman <[email protected]>

* Make the linter happy

Signed-off-by: Lior Okman <[email protected]>

* Added a unit-test

Signed-off-by: Lior Okman <[email protected]>

* Make the linter happy

Signed-off-by: Lior Okman <[email protected]>

* Update an e2e test with the new filterChain patch

Signed-off-by: Lior Okman <[email protected]>

* Revert changing the XDS translation, since a new listener is created
anyways for each port.

Signed-off-by: Lior Okman <[email protected]>

* Also revert the xds change in the e2e test.

Signed-off-by: Lior Okman <[email protected]>

* Don't need to go over the full XDSIR map - just the current gateway.

Signed-off-by: Lior Okman <[email protected]>

* Refactored to separate the validation and the translation.

Renamed the helper method to a more generic name.

Signed-off-by: Lior Okman <[email protected]>

---------

Signed-off-by: Lior Okman <[email protected]>
Co-authored-by: Guy Daich <[email protected]>
* remove ProcessBackendTLSPoliciesAncestorRef

Signed-off-by: huabing zhao <[email protected]>

* address comments

Signed-off-by: huabing zhao <[email protected]>

---------

Signed-off-by: huabing zhao <[email protected]>
…roxy#2888)

* Change the Merge behavior to Replace for BackendTrafficPolicy

Signed-off-by: huabing zhao <[email protected]>

* address comments

Signed-off-by: huabing zhao <[email protected]>

---------

Signed-off-by: huabing zhao <[email protected]>
* skip publishing empty status for policies

* envoyproxy#2802 skips computing status
if a target resource cannot be found, mainly because that target maybe
irrelevant to this specific translation, its hard to proactively find
that out in the provider layer

* This fix ensures that any empty status is not published and resets any
existing status for a policy

Signed-off-by: Arko Dasgupta <[email protected]>

* also fix for envoypatchpolicy

Signed-off-by: Arko Dasgupta <[email protected]>

* also discard status for backendtlspolicy

Signed-off-by: Arko Dasgupta <[email protected]>

---------

Signed-off-by: Arko Dasgupta <[email protected]>
…proxy#2881)

* docs: multiple gatewayclass and merge gateways deployment mode

Signed-off-by: Karol Szwaj <[email protected]>

* add merged-gateways example

Signed-off-by: Karol Szwaj <[email protected]>

* md lint

Signed-off-by: Karol Szwaj <[email protected]>

* yaml lint

Signed-off-by: Karol Szwaj <[email protected]>

* add user guides

Signed-off-by: Karol Szwaj <[email protected]>

---------

Signed-off-by: Karol Szwaj <[email protected]>
Co-authored-by: Xunzhuo <[email protected]>
* add PolicyStatus for CTP

Signed-off-by: shawnh2 <[email protected]>

* fix gen-check

Signed-off-by: shawnh2 <[email protected]>

* revert discard policy status

Signed-off-by: shawnh2 <[email protected]>

---------

Signed-off-by: shawnh2 <[email protected]>
* use gwapiv1a2.PolicyStatus for SecurityPolicy Status

Signed-off-by: huabing zhao <[email protected]>

* fix lint

Signed-off-by: huabing zhao <[email protected]>

* add test for cross-ns refs

Signed-off-by: huabing zhao <[email protected]>

* add todo

Signed-off-by: huabing zhao <[email protected]>

* Update internal/gatewayapi/securitypolicy.go

Co-authored-by: sh2 <[email protected]>
Signed-off-by: Huabing Zhao <[email protected]>

* address comments

Signed-off-by: huabing zhao <[email protected]>

---------

Signed-off-by: huabing zhao <[email protected]>
Signed-off-by: Huabing Zhao <[email protected]>
Co-authored-by: sh2 <[email protected]>
fix oidc doc

Signed-off-by: huabing zhao <[email protected]>
* add v1.0.0 release note

Signed-off-by: bitliu <[email protected]>

* generate v1.0 release page

Signed-off-by: bitliu <[email protected]>

* add v1.0.0 release announcement

Signed-off-by: bitliu <[email protected]>

* generate v1.0.0 docs

Signed-off-by: bitliu <[email protected]>

* update site links

Signed-off-by: bitliu <[email protected]>

* fix linter

Signed-off-by: bitliu <[email protected]>

---------

Signed-off-by: bitliu <[email protected]>
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 13, 2024

Codecov Report

Attention: Patch coverage is 72.26833% with 401 lines in your changes are missing coverage. Please review.

Project coverage is 64.45%. Comparing base (b34aef9) to head (8bb6ed1).

Files Patch % Lines
internal/provider/kubernetes/controller.go 39.24% 117 Missing and 27 partials ⚠️
internal/gatewayapi/runner/runner.go 53.79% 61 Missing and 12 partials ⚠️
internal/status/policy.go 0.00% 47 Missing ⚠️
internal/gatewayapi/backendtrafficpolicy.go 88.03% 23 Missing and 5 partials ⚠️
internal/xds/translator/translator.go 72.58% 16 Missing and 1 partial ⚠️
internal/xds/translator/runner/runner.go 18.75% 11 Missing and 2 partials ⚠️
internal/xds/bootstrap/util.go 40.00% 8 Missing and 4 partials ⚠️
internal/gatewayapi/backendtlspolicy.go 90.43% 7 Missing and 4 partials ⚠️
internal/gatewayapi/securitypolicy.go 94.28% 8 Missing and 2 partials ⚠️
internal/cmd/egctl/translate.go 77.50% 6 Missing and 3 partials ⚠️
... and 7 more
Additional details and impacted files
@@               Coverage Diff                @@
##           release/v1.0    #2911      +/-   ##
================================================
+ Coverage         63.44%   64.45%   +1.00%     
================================================
  Files               125      122       -3     
  Lines             20604    21039     +435     
================================================
+ Hits              13073    13561     +488     
+ Misses             6693     6630      -63     
- Partials            838      848      +10     

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

@Xunzhuo
Copy link
Copy Markdown
Member Author

Xunzhuo commented Mar 13, 2024

/retest

1 similar comment
@Xunzhuo
Copy link
Copy Markdown
Member Author

Xunzhuo commented Mar 13, 2024

/retest

@Xunzhuo Xunzhuo merged commit 72c0cc7 into envoyproxy:release/v1.0 Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.