Skip to content

Commit

Permalink
nit
Browse files Browse the repository at this point in the history
Signed-off-by: Kevin Su <[email protected]>
  • Loading branch information
pingsutw committed Sep 11, 2024
1 parent fdd97eb commit 69caae5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions flytekit/image_spec/image_spec.py
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ def __post_init__(self):
self._is_force_push = os.environ.get(FLYTE_FORCE_PUSH_IMAGE_SPEC, False) # False by default
if self.registry:
self.registry = self.registry.lower()
if not validate_docker_registry_name(self.registry):
if not validate_container_registry_name(self.registry):
raise ValueError(
f"Invalid container registry name: '{self.registry}'.\n Expected formats:\n"
f"- 'localhost:30000' (for local registries)\n"
Expand Down Expand Up @@ -416,7 +416,7 @@ def _get_builder(cls, builder: str) -> ImageSpecBuilder:
return cls._REGISTRY[builder][0]


def validate_docker_registry_name(name: str) -> bool:
def validate_container_registry_name(name: str) -> bool:
"""Validate Docker container registry name."""
# Define the regular expression for the registry name
registry_pattern = r"^(localhost:\d{1,5}|([a-z0-9._-]+)(:[0-9]{1,5})?)(/[a-zA-Z0-9._-]+)*$"
Expand Down

0 comments on commit 69caae5

Please sign in to comment.