Skip to content

Commit e854f36

Browse files
authored
Merge pull request #2741 from sunjayBhatia/backendtlspolicy-wellknown-system-impl-specific
Move BackendTLSPolicy WellKnownCACerts field to implementation-specific
2 parents 8e9525b + d7ccab6 commit e854f36

File tree

2 files changed

+14
-6
lines changed

2 files changed

+14
-6
lines changed

apis/v1alpha2/backendtlspolicy_types.go

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,8 @@ type BackendTLSPolicyConfig struct {
8585
// If CACertRefs is empty or unspecified, then WellKnownCACerts must be
8686
// specified. Only one of CACertRefs or WellKnownCACerts may be specified,
8787
// not both. If CACertRefs is empty or unspecified, the configuration for
88-
// WellKnownCACerts MUST be honored instead.
88+
// WellKnownCACerts MUST be honored instead if supported by the
89+
// implementation.
8990
//
9091
// References to a resource in a different namespace are invalid for the
9192
// moment, although we will revisit this in the future.
@@ -109,9 +110,12 @@ type BackendTLSPolicyConfig struct {
109110
//
110111
// If WellKnownCACerts is unspecified or empty (""), then CACertRefs must be
111112
// specified with at least one entry for a valid configuration. Only one of
112-
// CACertRefs or WellKnownCACerts may be specified, not both.
113+
// CACertRefs or WellKnownCACerts may be specified, not both. If an
114+
// implementation does not support the WellKnownCACerts field or the value
115+
// supplied is not supported, the Status Conditions on the Policy MUST be
116+
// updated to include an Accepted: False Condition with Reason: Invalid.
113117
//
114-
// Support: Core for "System"
118+
// Support: Implementation-specific
115119
//
116120
// +optional
117121
WellKnownCACerts *WellKnownCACertType `json:"wellKnownCACerts,omitempty"`

config/crd/experimental/gateway.networking.k8s.io_backendtlspolicies.yaml

Lines changed: 7 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)