Addressing comment

Author: simrankaurb

community/examples/hpc-slurm6-tpu.yaml

@@ -21,7 +21,6 @@ vars:
   deployment_name: slurm6-tpu
   region: us-central1
   zone: us-central1-a
-  nodeset_preemptible: false
 
 deployment_groups:
 - group: primary
@@ -38,7 +37,7 @@ deployment_groups:
       # Preemptible TPUs cost much less than non-preemptible TPUs.
       # The Cloud TPU service might preempt (shut down) these TPUs at any time.
       # https://cloud.google.com/tpu/docs/preemptible
-      preemptible: $(vars.nodeset_preemptible)
+      preemptible: false
       # Specify whether to preserve TPU on suspend.
       # If set to true, suspended VM will be stopped.
       # If set to false, suspended VM will be deleted.

tools/cloud-build/daily-tests/builds/slurm-gcp-v6-tpu.yaml

@@ -57,6 +57,8 @@ steps:
     cd /workspace && make
     REGION="$${ZONE%-*}"
     BUILD_ID_SHORT="$${BUILD_ID:0:6}"
+    BLUEPRINT="/workspace/community/examples/hpc-slurm6-tpu.yaml"
+    sed -i 's/^[ ]*preemptible: false/      preemptible: true/' $${BLUEPRINT}
 
     ansible-playbook tools/cloud-build/daily-tests/ansible_playbooks/slurm-integration-test.yml \
       --user=sa_106486320838376751393 \

tools/cloud-build/daily-tests/tests/slurm-v6-tpu.yml

@@ -23,7 +23,6 @@ slurm_cluster_name: "v6tpu{{ build[0:5] }}"
 cli_deployment_vars:
    region: "{{ region }}"
    zone: "{{ zone }}"
-   nodeset_preemptible: true
 
 workspace: /workspace
 blueprint_yaml: "{{ workspace }}/community/examples/hpc-slurm6-tpu.yaml"