Set the default SKYPILOT_BUCKET to "" so users are forced to set it a…

…s an env var.
mjkanji · Mar 15, 2024 · d14e57c · d14e57c
1 parent 298a5b6
commit d14e57c
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/dagster_skypilot/finetune.yaml b/dagster_skypilot/finetune.yaml
@@ -6,11 +6,10 @@ resources:
   disk_tier: best
 
 envs:
-  # The ID of the Dagster run that trggered the job.
-  # Overwritten by the Dagster process.
-  DAGSTER_RUN_ID: "no-run"
-  HF_TOKEN: "" # We'll pass this via the Dagster Cloud UI or a .env file instead
-  SKYPILOT_BUCKET: s3://dagster-skypilot-bucket # Change to your own bucket name
+  # The first three env vars are left empty and overwritten by the Dagster process.
+  DAGSTER_RUN_ID: "no-run" # The ID of the Dagster run that triggered the job.
+  HF_TOKEN: ""
+  SKYPILOT_BUCKET: ""
   MAX_STEPS: 10
   TERM: "dumb"
   NO_COLOR: 1