Merge pull request #69 from epics-containers/fix-tests

fix dependabot tag issue

Author: gilesknap

src/epics_containers_cli/dev/dev_commands.py

@@ -8,7 +8,11 @@
 from epics_containers_cli.git import get_git_name, get_image_name
 from epics_containers_cli.logging import log
 from epics_containers_cli.shell import run_command
-from epics_containers_cli.utils import check_ioc_instance_path, get_instance_image_name
+from epics_containers_cli.utils import (
+    check_ioc_instance_path,
+    get_instance_image_name,
+    normalize_tag,
+)
 
 from ..globals import (
     CONFIG_FOLDER,
@@ -91,6 +95,7 @@ def launch_local(
             execute = f"{IOC_START}; bash"
 
         repo, _ = get_git_name(generic_ioc)
+        tag = normalize_tag(tag)
         image = get_image_name(repo, target=target) + f":{tag}"
 
         self._do_launch(ioc_name, target, image, execute, args, mounts)
@@ -198,6 +203,7 @@ def build(
         """
         repo, _ = get_git_name(generic_ioc)
         args = f"--platform {platform} {'--no-cache' if not cache else ''}"
+        tag = normalize_tag(tag)
 
         if target is None:
             targets = [Targets.developer.value, Targets.runtime.value]

src/epics_containers_cli/utils.py

@@ -96,3 +96,13 @@ def chdir(path):
         yield
     finally:
         os.chdir(curdir)
+
+
+def normalize_tag(tag: str) -> str:
+    """
+    normalize a tag to be lowercase and replace any '/'
+    this is needed in CI because dependabot tags
+    """
+    tag = tag.lower()
+    tag = tag.replace("/", "-")
+    return tag