Skip to content

Commit

Permalink
fix: det deploy gcp down shouldn't check quotas (#8931)
Browse files Browse the repository at this point in the history
For some reason down calls dry_run, which checks quotas. The output of dry_run isn't relevant 
to a delete, and we don't need to check quotas when destroying a cluster.
  • Loading branch information
stoksc authored Mar 1, 2024
1 parent 21fb6d1 commit b99ad9f
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion harness/determined/deploy/gcp/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,6 @@ def deploy_gcp(command: str, args: argparse.Namespace) -> None:
"tf_state_gcs_bucket_name": args.project_id + "-determined-deploy",
}

gcp.dry_run(det_configs, env, variables_to_exclude)
gcp.delete(det_configs, env, args.yes)
print("Delete Successful")
return
Expand Down

0 comments on commit b99ad9f

Please sign in to comment.