Rob Scott

icon indicating a website link https://robertjscott.ca icon indicating an email [email protected]

icon company @Google icon location Sunnyvale, CA icon location Kubernetes Networking @ Google

Results 477 comments of Rob Scott

Conformance test for multiple TLS certificates

This is one of the bullet points of #1104 (although that doesn't explicitly require multiple certs), adding a link from there as well.

Add support for manipulating response headers in the `HTTPRouteFilter` API

/kind gep

Create utils to make it easier to work with custom types

@hbagdi yep, thanks for adding the examples!

Create utils to make it easier to work with custom types

That's a great set of helpers! I think I've seen similar helpers in at least the Istio code base, @howardjohn is there much overlap here? Agree that it would be...

conformance: test gateway when an unsupported Route kind is specified by the Listener.

How does this differ from the [HTTPRouteDisallowedKind](https://github.com/kubernetes-sigs/gateway-api/blob/main/conformance/tests/httproute-disallowed-kind.go) test? Maybe we just need to expand that one?

GEP: Allow Multiple Certificate Refs per Gateway Listener

Reopening because we need conformance test coverage for this.

apply httproute.yaml err info： error validating data: ValidationError(HTTPRoute.spec.rules[0].filters[0].requestRedirect): unknown field "path" in io.k8s.networking.gateway.v1alpha2.HTTPRoute.spec.rules.filters.requestRedirect; if you choose to ignore these errors, turn validation off with --validate=false

Going to close this one due to lack of response, feel free to reopen if you're still running into this.

discrepancies between Accepted, Ready, and Detached condition types and reasons

We need to make sure this is compatible with Route inclusion status proposal

conformance: refactor listenersMatch and parentsMatch helpers into a single generic function

@FlorentFlament that sounds like it could work, thanks for volunteering! /assign @FlorentFlament

Update HTTPRoute and other status definitions and documentation

I think #1110, #1111, and #1112 by @mikemorris significantly overlap with this issue. Not sure if there's room for consolidation, just wanted to make sure these were all linked together.