Skip to content

Commit a5df2ac

Browse files
ronaldvb-kgambol99
andcommitted
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 <gambol99@gmail.com>
1 parent 0a9a2e5 commit a5df2ac

1 file changed

Lines changed: 14 additions & 10 deletions

File tree

pkg/controller/configuration/ensure.go

Lines changed: 14 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1254,16 +1254,20 @@ func (c *Controller) ensureTerraformApply(configuration *terraformv1alpha1.Confi
12541254
terraformv1alpha1.JobTemplateHashLabel: state.jobTemplateHash,
12551255
},
12561256
),
1257-
BackoffLimit: c.BackoffLimit,
1258-
EnableInfraCosts: c.EnableInfracosts,
1259-
ExecutorImage: c.ExecutorImage,
1260-
ExecutorSecrets: c.ExecutorSecrets,
1261-
InfracostsImage: c.InfracostsImage,
1262-
InfracostsSecret: c.InfracostsSecretName,
1263-
Namespace: c.ControllerNamespace,
1264-
SaveTerraformState: saveState,
1265-
Template: state.jobTemplate,
1266-
TerraformImage: GetTerraformImage(configuration, c.TerraformImage),
1257+
BackoffLimit: c.BackoffLimit,
1258+
DefaultExecutorCPULimit: c.DefaultExecutorCPULimit,
1259+
DefaultExecutorCPURequest: c.DefaultExecutorCPURequest,
1260+
DefaultExecutorMemoryLimit: c.DefaultExecutorMemoryLimit,
1261+
DefaultExecutorMemoryRequest: c.DefaultExecutorMemoryRequest,
1262+
EnableInfraCosts: c.EnableInfracosts,
1263+
ExecutorImage: c.ExecutorImage,
1264+
ExecutorSecrets: c.ExecutorSecrets,
1265+
InfracostsImage: c.InfracostsImage,
1266+
InfracostsSecret: c.InfracostsSecretName,
1267+
Namespace: c.ControllerNamespace,
1268+
SaveTerraformState: saveState,
1269+
Template: state.jobTemplate,
1270+
TerraformImage: GetTerraformImage(configuration, c.TerraformImage),
12671271
})
12681272
if err != nil {
12691273
cond.Failed(err, "Failed to create the terraform apply job")

0 commit comments

Comments
 (0)