Skip to content

Commit efd596d

Browse files
caxiaohuchuyang-deng
authored andcommitted
update: disable efs fsx integ tests in non-pdx regions (#1018)
1 parent ef932b3 commit efd596d

File tree

2 files changed

+34
-0
lines changed

2 files changed

+34
-0
lines changed

tests/integ/test_kmeans_efs_fsx.py

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
from sagemaker.parameter import IntegerParameter, CategoricalParameter
2020
from sagemaker.tuner import HyperparameterTuner
2121
from sagemaker.utils import unique_name_from_base
22+
import tests
2223
from tests.integ import TRAINING_DEFAULT_TIMEOUT_MINUTES, TUNING_DEFAULT_TIMEOUT_MINUTES
2324
from tests.integ.file_system_input_utils import set_up_efs_fsx, tear_down
2425
from tests.integ.s3_utils import assert_s3_files_exist
@@ -46,6 +47,10 @@ def efs_fsx_setup(sagemaker_session, ec2_instance_type):
4647
tear_down(sagemaker_session, fs_resources)
4748

4849

50+
@pytest.mark.skipif(
51+
tests.integ.test_region() not in tests.integ.EFS_TEST_ENABLED_REGION,
52+
reason="EFS integration tests need to be fixed before running in all regions.",
53+
)
4954
def test_kmeans_efs(efs_fsx_setup, sagemaker_session, cpu_instance_type):
5055
with timeout(minutes=TRAINING_DEFAULT_TIMEOUT_MINUTES):
5156
role = efs_fsx_setup["role_name"]
@@ -77,6 +82,10 @@ def test_kmeans_efs(efs_fsx_setup, sagemaker_session, cpu_instance_type):
7782
assert_s3_files_exist(sagemaker_session, model_path, ["model.tar.gz"])
7883

7984

85+
@pytest.mark.skipif(
86+
tests.integ.test_region() not in tests.integ.EFS_TEST_ENABLED_REGION,
87+
reason="EFS integration tests need to be fixed before running in all regions.",
88+
)
8089
def test_kmeans_fsx(efs_fsx_setup, sagemaker_session, cpu_instance_type):
8190
with timeout(minutes=TRAINING_DEFAULT_TIMEOUT_MINUTES):
8291
role = efs_fsx_setup["role_name"]
@@ -107,6 +116,10 @@ def test_kmeans_fsx(efs_fsx_setup, sagemaker_session, cpu_instance_type):
107116
assert_s3_files_exist(sagemaker_session, model_path, ["model.tar.gz"])
108117

109118

119+
@pytest.mark.skipif(
120+
tests.integ.test_region() not in tests.integ.EFS_TEST_ENABLED_REGION,
121+
reason="EFS integration tests need to be fixed before running in all regions.",
122+
)
110123
def test_tuning_kmeans_efs(efs_fsx_setup, sagemaker_session, cpu_instance_type):
111124
role = efs_fsx_setup["role_name"]
112125
subnets = [efs_fsx_setup["subnet_id"]]
@@ -163,6 +176,10 @@ def test_tuning_kmeans_efs(efs_fsx_setup, sagemaker_session, cpu_instance_type):
163176
assert best_training_job
164177

165178

179+
@pytest.mark.skipif(
180+
tests.integ.test_region() not in tests.integ.EFS_TEST_ENABLED_REGION,
181+
reason="EFS integration tests need to be fixed before running in all regions.",
182+
)
166183
def test_tuning_kmeans_fsx(efs_fsx_setup, sagemaker_session, cpu_instance_type):
167184
role = efs_fsx_setup["role_name"]
168185
subnets = [efs_fsx_setup["subnet_id"]]

tests/integ/test_tf_efs_fsx.py

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
from sagemaker.tensorflow import TensorFlow
2323
from sagemaker.tuner import HyperparameterTuner
2424
from sagemaker.utils import unique_name_from_base
25+
import tests
2526
from tests.integ import TRAINING_DEFAULT_TIMEOUT_MINUTES, TUNING_DEFAULT_TIMEOUT_MINUTES
2627
from tests.integ.file_system_input_utils import tear_down, set_up_efs_fsx
2728
from tests.integ.s3_utils import assert_s3_files_exist
@@ -49,6 +50,10 @@ def efs_fsx_setup(sagemaker_session, ec2_instance_type):
4950
tear_down(sagemaker_session, fs_resources)
5051

5152

53+
@pytest.mark.skipif(
54+
tests.integ.test_region() not in tests.integ.EFS_TEST_ENABLED_REGION,
55+
reason="EFS integration tests need to be fixed before running in all regions.",
56+
)
5257
def test_mnist_efs(efs_fsx_setup, sagemaker_session, cpu_instance_type):
5358
role = efs_fsx_setup["role_name"]
5459
subnets = [efs_fsx_setup["subnet_id"]]
@@ -81,6 +86,10 @@ def test_mnist_efs(efs_fsx_setup, sagemaker_session, cpu_instance_type):
8186
)
8287

8388

89+
@pytest.mark.skipif(
90+
tests.integ.test_region() not in tests.integ.EFS_TEST_ENABLED_REGION,
91+
reason="EFS integration tests need to be fixed before running in all regions.",
92+
)
8493
def test_mnist_lustre(efs_fsx_setup, sagemaker_session, cpu_instance_type):
8594
role = efs_fsx_setup["role_name"]
8695
subnets = [efs_fsx_setup["subnet_id"]]
@@ -113,6 +122,10 @@ def test_mnist_lustre(efs_fsx_setup, sagemaker_session, cpu_instance_type):
113122
)
114123

115124

125+
@pytest.mark.skipif(
126+
tests.integ.test_region() not in tests.integ.EFS_TEST_ENABLED_REGION,
127+
reason="EFS integration tests need to be fixed before running in all regions.",
128+
)
116129
def test_tuning_tf_script_mode_efs(efs_fsx_setup, sagemaker_session, cpu_instance_type):
117130
role = efs_fsx_setup["role_name"]
118131
subnets = [efs_fsx_setup["subnet_id"]]
@@ -157,6 +170,10 @@ def test_tuning_tf_script_mode_efs(efs_fsx_setup, sagemaker_session, cpu_instanc
157170
assert best_training_job
158171

159172

173+
@pytest.mark.skipif(
174+
tests.integ.test_region() not in tests.integ.EFS_TEST_ENABLED_REGION,
175+
reason="EFS integration tests need to be fixed before running in all regions.",
176+
)
160177
def test_tuning_tf_script_mode_lustre(efs_fsx_setup, sagemaker_session, cpu_instance_type):
161178
role = efs_fsx_setup["role_name"]
162179
subnets = [efs_fsx_setup["subnet_id"]]

0 commit comments

Comments
 (0)