Skip to content

Commit 974afa4

Browse files
evakraviNamrata Madan
authored and
Namrata Madan
committed
fix: Fix failing jumpstart cache unit tests (aws#3514)
1 parent d87d13a commit 974afa4

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

setup.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ def read_requirements(filename):
5555
"protobuf3-to-dict>=0.1.5,<1.0",
5656
"smdebug_rulesconfig==1.0.1",
5757
"importlib-metadata>=1.4.0,<5.0",
58-
"packaging>=20.0",
58+
"packaging==20.9",
5959
"pandas",
6060
"pathos",
6161
"schema",

src/sagemaker/jumpstart/cache.py

+5-2
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
import boto3
2121
import botocore
2222
from packaging.version import Version
23-
from packaging.specifiers import SpecifierSet
23+
from packaging.specifiers import SpecifierSet, InvalidSpecifier
2424
from sagemaker.jumpstart.constants import (
2525
ENV_VARIABLE_JUMPSTART_MANIFEST_LOCAL_ROOT_DIR_OVERRIDE,
2626
ENV_VARIABLE_JUMPSTART_SPECS_LOCAL_ROOT_DIR_OVERRIDE,
@@ -371,7 +371,10 @@ def _select_version(
371371
return None
372372
return str(max(available_versions))
373373

374-
spec = SpecifierSet(f"=={semantic_version_str}")
374+
try:
375+
spec = SpecifierSet(f"=={semantic_version_str}")
376+
except InvalidSpecifier:
377+
raise KeyError(f"Bad semantic version: {semantic_version_str}")
375378
available_versions_filtered = list(spec.filter(available_versions))
376379
return (
377380
str(max(available_versions_filtered)) if available_versions_filtered != [] else None

0 commit comments

Comments
 (0)