-
Notifications
You must be signed in to change notification settings - Fork 362
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
AttachedRoutes not necessarily on accepted routes and listeners #1916
Labels
area/conformance
Gateway API Conformance Related Issues
area/translator
Issues related to Gateway's translation service, e.g. translating Gateway APIs into the IR.
Milestone
Comments
tanujd11
changed the title
AttachedRoutes not necessarily on allowed routes and listeners
AttachedRoutes not necessarily on accepted routes and listeners
Sep 28, 2023
arkodg
added
help wanted
Extra attention is needed
area/translator
Issues related to Gateway's translation service, e.g. translating Gateway APIs into the IR.
area/conformance
Gateway API Conformance Related Issues
labels
Sep 28, 2023
/assign |
xref kubernetes-sigs/gateway-api#2477 adds a conformance test upstream that exercises the attachedRoutes spec update. |
This was referenced Oct 18, 2023
@arkodg this issue is more involved than what I expected. I created the above draft PR that can be used as guidance for a proper PR to fix this issue. |
arkodg
added a commit
to arkodg/gateway
that referenced
this issue
Oct 27, 2023
* Fixes: envoyproxy#2077 * Fixes: envoyproxy#1916 Signed-off-by: Arko Dasgupta <[email protected]>
arkodg
added a commit
to arkodg/gateway
that referenced
this issue
Oct 27, 2023
* Fixes: envoyproxy#2077 * Fixes: envoyproxy#1916 Signed-off-by: Arko Dasgupta <[email protected]>
arkodg
added a commit
to arkodg/gateway
that referenced
this issue
Oct 27, 2023
* Fixes: envoyproxy#2077 * Fixes: envoyproxy#1916 Signed-off-by: Arko Dasgupta <[email protected]>
arkodg
added a commit
to arkodg/gateway
that referenced
this issue
Nov 1, 2023
* Fixes: envoyproxy#2077 * Fixes: envoyproxy#1916 Signed-off-by: Arko Dasgupta <[email protected]>
arkodg
added a commit
to arkodg/gateway
that referenced
this issue
Nov 1, 2023
* Fixes: envoyproxy#2077 * Fixes: envoyproxy#1916 Signed-off-by: Arko Dasgupta <[email protected]>
Xunzhuo
pushed a commit
that referenced
this issue
Nov 1, 2023
* Fix attachedRoutes computation * Fixes: #2077 * Fixes: #1916 Signed-off-by: Arko Dasgupta <[email protected]> * make testdata Signed-off-by: Arko Dasgupta <[email protected]> --------- Signed-off-by: Arko Dasgupta <[email protected]>
arkodg
added a commit
to arkodg/gateway
that referenced
this issue
Nov 1, 2023
* Fix attachedRoutes computation * Fixes: envoyproxy#2077 * Fixes: envoyproxy#1916 Signed-off-by: Arko Dasgupta <[email protected]> * make testdata Signed-off-by: Arko Dasgupta <[email protected]> --------- Signed-off-by: Arko Dasgupta <[email protected]> (cherry picked from commit 93a12e7) Signed-off-by: Arko Dasgupta <[email protected]>
arkodg
added a commit
that referenced
this issue
Nov 1, 2023
* Fix TestE2E/RateLimitBasedJwtClaims test (#2097) * Revert "Skip RateLimitBasedJwtClaimsTest test (#2096)" This reverts commit ef7a2a4. fix e2e test Signed-off-by: huabing zhao <[email protected]> * fix XValidation Signed-off-by: huabing zhao <[email protected]> --------- Signed-off-by: huabing zhao <[email protected]> (cherry picked from commit f301527) Signed-off-by: Arko Dasgupta <[email protected]> * add a newer PR reference in rc release docs (#2101) Signed-off-by: Arko Dasgupta <[email protected]> (cherry picked from commit fae8cd8) Signed-off-by: Arko Dasgupta <[email protected]> * fix: panic when using nil xdsRouteAction (#2104) Signed-off-by: bitliu <[email protected]> (cherry picked from commit 77445de) Signed-off-by: Arko Dasgupta <[email protected]> * remove cors, jwt, rl assignment from route translator (#2105) All the translations and assignments now happen in the policy translators Signed-off-by: Arko Dasgupta <[email protected]> (cherry picked from commit d65ab59) Signed-off-by: Arko Dasgupta <[email protected]> * fix: use lowercases of eg admin config fields (#2107) (cherry picked from commit ef5e450) Signed-off-by: Arko Dasgupta <[email protected]> * fix: testGatewayClassWithParamRef e2e test flaky (#2114) (cherry picked from commit 4047268) Signed-off-by: Arko Dasgupta <[email protected]> * build(deps): bump sigs.k8s.io/yaml from 1.3.0 to 1.4.0 (#2118) Bumps [sigs.k8s.io/yaml](https://github.com/kubernetes-sigs/yaml) from 1.3.0 to 1.4.0. - [Release notes](https://github.com/kubernetes-sigs/yaml/releases) - [Changelog](https://github.com/kubernetes-sigs/yaml/blob/master/RELEASE.md) - [Commits](kubernetes-sigs/yaml@v1.3.0...v1.4.0) --- updated-dependencies: - dependency-name: sigs.k8s.io/yaml 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> (cherry picked from commit bf93a4e) Signed-off-by: Arko Dasgupta <[email protected]> * build(deps): bump github.com/bufbuild/buf from 1.27.1 to 1.27.2 in /tools/src/buf (#2121) build(deps): bump github.com/bufbuild/buf in /tools/src/buf Bumps [github.com/bufbuild/buf](https://github.com/bufbuild/buf) from 1.27.1 to 1.27.2. - [Release notes](https://github.com/bufbuild/buf/releases) - [Changelog](https://github.com/bufbuild/buf/blob/main/CHANGELOG.md) - [Commits](bufbuild/buf@v1.27.1...v1.27.2) --- updated-dependencies: - dependency-name: github.com/bufbuild/buf 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> (cherry picked from commit 58146ff) Signed-off-by: Arko Dasgupta <[email protected]> * build(deps): bump github.com/go-logr/logr from 1.2.4 to 1.3.0 (#2117) Bumps [github.com/go-logr/logr](https://github.com/go-logr/logr) from 1.2.4 to 1.3.0. - [Release notes](https://github.com/go-logr/logr/releases) - [Changelog](https://github.com/go-logr/logr/blob/master/CHANGELOG.md) - [Commits](go-logr/logr@v1.2.4...v1.3.0) --- updated-dependencies: - dependency-name: github.com/go-logr/logr 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> (cherry picked from commit d1cbf5e) Signed-off-by: Arko Dasgupta <[email protected]> * build(deps): bump envoyproxy/toolshed from actions-v0.0.25 to 0.1.2 (#2116) Bumps [envoyproxy/toolshed](https://github.com/envoyproxy/toolshed) from actions-v0.0.25 to 0.1.2. This release includes the previously tagged commit. - [Release notes](https://github.com/envoyproxy/toolshed/releases) - [Commits](envoyproxy/toolshed@actions-v0.0.25...actions-v0.1.2) --- updated-dependencies: - dependency-name: envoyproxy/toolshed dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> (cherry picked from commit 7b6344d) Signed-off-by: Arko Dasgupta <[email protected]> * build(deps): bump actions/setup-node from 3 to 4 (#2115) Bumps [actions/setup-node](https://github.com/actions/setup-node) from 3 to 4. - [Release notes](https://github.com/actions/setup-node/releases) - [Commits](actions/setup-node@v3...v4) --- updated-dependencies: - dependency-name: actions/setup-node dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> (cherry picked from commit 5523099) Signed-off-by: Arko Dasgupta <[email protected]> * chore: add more EnvoyProxy cases (#2120) Signed-off-by: zirain <[email protected]> (cherry picked from commit 268a6ef) Signed-off-by: Arko Dasgupta <[email protected]> * build(deps): bump github.com/golangci/golangci-lint from 1.55.0 to 1.55.1 in /tools/src/golangci-lint (#2119) build(deps): bump github.com/golangci/golangci-lint Bumps [github.com/golangci/golangci-lint](https://github.com/golangci/golangci-lint) from 1.55.0 to 1.55.1. - [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.55.0...v1.55.1) --- updated-dependencies: - dependency-name: github.com/golangci/golangci-lint 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> (cherry picked from commit be86295) Signed-off-by: Arko Dasgupta <[email protected]> * feat: CEL Validation in BackendTrafficPolicy (#2110) * feat: CEL Validation in BackendTrafficPolicy Signed-off-by: slayer321 <[email protected]> * leastRequest with consistentHash nil test Signed-off-by: slayer321 <[email protected]> --------- Signed-off-by: slayer321 <[email protected]> (cherry picked from commit b45ae24) Signed-off-by: Arko Dasgupta <[email protected]> * e2e: eg controlplane metrics (#2106) * e2e: eg controlplane metrics Signed-off-by: zirain <[email protected]> * update Signed-off-by: zirain <[email protected]> * update Signed-off-by: zirain <[email protected]> * update name Signed-off-by: zirain <[email protected]> * rename Signed-off-by: zirain <[email protected]> * wait more Signed-off-by: zirain <[email protected]> * add comment Signed-off-by: zirain <[email protected]> * update Signed-off-by: zirain <[email protected]> --------- Signed-off-by: zirain <[email protected]> (cherry picked from commit 046a593) Signed-off-by: Arko Dasgupta <[email protected]> * chore: fix http2_protocol_options warning message (#2048) chore:fix http2_protocol_options warning message Signed-off-by: zhaonan <[email protected]> (cherry picked from commit 179d265) Signed-off-by: Arko Dasgupta <[email protected]> * fix Failed to update SecurityPolicy status (#2128) fix #2127 Signed-off-by: Huabing Zhao <[email protected]> (cherry picked from commit 57e1aec) Signed-off-by: Arko Dasgupta <[email protected]> * fix: add missing status equal for SecurityPolicy (#2134) (cherry picked from commit 71c09f2) Signed-off-by: Arko Dasgupta <[email protected]> * fix jwt doc (#2135) Signed-off-by: huabing zhao <[email protected]> (cherry picked from commit df1e209) Signed-off-by: Arko Dasgupta <[email protected]> * Bump Gateway API to v1.0.0 (#2142) (cherry picked from commit 3dd0ee7) Signed-off-by: Arko Dasgupta <[email protected]> * Enable HTTPRouteRewritePath test (#2112) * Enable HTTPRouteRewritePath test Fixes: #2004 Signed-off-by: Arko Dasgupta <[email protected]> * fix prefix match Signed-off-by: Arko Dasgupta <[email protected]> * make testdata Signed-off-by: Arko Dasgupta <[email protected]> * fix path match Signed-off-by: Arko Dasgupta <[email protected]> * rm trailing / Signed-off-by: Arko Dasgupta <[email protected]> * sort on path match type Signed-off-by: Arko Dasgupta <[email protected]> * make testdata Signed-off-by: Arko Dasgupta <[email protected]> * temp var Signed-off-by: Arko Dasgupta <[email protected]> * fix url rewrite Signed-off-by: Arko Dasgupta <[email protected]> --------- Signed-off-by: Arko Dasgupta <[email protected]> (cherry picked from commit 2b3bc9f) Signed-off-by: Arko Dasgupta <[email protected]> * helm: remove kube-rbac-proxy and update metrics service (#2108) (cherry picked from commit 0c5b054) Signed-off-by: Arko Dasgupta <[email protected]> * doc: user doc for CORS (#2137) * cors docs Signed-off-by: huabing zhao <[email protected]> * address comments Signed-off-by: huabing zhao <[email protected]> --------- Signed-off-by: huabing zhao <[email protected]> (cherry picked from commit d0dc987) Signed-off-by: Arko Dasgupta <[email protected]> * fix(ci): bump go version to 1.21 (#2144) * fix(ci): bump go version to 1.21 Signed-off-by: bitliu <[email protected]> * update Signed-off-by: bitliu <[email protected]> --------- Signed-off-by: bitliu <[email protected]> (cherry picked from commit 6e81fb3) Signed-off-by: Arko Dasgupta <[email protected]> * conformance: Enable HTTPRouteBackendProtocolH2C conformance test (#2136) * nit Signed-off-by: zirain <[email protected]> * build Cluster depends on route type and service appProtocol Signed-off-by: zirain <[email protected]> * enable HTTPRouteBackendProtocolH2C conformance test Signed-off-by: zirain <[email protected]> * lint Signed-off-by: zirain <[email protected]> * fix Signed-off-by: zirain <[email protected]> * address comment Signed-off-by: zirain <[email protected]> * move to package internal/ir Signed-off-by: zirain <[email protected]> --------- Signed-off-by: zirain <[email protected]> (cherry picked from commit 6d532c1) Signed-off-by: Arko Dasgupta <[email protected]> * Fix attachedRoutes computation (#2085) * Fix attachedRoutes computation * Fixes: #2077 * Fixes: #1916 Signed-off-by: Arko Dasgupta <[email protected]> * make testdata Signed-off-by: Arko Dasgupta <[email protected]> --------- Signed-off-by: Arko Dasgupta <[email protected]> (cherry picked from commit 93a12e7) Signed-off-by: Arko Dasgupta <[email protected]> * fix comments in loadbalancer api (#2145) Signed-off-by: zhaonan <[email protected]> (cherry picked from commit 77bcb3c) Signed-off-by: Arko Dasgupta <[email protected]> * fix(docs): incorrect quickstart links (#2146) Signed-off-by: bitliu <[email protected]> (cherry picked from commit defed57) Signed-off-by: Arko Dasgupta <[email protected]> * fix: null pointer when CORS maxAge is not specified (#2133) fix nullpointer Signed-off-by: huabing zhao <[email protected]> Signed-off-by: Huabing Zhao <[email protected]> (cherry picked from commit 35c4fea) Signed-off-by: Arko Dasgupta <[email protected]> --------- Signed-off-by: huabing zhao <[email protected]> Signed-off-by: Arko Dasgupta <[email protected]> Signed-off-by: bitliu <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: zirain <[email protected]> Signed-off-by: slayer321 <[email protected]> Signed-off-by: zhaonan <[email protected]> Signed-off-by: Huabing Zhao <[email protected]> Co-authored-by: Huabing Zhao <[email protected]> Co-authored-by: Xunzhuo <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: zirain <[email protected]> Co-authored-by: Sachin Maurya <[email protected]> Co-authored-by: tmsnan <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area/conformance
Gateway API Conformance Related Issues
area/translator
Issues related to Gateway's translation service, e.g. translating Gateway APIs into the IR.
Description:
With the PR given below, change in Listener or Route status does not impact successful attachment. At the moment we only increment this count when route and listener is accepted.
[optional Relevant Links:]
kubernetes-sigs/gateway-api#2396
The text was updated successfully, but these errors were encountered: