Skip to content

Commit

Permalink
Merge branch 'master' of github.com:skypilot-org/skypilot into client…
Browse files Browse the repository at this point in the history
…-server
  • Loading branch information
Michaelvll committed Feb 7, 2025
2 parents a5c3b52 + 5e6b39c commit 190f63a
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions tests/smoke_tests/test_managed_job.py
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ def test_job_pipeline(generic_cloud: str):
f'sky jobs launch -n {name} tests/test_yamls/pipeline.yaml --cloud {generic_cloud} -y -d',
# Need to wait for setup and job initialization.
'sleep 30',
f'{smoke_tests_utils.GET_JOB_QUEUE}| grep {name} | head -n1 | grep "STARTING\|RUNNING"',
rf'{smoke_tests_utils.GET_JOB_QUEUE} | grep {name} | head -n1 | grep "STARTING\|RUNNING"',
# `grep -A 4 {name}` finds the job with {name} and the 4 lines
# after it, i.e. the 4 tasks within the job.
# `sed -n 2p` gets the second line of the 4 lines, i.e. the first
Expand Down Expand Up @@ -299,7 +299,7 @@ def test_managed_jobs_pipeline_recovery_aws(aws_config_region):
'managed_jobs_pipeline_recovery_aws',
[
smoke_tests_utils.launch_cluster_for_cloud_cmd('aws', name),
f'sky jobs launch -n {name} tests/test_yamls/pipeline_aws.yaml -y -d',
f'sky jobs launch -n {name} tests/test_yamls/pipeline_aws.yaml -y -d',
smoke_tests_utils.
get_cmd_wait_until_managed_job_status_contains_matching_job_name(
job_name=name,
Expand Down Expand Up @@ -361,7 +361,7 @@ def test_managed_jobs_pipeline_recovery_gcp():
'managed_jobs_pipeline_recovery_gcp',
[
smoke_tests_utils.launch_cluster_for_cloud_cmd('gcp', name),
f'sky jobs launch -n {name} tests/test_yamls/pipeline_gcp.yaml -y -d',
f'sky jobs launch -n {name} tests/test_yamls/pipeline_gcp.yaml -y -d',
smoke_tests_utils.
get_cmd_wait_until_managed_job_status_contains_matching_job_name(
job_name=name,
Expand Down

0 comments on commit 190f63a

Please sign in to comment.