Switch from docker-compose to docker compose

dandi · Jul 23, 2024 · 76e7d14 · 76e7d14
1 parent f3d2f7f
commit 76e7d14
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 10 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -115,14 +115,14 @@ jobs:
     - name: Dump Docker Compose logs
       if: failure() && startsWith(matrix.os, 'ubuntu')
       run: |
-        docker-compose \
+        docker compose \
           -f dandi/tests/data/dandiarchive-docker/docker-compose.yml \
           logs --timestamps
 
     - name: Shut down Docker Compose
       if: startsWith(matrix.os, 'ubuntu')
       run: |
-        docker-compose \
+        docker compose \
           -f dandi/tests/data/dandiarchive-docker/docker-compose.yml \
           down -v
 

diff --git a/dandi/tests/fixtures.py b/dandi/tests/fixtures.py
@@ -395,16 +395,27 @@ def docker_compose_setup() -> Iterator[dict[str, str]]:
     try:
         if create:
             if os.environ.get("DANDI_TESTS_PULL_DOCKER_COMPOSE", "1") not in ("", "0"):
-                run(["docker-compose", "pull"], cwd=str(LOCAL_DOCKER_DIR), check=True)
+                run(
+                    ["docker", "compose", "pull"], cwd=str(LOCAL_DOCKER_DIR), check=True
+                )
             run(
-                ["docker-compose", "run", "--rm", "django", "./manage.py", "migrate"],
+                [
+                    "docker",
+                    "compose",
+                    "run",
+                    "--rm",
+                    "django",
+                    "./manage.py",
+                    "migrate",
+                ],
                 cwd=str(LOCAL_DOCKER_DIR),
                 env=env,
                 check=True,
             )
             run(
                 [
-                    "docker-compose",
+                    "docker",
+                    "compose",
                     "run",
                     "--rm",
                     "django",
@@ -417,7 +428,8 @@ def docker_compose_setup() -> Iterator[dict[str, str]]:
             )
             run(
                 [
-                    "docker-compose",
+                    "docker",
+                    "compose",
                     "run",
                     "--rm",
                     "-e",
@@ -436,7 +448,8 @@ def docker_compose_setup() -> Iterator[dict[str, str]]:
 
         r = check_output(
             [
-                "docker-compose",
+                "docker",
+                "compose",
                 "run",
                 "--rm",
                 "-T",
@@ -458,7 +471,7 @@ def docker_compose_setup() -> Iterator[dict[str, str]]:
 
         if create:
             run(
-                ["docker-compose", "up", "-d", "django", "celery"],
+                ["docker", "compose", "up", "-d", "django", "celery"],
                 cwd=str(LOCAL_DOCKER_DIR),
                 env=env,
                 check=True,
@@ -476,7 +489,11 @@ def docker_compose_setup() -> Iterator[dict[str, str]]:
         yield {"django_api_key": django_api_key}
     finally:
         if persist in (None, "0"):
-            run(["docker-compose", "down", "-v"], cwd=str(LOCAL_DOCKER_DIR), check=True)
+            run(
+                ["docker", "compose", "down", "-v"],
+                cwd=str(LOCAL_DOCKER_DIR),
+                check=True,
+            )
 
 
 @dataclass

diff --git a/dandi/tests/skip.py b/dandi/tests/skip.py
@@ -101,7 +101,7 @@ def windows():
 
 def no_docker_commands():
     missing_cmds = []
-    for cmd in ("docker", "docker-compose"):
+    for cmd in ("docker",):
         if shutil.which(cmd) is None:
             missing_cmds.append(cmd)
     msg = "missing Docker commands: {}".format(", ".join(missing_cmds))