Merge pull request #60 from GDATASoftwareAG/mutable_ram

Make machinetemplate.spec.ram mutable

Author: Mattes83

api/v1alpha1/ionoscloudmachine_types.go

@@ -50,7 +50,6 @@ type IONOSCloudMachineSpec struct {
 	// The memory size for the enterprise server in MB, such as 2048.
 	// +kubebuilder:validation:Minimum=256
 	// +kubebuilder:validation:MultipleOf=256
-	// +kubebuilder:validation:XValidation:rule="self == oldSelf",message="Ram is immutable"
 	Ram        *int32          `json:"ram"`
 	BootVolume IONOSVolumeSpec `json:"bootVolume"`
 

config/crd/bases/infrastructure.cluster.x-k8s.io_ionoscloudmachines.yaml

@@ -134,9 +134,6 @@ spec:
                 minimum: 256
                 multipleOf: 256
                 type: integer
-                x-kubernetes-validations:
-                - message: Ram is immutable
-                  rule: self == oldSelf
             required:
             - bootVolume
             - cores

config/crd/bases/infrastructure.cluster.x-k8s.io_ionoscloudmachinetemplates.yaml

@@ -135,9 +135,6 @@ spec:
                         minimum: 256
                         multipleOf: 256
                         type: integer
-                        x-kubernetes-validations:
-                        - message: Ram is immutable
-                          rule: self == oldSelf
                     required:
                     - bootVolume
                     - cores

internal/controller/ionoscloudmachine_controller_test.go

@@ -234,12 +234,6 @@ var _ = Describe("IONOSCloudMachine controller", func() {
 				return 0
 			}, timeout, interval).Should(Equal(1))
 		})
-
-		It("should not be possible to change immutable fields", func() {
-			capicMachine.Spec.Ram = ionoscloud.ToPtr(int32(256))
-			err := k8sClient.Update(ctx, capicMachine)
-			Expect(err).Should(HaveOccurred())
-		})
 	})
 
 	When("IONOSCloudMachine has been created", func() {