fix(api): incomplete CEL for namespace immutability check in GrafanaManifest (#2584)

* fix(api): CEL validation for namespace immutability in GrafanaManifest

* chore: regenerate manifests

Author: weisdd

api/v1beta1/typeoverrides.go

@@ -20,7 +20,7 @@ import (
 
 // RequiredObjectMeta contains only a [subset of the fields included in k8s.io/apimachinery/pkg/apis/meta/v1.ObjectMeta](https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.27/#objectmeta-v1-meta).
 // It requires `name` to be set
-// +kubebuilder:validation:XValidation:rule="(!(has(oldSelf.__namespace__) && !has(self.__namespace__)))", message="namespace is immutable"
+// +kubebuilder:validation:XValidation:rule="((!has(oldSelf.__namespace__) && !has(self.__namespace__)) || (has(oldSelf.__namespace__) && has(self.__namespace__)))", message="namespace is immutable"
 type RequiredObjectMeta struct {
 	// +kubebuilder:validation:XValidation:rule="self == oldSelf",message="Value is immutable"
 	Name string `json:"name" protobuf:"bytes,1,opt,name=name"`

config/crd/bases/grafana.integreatly.org_grafanamanifests.yaml

@@ -241,7 +241,8 @@ spec:
                     type: object
                     x-kubernetes-validations:
                     - message: namespace is immutable
-                      rule: (!(has(oldSelf.__namespace__) && !has(self.__namespace__)))
+                      rule: ((!has(oldSelf.__namespace__) && !has(self.__namespace__))
+                        || (has(oldSelf.__namespace__) && has(self.__namespace__)))
                   spec:
                     x-kubernetes-preserve-unknown-fields: true
                 required:

deploy/helm/grafana-operator/files/crds/grafana.integreatly.org_grafanamanifests.yaml

@@ -241,7 +241,8 @@ spec:
                     type: object
                     x-kubernetes-validations:
                     - message: namespace is immutable
-                      rule: (!(has(oldSelf.__namespace__) && !has(self.__namespace__)))
+                      rule: ((!has(oldSelf.__namespace__) && !has(self.__namespace__))
+                        || (has(oldSelf.__namespace__) && has(self.__namespace__)))
                   spec:
                     x-kubernetes-preserve-unknown-fields: true
                 required:

deploy/kustomize/base/crds.yaml

@@ -2644,7 +2644,8 @@ spec:
                     type: object
                     x-kubernetes-validations:
                     - message: namespace is immutable
-                      rule: (!(has(oldSelf.__namespace__) && !has(self.__namespace__)))
+                      rule: ((!has(oldSelf.__namespace__) && !has(self.__namespace__))
+                        || (has(oldSelf.__namespace__) && has(self.__namespace__)))
                   spec:
                     x-kubernetes-preserve-unknown-fields: true
                 required:

docs/docs/api.md

@@ -4992,7 +4992,7 @@ merge patch.<br/>
           RequiredObjectMeta contains only a [subset of the fields included in k8s.io/apimachinery/pkg/apis/meta/v1.ObjectMeta](https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.27/#objectmeta-v1-meta).
 It requires `name` to be set<br/>
           <br/>
-            <i>Validations</i>:<li>(!(has(oldSelf.__namespace__) && !has(self.__namespace__))): namespace is immutable</li>
+            <i>Validations</i>:<li>((!has(oldSelf.__namespace__) && !has(self.__namespace__)) || (has(oldSelf.__namespace__) && has(self.__namespace__))): namespace is immutable</li>
         </td>
         <td>true</td>
       </tr><tr>