Skip to content

Commit e2bbfa9

Browse files
committed
fix: Fix Experiment Run integ test w.r.t unexpected boto3 version
1 parent 0e664c8 commit e2bbfa9

File tree

4 files changed

+14
-20
lines changed

4 files changed

+14
-20
lines changed

tests/data/experiment/inference.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -10,21 +10,21 @@
1010
# on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
1111
# express or implied. See the License for the specific language governing
1212
# permissions and limitations under the License.
13-
import json
14-
import logging
1513
import os
16-
import pickle as pkl
17-
18-
import boto3
19-
import numpy as np
20-
import sagemaker_xgboost_container.encoder as xgb_encoders
2114

2215
sdk_name = "sagemaker-dev-1.0.tar.gz"
2316
code_dir = "/opt/ml/code"
2417

2518
sdk_file = f"{code_dir}/{sdk_name}"
2619
os.system(f"pip install {sdk_file}")
2720

21+
import json
22+
import logging
23+
import pickle as pkl
24+
import boto3
25+
import numpy as np
26+
import sagemaker_xgboost_container.encoder as xgb_encoders
27+
2828

2929
def _get_client_config_in_dict(cfg_in_str) -> dict:
3030
return json.loads(cfg_in_str) if cfg_in_str else None

tests/data/experiment/process_job_script_for_run_clz.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,15 +13,14 @@
1313
"""This script file runs on SageMaker processing job"""
1414
from __future__ import absolute_import
1515

16-
import json
17-
import logging
1816
import os
19-
import boto3
2017

2118
sdk_file = "sagemaker-dev-1.0.tar.gz"
2219
os.system(f"pip install {sdk_file}")
2320

24-
21+
import json
22+
import logging
23+
import boto3
2524
from sagemaker import Session
2625
from sagemaker.experiments import load_run
2726

tests/data/experiment/train_job_script_for_run_clz.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,15 +13,15 @@
1313
"""This script file runs on SageMaker training job"""
1414
from __future__ import absolute_import
1515

16-
import json
17-
import logging
18-
import time
1916
import os
20-
import boto3
2117

2218
sdk_file = "sagemaker-dev-1.0.tar.gz"
2319
os.system(f"pip install {sdk_file}")
2420

21+
import json
22+
import logging
23+
import time
24+
import boto3
2525
from sagemaker import Session
2626
from sagemaker.experiments import load_run, Run
2727

tests/integ/sagemaker/experiments/test_run.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,6 @@ def verify_is_run():
171171
_RUN_LOAD = "load"
172172

173173

174-
@pytest.mark.skip(reason="re:Invent keynote3 blocker. Revisit after release")
175174
def test_run_from_local_and_train_job_and_all_exp_cfg_match(
176175
sagemaker_session,
177176
dev_sdk_tar,
@@ -272,7 +271,6 @@ def test_run_from_local_and_train_job_and_all_exp_cfg_match(
272271
)
273272

274273

275-
@pytest.mark.skip(reason="re:Invent keynote3 blocker. Revisit after release")
276274
def test_run_from_local_and_train_job_and_exp_cfg_not_match(
277275
sagemaker_session,
278276
dev_sdk_tar,
@@ -359,7 +357,6 @@ def test_run_from_local_and_train_job_and_exp_cfg_not_match(
359357
)
360358

361359

362-
@pytest.mark.skip(reason="re:Invent keynote3 blocker. Revisit after release")
363360
def test_run_from_train_job_only(
364361
sagemaker_session,
365362
dev_sdk_tar,
@@ -409,7 +406,6 @@ def test_run_from_train_job_only(
409406

410407

411408
# dev_sdk_tar is required to trigger generating the dev SDK tar
412-
@pytest.mark.skip(reason="re:Invent keynote3 blocker. Revisit after release")
413409
def test_run_from_processing_job_and_override_default_exp_config(
414410
sagemaker_session,
415411
dev_sdk_tar,
@@ -571,7 +567,6 @@ def test_run_from_transform_job(
571567

572568

573569
# dev_sdk_tar is required to trigger generating the dev SDK tar
574-
@pytest.mark.skip(reason="re:Invent keynote3 blocker. Revisit after release")
575570
def test_load_run_auto_pass_in_exp_config_to_job(
576571
sagemaker_session,
577572
dev_sdk_tar,

0 commit comments

Comments
 (0)