From f83d11429d18e1ef56cca75531b775d2991435a4 Mon Sep 17 00:00:00 2001 From: Chris McKenzie Date: Wed, 22 Jan 2025 10:15:55 -0800 Subject: [PATCH] tests: removing a useful extratimeout feature --- tests/smoke_tests/test_basic.py | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/tests/smoke_tests/test_basic.py b/tests/smoke_tests/test_basic.py index 1b2adffe70f..e837e0cf145 100644 --- a/tests/smoke_tests/test_basic.py +++ b/tests/smoke_tests/test_basic.py @@ -33,8 +33,6 @@ from sky.skylet import events from sky.utils import common_utils -EXTRA_TIMEOUT = int(os.getenv('EXTRA_TIMEOUT') or 0) - # ---------- Dry run: 2 Tasks in a chain. ---------- @pytest.mark.no_vast #requires GCP and AWS set up @@ -84,7 +82,7 @@ def test_minimal(generic_cloud: str): f'sky exec -c {name} && exit 1 || true', ], f'sky down -y {name}', - smoke_tests_utils.get_timeout(generic_cloud) + EXTRA_TIMEOUT, + smoke_tests_utils.get_timeout(generic_cloud), ) smoke_tests_utils.run_one_test(test) @@ -113,7 +111,7 @@ def test_launch_fast(generic_cloud: str): f'sky status -r {name} | grep UP', ], f'sky down -y {name}', - timeout=smoke_tests_utils.get_timeout(generic_cloud) + EXTRA_TIMEOUT, + timeout=smoke_tests_utils.get_timeout(generic_cloud), ) smoke_tests_utils.run_one_test(test) @@ -151,7 +149,7 @@ def test_launch_fast_with_autostop(generic_cloud: str): ], f'sky down -y {name}', timeout=smoke_tests_utils.get_timeout(generic_cloud) + - autostop_timeout + EXTRA_TIMEOUT, + autostop_timeout, ) smoke_tests_utils.run_one_test(test)