Also set resource k8s limits/requests on the apply job (#1553)

* [HELM] - Job Label Chart Fix (#1551) The current helm chart has a typo in the 'controller.jobsLabels', this should have been 'controller.jobLabels', which is what the actual template references * Also set resource k8s limits/requests on the apply job --------- Co-authored-by: Rohith Jayawardene <[email protected]>
appvia · Nov 15, 2024 · a5df2ac · a5df2ac
1 parent 0a9a2e5
commit a5df2ac
Showing 1 changed file with 14 additions and 10 deletions.
diff --git a/pkg/controller/configuration/ensure.go b/pkg/controller/configuration/ensure.go
@@ -1254,16 +1254,20 @@ func (c *Controller) ensureTerraformApply(configuration *terraformv1alpha1.Confi
 					terraformv1alpha1.JobTemplateHashLabel: state.jobTemplateHash,
 				},
 			),
-			BackoffLimit:       c.BackoffLimit,
-			EnableInfraCosts:   c.EnableInfracosts,
-			ExecutorImage:      c.ExecutorImage,
-			ExecutorSecrets:    c.ExecutorSecrets,
-			InfracostsImage:    c.InfracostsImage,
-			InfracostsSecret:   c.InfracostsSecretName,
-			Namespace:          c.ControllerNamespace,
-			SaveTerraformState: saveState,
-			Template:           state.jobTemplate,
-			TerraformImage:     GetTerraformImage(configuration, c.TerraformImage),
+			BackoffLimit:                 c.BackoffLimit,
+			DefaultExecutorCPULimit:      c.DefaultExecutorCPULimit,
+			DefaultExecutorCPURequest:    c.DefaultExecutorCPURequest,
+			DefaultExecutorMemoryLimit:   c.DefaultExecutorMemoryLimit,
+			DefaultExecutorMemoryRequest: c.DefaultExecutorMemoryRequest,
+			EnableInfraCosts:             c.EnableInfracosts,
+			ExecutorImage:                c.ExecutorImage,
+			ExecutorSecrets:              c.ExecutorSecrets,
+			InfracostsImage:              c.InfracostsImage,
+			InfracostsSecret:             c.InfracostsSecretName,
+			Namespace:                    c.ControllerNamespace,
+			SaveTerraformState:           saveState,
+			Template:                     state.jobTemplate,
+			TerraformImage:               GetTerraformImage(configuration, c.TerraformImage),
 		})
 		if err != nil {
 			cond.Failed(err, "Failed to create the terraform apply job")