make fields optional on webhook options (#818)

Signed-off-by: aerosouund <[email protected]>
kyverno · Mar 11, 2025 · 6fd5acf · 6fd5acf
1 parent 86f8d25
commit 6fd5acf
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 16 deletions.
diff --git a/config/crds/policyreporter.kyverno.io_targetconfigs.yaml b/config/crds/policyreporter.kyverno.io_targetconfigs.yaml
@@ -314,10 +314,7 @@ spec:
                   webhook:
                     type: string
                 required:
-                - certificate
                 - channel
-                - headers
-                - skipTLS
                 - webhook
                 type: object
               sources:
@@ -337,9 +334,6 @@ spec:
                   webhook:
                     type: string
                 required:
-                - certificate
-                - headers
-                - skipTLS
                 - webhook
                 type: object
               telegram:
@@ -359,10 +353,7 @@ spec:
                   webhook:
                     type: string
                 required:
-                - certificate
                 - chatId
-                - headers
-                - skipTLS
                 - token
                 - webhook
                 type: object
@@ -379,9 +370,6 @@ spec:
                   webhook:
                     type: string
                 required:
-                - certificate
-                - headers
-                - skipTLS
                 - webhook
                 type: object
             type: object

diff --git a/pkg/crd/api/targetconfig/v1alpha1/configs.go b/pkg/crd/api/targetconfig/v1alpha1/configs.go
@@ -12,10 +12,13 @@ type AWSConfig struct {
 }
 
 type WebhookOptions struct {
-	Webhook     string            `mapstructure:"webhook" json:"webhook"`
-	SkipTLS     bool              `mapstructure:"skipTLS" json:"skipTLS"`
-	Certificate string            `mapstructure:"certificate" json:"certificate"`
-	Headers     map[string]string `mapstructure:"headers" json:"headers"`
+	Webhook string `mapstructure:"webhook" json:"webhook"`
+	// +optional
+	SkipTLS bool `mapstructure:"skipTLS" json:"skipTLS"`
+	// +optional
+	Certificate string `mapstructure:"certificate" json:"certificate"`
+	// +optional
+	Headers map[string]string `mapstructure:"headers" json:"headers"`
 }
 
 type HostOptions struct {