Skip to content

Commit

Permalink
fix docstring for write_cluster_config
Browse files Browse the repository at this point in the history
This used to be true, but since #2943, 'ray' is the only provisioner.
Add other keys that are now present instead.
  • Loading branch information
cg505 committed Nov 7, 2024
1 parent d833bcb commit fab22d0
Showing 1 changed file with 8 additions and 5 deletions.
13 changes: 8 additions & 5 deletions sky/backends/backend_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -641,11 +641,14 @@ def write_cluster_config(
keep_launch_fields_in_existing_config: bool = True) -> Dict[str, str]:
"""Fills in cluster configuration templates and writes them out.
Returns: {provisioner: path to yaml, the provisioning spec}.
'provisioner' can be
- 'ray'
- 'tpu-create-script' (if TPU is requested)
- 'tpu-delete-script' (if TPU is requested)
Returns:
Dict with the following keys:
- 'ray': Path to the generated Ray yaml config file
- 'cluster_name': Name of the cluster
- 'cluster_name_on_cloud': Name of the cluster as it appears in the
cloud provider
- 'config_hash': Hash of the cluster config and file mounts contents
Raises:
exceptions.ResourcesUnavailableError: if the region/zones requested does
not appear in the catalog, or an ssh_proxy_command is specified but
Expand Down

0 comments on commit fab22d0

Please sign in to comment.