Merge pull request #117 from epics-containers/instance-path

check for Chart.yaml instead of optional values.yaml
epics-containers · Mar 26, 2024 · 88b0b0c · 88b0b0c
2 parents 08ecdb2 + 01a0e31
commit 88b0b0c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/edge_containers_cli/utils.py b/src/edge_containers_cli/utils.py
@@ -49,8 +49,8 @@ def check_instance_path(service_path: Path):
 
     log.info(f"checking instance {service_name} at {service_path}")
     if service_path.is_dir():
-        if not (service_path / "values.yaml").exists():
-            log.error("IOC instance requires values.yaml")
+        if not (service_path / "Chart.yaml").exists():
+            log.error("A Service instance requires Chart.yaml")
             raise typer.Exit(1)
     else:
         log.error(f"instance path {service_path} does not exist")

diff --git a/tests/data/services/bl45p-ea-ioc-01/Chart.yaml b/tests/data/services/bl45p-ea-ioc-01/Chart.yaml
@@ -0,0 +1 @@
+# placeholder