Skip to content

Commit

Permalink
fix rsync bug due to refactor
Browse files Browse the repository at this point in the history
  • Loading branch information
KeplerC committed Jan 7, 2025
1 parent 8c7bacf commit 9d34510
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions sky/backends/cloud_vm_ray_backend.py
Original file line number Diff line number Diff line change
Expand Up @@ -3921,7 +3921,7 @@ def sync_down_managed_job_logs(
f'Multiple jobs IDs found under the name {job_name}. '
'Downloading the latest job logs.'
f'{colorama.Style.RESET_ALL}')
job_ids = [job_ids[-1]]
job_ids = [job_ids[0]] # descending order
else:
job_ids = [job_id]

Expand Down Expand Up @@ -3954,7 +3954,7 @@ def sync_down_managed_job_logs(
os.path.join(local_dir, run_timestamp))

logger.info(f'{colorama.Fore.CYAN}'
f'Job {job_ids} logs: {local_log_dir}'
f'Job {job_ids} local logs: {local_log_dir} remote logs: {remote_log_dir}'
f'{colorama.Style.RESET_ALL}')

runners = handle.get_command_runners()
Expand Down Expand Up @@ -3984,7 +3984,7 @@ def _rsync_down(args) -> None:
raise

parallel_args = [[runner, *item]
for item in zip(local_log_dir, remote_log_dir)
for item in zip([local_log_dir], [remote_log_dir])
for runner in runners]
subprocess_utils.run_in_parallel(_rsync_down, parallel_args)
else: # download job logs
Expand Down

0 comments on commit 9d34510

Please sign in to comment.