search

Kuadrant / architecture

Architecture Documents
0 stars 10 forks source link

RFC: TLSPolicy #22

Closed jasonmadigan closed 9 months ago

jasonmadigan commented 1 year ago

Moving from its previous home in https://github.com/Kuadrant/multicluster-gateway-controller/pull/345, as architecture is where we're coalescing RFCs.

An initial implementation has been created and landed by @mikenairn here: https://github.com/Kuadrant/multicluster-gateway-controller/pull/402/files

Setting this as draft as @mikenairn will want another pass at it, given he recently implemented it while I've been away.

jasonmadigan commented 1 year ago

Parking for a moment, will come back to this and layer in content from docs/tls-policy.md

pmccarthy commented 10 months ago

@jasonmadigan do we think this can be merged given that TLSPolicy is a supported policy in Kuadrant now?

jasonmadigan commented 10 months ago

@pmccarthy think so. it's one of those ones that fell into that "we've done it, here is the RFC now" traps. And I think we agreed we'd just merge those?

pmccarthy commented 10 months ago

@pmccarthy think so. it's one of those ones that fell into that "we've done it, here is the RFC now" traps. And I think we agreed we'd just merge those?

Thoughts on this @maleck13 @alexsnaps ?

maleck13 commented 10 months ago

No this one really needs an update tbh or we close it. The exampes etc are all wrong see: https://github.com/Kuadrant/multicluster-gateway-controller/blob/main/docs/reference/tlspolicy.md

jasonmadigan commented 9 months ago

@jasonmadigan wondering what we want to do with this?

Think we close

maleck13 commented 9 months ago

out of date