add support for tags (#76)

Signed-off-by: David J. M. Karlsen <[email protected]>
rh-mobb · Apr 30, 2024 · d38a2cb · d38a2cb
1 parent 727c718
commit d38a2cb
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/charts/aro-machinesets/Chart.yaml b/charts/aro-machinesets/Chart.yaml
@@ -3,7 +3,7 @@ name: aro-machinesets
 description: A Helm chart to configure machinesets for ARO
 type: application
 
-version: 0.4.0
+version: 0.5.0
 maintainers:
   - name: paulczar
 home: https://github.com/rh-mobb/helm-charts
diff --git a/charts/aro-machinesets/templates/machineset.yaml b/charts/aro-machinesets/templates/machineset.yaml
@@ -60,6 +60,9 @@ spec:
           publicLoadBalancer: {{ $machineSetSpec.publicLoadBalancer }}
           resourceGroup: {{ $machineSetSpec.resourceGroup }}
           subnet: {{ $machineSetSpec.subnet }}
+          {{- if $machineSetSpec.tags }}
+          tags: {{ $machineSetSpec.tags | toYaml | nindent 12 }}
+          {{- end }}
           userDataSecret:
             name: worker-user-data
           vmSize: {{ $.Values.vmSize }}

diff --git a/charts/aro-machinesets/values.yaml b/charts/aro-machinesets/values.yaml
@@ -51,6 +51,10 @@ machineSetSpec:
   resourceGroup: aro-cddajpe1
   subnet: aro-cluster-machine-subnet
   vnet: aro-cluster-vnet
+  ## uncomment to add tags on the vms
+  # tags:
+  #   key1: value1
+  #   key2: value2
   ## spotVMOptions - optionally declare spotVMOptions when setting enableSpot to true as described on
   ## https://docs.openshift.com/container-platform/4.12/machine_management/creating_machinesets/creating-machineset-azure.html#machineset-non-guaranteed-instance_creating-machineset-azure
   # spotVMOptions: