Merge pull request #161 from Madhu-1/add-resize-res

feat: add resource limit and request for csiResizer
openshift · Apr 20, 2022 · b346b4f · b346b4f
2 parents 4bb8d1d + 56a3568
commit b346b4f
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/controllers/defaults.go b/controllers/defaults.go
@@ -76,6 +76,11 @@ var (
 	TopolvmCsiProvisionerCPURequest = "100m"
 	TopolvmCsiProvisionerCPULimit   = "100m"
 
+	TopolvmCsiResizerMemRequest = "100Mi"
+	TopolvmCsiResizerMemLimit   = "100Mi"
+	TopolvmCsiResizerCPURequest = "100m"
+	TopolvmCsiResizerCPULimit   = "100m"
+
 	// CSI Provisioner requires below environment values to make use of CSIStorageCapacity
 	PodNameEnv   = "POD_NAME"
 	NameSpaceEnv = "NAMESPACE"

diff --git a/controllers/topolvm_controller.go b/controllers/topolvm_controller.go
@@ -313,10 +313,22 @@ func getCsiResizerContainer() *corev1.Container {
 		{Name: "socket-dir", MountPath: filepath.Dir(DefaultCSISocket)},
 	}
 
+	resourceRequirements := corev1.ResourceRequirements{
+		Limits: corev1.ResourceList{
+			corev1.ResourceCPU:    resource.MustParse(TopolvmCsiResizerCPULimit),
+			corev1.ResourceMemory: resource.MustParse(TopolvmCsiResizerMemLimit),
+		},
+		Requests: corev1.ResourceList{
+			corev1.ResourceCPU:    resource.MustParse(TopolvmCsiResizerCPURequest),
+			corev1.ResourceMemory: resource.MustParse(TopolvmCsiResizerMemRequest),
+		},
+	}
+
 	csiResizer := &corev1.Container{
 		Name:         CsiResizerContainerName,
 		Image:        CsiResizerImage,
 		Args:         args,
+		Resources:    resourceRequirements,
 		VolumeMounts: volumeMounts,
 	}
 	return csiResizer