Skip to content

Commit 95cde2e

Browse files
dhaiducekopenshift-merge-bot[bot]
authored andcommittedOct 3, 2024
Remove requirement for eventHook in automation
ref: https://issues.redhat.com/browse/ACM-14657 Signed-off-by: Dale Haiducek <19750917+dhaiducek@users.noreply.github.com>
1 parent 706f577 commit 95cde2e

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed
 

‎api/v1beta1/policyautomation_types.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ type PolicyAutomationSpec struct {
6868
// "noncompliant".
6969
//
7070
// +kubebuilder:validation:Enum={noncompliant}
71-
// +kubebuilder:validation:Required
71+
// +kubebuilder:default=noncompliant
7272
EventHook string `json:"eventHook,omitempty"`
7373

7474
// RescanAfter is reserved for future use and should not be set.

‎deploy/crds/policy.open-cluster-management.io_policyautomations.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,7 @@ spec:
9191
minimum: 0
9292
type: integer
9393
eventHook:
94+
default: noncompliant
9495
description: >-
9596
EventHook specifies the compliance state that initiates automation. This must be set to
9697
"noncompliant".
@@ -116,7 +117,6 @@ spec:
116117
type: string
117118
required:
118119
- automationDef
119-
- eventHook
120120
- mode
121121
- policyRef
122122
type: object

0 commit comments

Comments
 (0)
Failed to load comments.