Quick fix for container resources.

Author: lamai93

pkg/deployment/reconcile/plan_builder.go

@@ -351,9 +351,10 @@ func podNeedsRotation(log zerolog.Logger, p v1.Pod, apiObject metav1.Object, spe
 	}
 
 	// Check resource requirements
-	if resourcesRequireRotation(spec.GetServerGroupSpec(group).Resources, k8sutil.GetArangoDBContainerFromPod(&p).Resources) {
+	if resourcesRequireRotation(
+		k8sutil.FilterStorageResourceRequirement(spec.GetServerGroupSpec(group).Resources),
+		k8sutil.GetArangoDBContainerFromPod(&p).Resources) {
 		return true, "Resource Requirements changed"
-
 	}
 
 	return false, ""

pkg/util/k8sutil/pods.go

@@ -263,8 +263,8 @@ func arangodInitContainer(name, id, engine, alpineImage string, requireUUID bool
 	return c
 }
 
-// filterStorageResourceRequirement filters resource requirements for Storage.
-func filterStorageResourceRequirement(resources v1.ResourceRequirements) v1.ResourceRequirements {
+// FilterStorageResourceRequirement filters resource requirements for Storage.
+func FilterStorageResourceRequirement(resources v1.ResourceRequirements) v1.ResourceRequirements {
 
 	filterStorage := func(list v1.ResourceList) v1.ResourceList {
 		newlist := make(v1.ResourceList)
@@ -299,7 +299,7 @@ func arangodContainer(image string, imagePullPolicy v1.PullPolicy, args []string
 				Protocol:      v1.ProtocolTCP,
 			},
 		},
-		Resources:    filterStorageResourceRequirement(resources), // Storage is handled via pvcs
+		Resources:    FilterStorageResourceRequirement(resources), // Storage is handled via pvcs
 		VolumeMounts: arangodVolumeMounts(),
 	}
 	for k, v := range env {
@@ -335,7 +335,7 @@ func arangosyncContainer(image string, imagePullPolicy v1.PullPolicy, args []str
 				Protocol:      v1.ProtocolTCP,
 			},
 		},
-		Resources: filterStorageResourceRequirement(resources), // Storage is handled via pvcs
+		Resources: FilterStorageResourceRequirement(resources), // Storage is handled via pvcs
 	}
 	for k, v := range env {
 		c.Env = append(c.Env, v.CreateEnvVar(k))