Merge pull request #7488 from Lyndon-Li/issue-fix-7391

Issue 7391:remove the default constraint for node-agent pods
vmware-tanzu · Mar 4, 2024 · f274fe7 · f274fe7
2 parents 97d276c + d558e49
commit f274fe7
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 9 deletions.
diff --git a/changelogs/unreleased/7488-Lyndon-Li b/changelogs/unreleased/7488-Lyndon-Li
@@ -0,0 +1 @@
+Fix issue #7391, remove the default constraint for node-agent pods
diff --git a/pkg/install/resources.go b/pkg/install/resources.go
@@ -39,15 +39,20 @@ const (
 )
 
 var (
-	DefaultVeleroPodCPURequest    = "500m"
-	DefaultVeleroPodMemRequest    = "128Mi"
-	DefaultVeleroPodCPULimit      = "1000m"
-	DefaultVeleroPodMemLimit      = "512Mi"
-	DefaultNodeAgentPodCPURequest = "500m"
-	DefaultNodeAgentPodMemRequest = "512Mi"
-	DefaultNodeAgentPodCPULimit   = "1000m"
-	DefaultNodeAgentPodMemLimit   = "1Gi"
-	DefaultVeleroNamespace        = "velero"
+	// default values for Velero server pod resource request/limit
+	DefaultVeleroPodCPURequest = "500m"
+	DefaultVeleroPodMemRequest = "128Mi"
+	DefaultVeleroPodCPULimit   = "1000m"
+	DefaultVeleroPodMemLimit   = "512Mi"
+
+	// default values for node-agent pod resource request/limit,
+	// "0" means no request/limit is set, so as to make the QoS as BestEffort
+	DefaultNodeAgentPodCPURequest = "0"
+	DefaultNodeAgentPodMemRequest = "0"
+	DefaultNodeAgentPodCPULimit   = "0"
+	DefaultNodeAgentPodMemLimit   = "0"
+
+	DefaultVeleroNamespace = "velero"
 )
 
 func Labels() map[string]string {