Skip to content

Commit af0c734

Browse files
committed
fix: suggestions from review
1 parent 49e5929 commit af0c734

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

charts/cluster-api-runtime-extensions-nutanix/addons/cluster-autoscaler/values-template.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
fullnameOverride: "cluster-autoscaler-{{ index .Cluster.Annotations "caren.nutanix.com/cluster-uuid" }}
2+
fullnameOverride: "cluster-autoscaler-{{ index .Cluster.Annotations "caren.nutanix.com/cluster-uuid" }}"
33

44
cloudProvider: clusterapi
55

hack/addons/kustomize/cluster-autoscaler/manifests/values-template.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
fullnameOverride: "cluster-autoscaler-{{ index .Cluster.Annotations "caren.nutanix.com/cluster-uuid" }}
2+
fullnameOverride: "cluster-autoscaler-{{ index .Cluster.Annotations "caren.nutanix.com/cluster-uuid" }}"
33

44
cloudProvider: clusterapi
55

hack/addons/update-cluster-autoscaler.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ kustomize build --enable-helm "${ASSETS_DIR}" >"${ASSETS_DIR}/${FILE_NAME}"
2727

2828
# this replaces the custom templating used for kustomize to generate the configmap to using the same templating as we do
2929
# with helm chart proxy
30-
# transforming tmpl-clustername-tmpl to -- {{ .Cluster.Name }}
30+
# transforming tmpl-clustername-tmpl to {{ .Cluster.Name }}
3131
sed -i -e "s/\([a-z-]*\)tmpl-clustername\([^-]*\)-tmpl\([a-z-]*\)/\1{{ \`{{ .Cluster.Name\2 }}\` }}\3/g" \
3232
-e "s/\([a-z-]*\)tmpl-clusteruuid-tmpl\([a-z-]*\)/\1{{ \`{{ index .Cluster.Annotations \"caren.nutanix.com\/cluster-uuid\" }}\` }}\2/g" \
3333
"${ASSETS_DIR}/${FILE_NAME}"

0 commit comments

Comments
 (0)