Fix interpolation for COMPOSE_PROJECT_NAME

Fixes #1073

Signed-off-by: Ruben Jenster <[email protected]>

Author: r10r

podman_compose.py

@@ -1980,6 +1980,20 @@ def _parse_compose_file(self):
                 sys.exit(1)
             content = normalize(content)
             # log(filename, json.dumps(content, indent = 2))
+
+            if not project_name:
+                project_name = content.get("name")
+                if project_name is None:
+                    # More strict then actually needed for simplicity:
+                    # podman requires [a-zA-Z0-9][a-zA-Z0-9_.-]*
+                    project_name = self.environ.get("COMPOSE_PROJECT_NAME", dir_basename.lower())
+                    project_name = norm_re.sub("", project_name)
+                    if not project_name:
+                        raise RuntimeError(f"Project name [{dir_basename}] normalized to empty")
+
+            self.project_name = project_name
+            self.environ.update({"COMPOSE_PROJECT_NAME": self.project_name})
+
             content = rec_subs(content, self.environ)
             if isinstance(services := content.get('services'), dict):
                 for service in services.values():
@@ -2012,19 +2026,6 @@ def _parse_compose_file(self):
             log.debug(" ** merged:\n%s", json.dumps(compose, indent=2))
         # ver = compose.get('version')
 
-        if not project_name:
-            project_name = compose.get("name")
-            if project_name is None:
-                # More strict then actually needed for simplicity:
-                # podman requires [a-zA-Z0-9][a-zA-Z0-9_.-]*
-                project_name = self.environ.get("COMPOSE_PROJECT_NAME", dir_basename.lower())
-                project_name = norm_re.sub("", project_name)
-                if not project_name:
-                    raise RuntimeError(f"Project name [{dir_basename}] normalized to empty")
-
-        self.project_name = project_name
-        self.environ.update({"COMPOSE_PROJECT_NAME": self.project_name})
-
         services = compose.get("services")
         if services is None:
             services = {}

tests/integration/env-tests/test_podman_compose_env.py

@@ -49,6 +49,7 @@ def test_env(self):
     - https://docs.docker.com/compose/how-tos/environment-variables/envvars/
     - https://github.com/compose-spec/compose-spec/blob/main/04-version-and-name.md
     """
+
     def test_project_name(self):
         try:
             output, _ = self.run_subprocess_assert_returncode([