feat: add api validation for auth config

Signed-off-by: Bence Csati <[email protected]>

Author: csatib02

api/telemetry/v1alpha1/output_types.go

@@ -69,10 +69,12 @@ type OutputSpec struct {
 	Batch          *Batch         `json:"batch,omitempty"`
 }
 
+// +kubebuilder:validation:XValidation:rule="(has(self.basicAuth) && has(self.bearerAuth)) == false",message="Only one authentication method can be specified: either basicAuth or bearerAuth, not both"
+
 // Output Authentication configuration.
 type OutputAuth struct {
-	BasicAuth  *BasicAuthConfig  `json:"basicauth,omitempty"`
-	BearerAuth *BearerAuthConfig `json:"bearerauth,omitempty"`
+	BasicAuth  *BasicAuthConfig  `json:"basicAuth,omitempty"`
+	BearerAuth *BearerAuthConfig `json:"bearerAuth,omitempty"`
 }
 
 type BasicAuthConfig struct {

charts/telemetry-controller/crds/telemetry.kube-logging.dev_outputs.yaml

@@ -51,7 +51,7 @@ spec:
               authentication:
                 description: Output Authentication configuration.
                 properties:
-                  basicauth:
+                  basicAuth:
                     properties:
                       passwordField:
                         type: string
@@ -73,7 +73,7 @@ spec:
                       usernameField:
                         type: string
                     type: object
-                  bearerauth:
+                  bearerAuth:
                     properties:
                       secretRef:
                         description: |-
@@ -94,6 +94,10 @@ spec:
                         type: string
                     type: object
                 type: object
+                x-kubernetes-validations:
+                - message: 'Only one authentication method can be specified: either
+                    basicAuth or bearerAuth, not both'
+                  rule: (has(self.basicAuth) && has(self.bearerAuth)) == false
               batch:
                 description: Batch processor configuration.
                 properties:

config/crd/bases/telemetry.kube-logging.dev_outputs.yaml

@@ -51,7 +51,7 @@ spec:
               authentication:
                 description: Output Authentication configuration.
                 properties:
-                  basicauth:
+                  basicAuth:
                     properties:
                       passwordField:
                         type: string
@@ -73,7 +73,7 @@ spec:
                       usernameField:
                         type: string
                     type: object
-                  bearerauth:
+                  bearerAuth:
                     properties:
                       secretRef:
                         description: |-
@@ -94,6 +94,10 @@ spec:
                         type: string
                     type: object
                 type: object
+                x-kubernetes-validations:
+                - message: 'Only one authentication method can be specified: either
+                    basicAuth or bearerAuth, not both'
+                  rule: (has(self.basicAuth) && has(self.bearerAuth)) == false
               batch:
                 description: Batch processor configuration.
                 properties: