Skip to content

Commit d5b79f7

Browse files
[CI] Fix Terraform Reconciliation Issue around Units
This patch updates the operationa metrics Cron job to use Mi for the memory requests rather than Gi as k8s will automatically convert them to Mi for decimal number of Gi which terraform then thinks differs from what it tried to apply. This leads to terraform trying to reapply the setting everytime one runs terraform apply despite the settings already being equivalent.
1 parent eb81671 commit d5b79f7

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

premerge/operational_metrics_cronjob.yaml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,10 @@ spec:
3838
resources:
3939
requests:
4040
cpu: "250m"
41-
memory: "1.75Gi"
41+
# We explicitly use Mi here instead of a decimal number of
42+
# Gi because k8s will automatically convert to Mi which
43+
# terraform then thinks differs from what it intended to apply.
44+
memory: "1792Mi"
4245
limits:
4346
cpu: "2"
4447
memory: "2Gi"

0 commit comments

Comments
 (0)