Skip to content

Commit 87a033c

Browse files
committed
Implement policy affected status
1 parent 200fa1f commit 87a033c

File tree

2 files changed

+21
-1
lines changed

2 files changed

+21
-1
lines changed

internal/controller/state/conditions/conditions.go

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,8 +127,12 @@ const (
127127
// ObservabilityPolicy is applied to a HTTPRoute, or GRPCRoute.
128128
ObservabilityPolicyAffected v1.PolicyConditionType = "ObservabilityPolicyAffected"
129129

130+
// ProxySettingsPolicyAffected is used with the "PolicyAffected" condition when a
131+
// ProxySettingsPolicy is applied to a Gateway, HTTPRoute, or GRPCRoute.
132+
ProxySettingsPolicyAffected v1.PolicyConditionType = "ProxySettingsPolicyAffected"
133+
130134
// PolicyAffectedReason is used with the "PolicyAffected" condition when a
131-
// ObservabilityPolicy or ClientSettingsPolicy is applied to Gateways or Routes.
135+
// ObservabilityPolicy, ClientSettingsPolicy, or ProxySettingsPolicy is applied to Gateways or Routes.
132136
PolicyAffectedReason v1.PolicyConditionReason = "PolicyAffected"
133137

134138
// GatewayResolvedRefs condition indicates whether the controller was able to resolve the
@@ -1145,6 +1149,17 @@ func NewClientSettingsPolicyAffected() Condition {
11451149
}
11461150
}
11471151

1152+
// NewProxySettingsPolicyAffected returns a Condition that indicates that a ProxySettingsPolicy
1153+
// is applied to the resource.
1154+
func NewProxySettingsPolicyAffected() Condition {
1155+
return Condition{
1156+
Type: string(ProxySettingsPolicyAffected),
1157+
Status: metav1.ConditionTrue,
1158+
Reason: string(PolicyAffectedReason),
1159+
Message: "The ProxySettingsPolicy is applied to the resource",
1160+
}
1161+
}
1162+
11481163
// NewBackendTLSPolicyResolvedRefs returns a Condition that indicates that all CACertificateRefs
11491164
// in the BackendTLSPolicy are resolved.
11501165
func NewBackendTLSPolicyResolvedRefs() Condition {

internal/controller/state/graph/policies.go

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -637,5 +637,10 @@ func addStatusToTargetRefs(policyKind string, conditionsList *[]conditions.Condi
637637
return
638638
}
639639
*conditionsList = append(*conditionsList, conditions.NewClientSettingsPolicyAffected())
640+
case kinds.ProxySettingsPolicy:
641+
if conditions.HasMatchingCondition(*conditionsList, conditions.NewProxySettingsPolicyAffected()) {
642+
return
643+
}
644+
*conditionsList = append(*conditionsList, conditions.NewProxySettingsPolicyAffected())
640645
}
641646
}

0 commit comments

Comments
 (0)