Remove resource limit

This patch removes the resource limit for test operator pods as we are hitting both issue when: - the limit is too low - the limit is too high (scheduler does not have where to schedule the pod) This is a hotfix before we expose the resource limit values through the test-operator CRs and find the correct default values. Related PRs: - openstack-k8s-operators#222 - openstack-k8s-operators#205
lpiwowar · Oct 9, 2024 · dd4c6fe · dd4c6fe
1 parent edd79b3
commit dd4c6fe
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 20 deletions.
diff --git a/pkg/ansibletest/job.go b/pkg/ansibletest/job.go
@@ -64,9 +64,6 @@ func Job(
 							Env:             env.MergeEnvs([]corev1.EnvVar{}, envVars),
 							VolumeMounts:    GetVolumeMounts(mountCerts, instance, externalWorkflowCounter),
 							SecurityContext: &securityContext,
-							Resources: corev1.ResourceRequirements{
-								Limits: util.GetResourceLimits(),
-							},
 						},
 					},
 					Volumes: GetVolumes(

diff --git a/pkg/horizontest/job.go b/pkg/horizontest/job.go
@@ -63,9 +63,6 @@ func Job(
 							Env:             env.MergeEnvs([]corev1.EnvVar{}, envVars),
 							VolumeMounts:    GetVolumeMounts(mountCerts, mountKeys, mountKubeconfig, instance),
 							SecurityContext: &securityContext,
-							Resources: corev1.ResourceRequirements{
-								Limits: util.GetResourceLimits(),
-							},
 						},
 					},
 					Volumes: GetVolumes(

diff --git a/pkg/tempest/job.go b/pkg/tempest/job.go
@@ -79,9 +79,6 @@ func Job(
 									},
 								},
 							},
-							Resources: corev1.ResourceRequirements{
-								Limits: util.GetResourceLimits(),
-							},
 						},
 					},
 					Volumes: GetVolumes(

diff --git a/pkg/tobiko/job.go b/pkg/tobiko/job.go
@@ -68,9 +68,6 @@ func Job(
 							Env:             env.MergeEnvs([]corev1.EnvVar{}, envVars),
 							VolumeMounts:    GetVolumeMounts(mountCerts, mountKeys, mountKubeconfig, instance),
 							SecurityContext: &securityContext,
-							Resources: corev1.ResourceRequirements{
-								Limits: util.GetResourceLimits(),
-							},
 						},
 					},
 					Volumes: GetVolumes(

diff --git a/pkg/util/common.go b/pkg/util/common.go
@@ -2,7 +2,6 @@ package util
 
 import (
 	corev1 "k8s.io/api/core/v1"
-	"k8s.io/apimachinery/pkg/api/resource"
 )
 
 func GetSecurityContext(
@@ -38,10 +37,3 @@ func GetSecurityContext(
 
 	return securityContext
 }
-
-func GetResourceLimits() corev1.ResourceList {
-	return corev1.ResourceList{
-		corev1.ResourceCPU:    resource.MustParse("2000m"),
-		corev1.ResourceMemory: resource.MustParse("8Gi"),
-	}
-}