GEP 91: Update Goals and Prior Art #3838
Conversation
Relates to kubernetes-sigs#3760 (comment) Signed-off-by: Arko Dasgupta <[email protected]>
k8s-ci-robot
added
kind/gep
k8s-ci-robot
added
the
size/M
Signed-off-by: Arko Dasgupta <[email protected]>
|
|
||
| * Define an API field to specify the CA Certificate within the Gateway Listener configuration that can be used as a trust anchor to validate the certificates presented by the client. This use case has been highlighted in the [TLS Configuration GEP][] under segment 1 and in the [Gateway API TLS Use Cases][] document under point 7. | ||
| * Define an API field to specify the CA Certificate within the Gateway configuration that can be used as a trust anchor to validate the certificates presented by the client. | ||
| This use case has been highlighted in the [TLS Configuration GEP][] under segment 1 and in the [Gateway API TLS Use Cases][] document under point 7. |
There was a problem hiding this comment.
We should fix these links while we're here.
There was a problem hiding this comment.
all these links work for me
There was a problem hiding this comment.
There was a problem hiding this comment.
I hadn't seen this link syntax before, that's probably where the confusion comes from. (the links are in the Reference section at the end.)
|
Thanks for updating this GEP, LGTM :) |
|
ptal @robscott @youngnick |
|
LGTM once the table is moved, with the caveat that I really want to see the API changes move the config out of the Listener level. |
There was a problem hiding this comment.
Thanks @arkodg! LGTM once implementation details/prior art are moved to bottom as suggested in https://github.com/kubernetes-sigs/gateway-api/pull/3838/files#r2132259700
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: arkodg, robscott The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
k8s-ci-robot
added
the
approved
|
/release-note-none |
k8s-ci-robot
added
release-note-none
Signed-off-by: Arko Dasgupta <[email protected]>
Signed-off-by: Arko Dasgupta <[email protected]>
|
Thanks @arkodg! /lgtm |
k8s-ci-robot
removed
the
do-not-merge/hold
k8s-ci-robot
added
the
lgtm
Relates to #3760 (comment)
Will update the API in a follow up PR
/kind gep