Fix deprecated mc call, add more logging on CI failure (#555)

* Tear down existing containers in CI & wait for buckets

* Print logs on failure

* Actually print logs

* Post all available logs

* Typo

* Add `FOLLOW_LOGS` variable to print everything via `just logs`

* Use updated `mc` command

Per: minio/minio#21327 (comment)

* Remove debug lines

Author: AetherUnbound

.github/workflows/pull_request.yml

@@ -32,3 +32,7 @@ jobs:
         with:
           name: playwright-traces
           path: build/test-results/
+      - if: ${{ failure() }}
+        name: Print logs
+        run: |
+          FOLLOW_LOGS='' just logs

docker-compose.dev.yml

@@ -31,7 +31,7 @@ services:
     entrypoint: >
       /bin/sh -c "
       sleep 5;
-      /usr/bin/mc config host add myminio http://minio:9000 minio ${MINIO_ROOT_PASSWORD};
+      /usr/bin/mc alias set myminio http://minio:9000 minio ${MINIO_ROOT_PASSWORD};
       /usr/bin/mc mb myminio/${S3_BUCKET_NAME};
       /usr/bin/mc anonymous set public myminio/${S3_BUCKET_NAME};
       exit 0;

justfile

@@ -6,6 +6,7 @@ COMPOSE_FILE := "--file=docker-compose.yml" + (
 DC := "docker compose " + COMPOSE_FILE
 RUN := DC + " run --rm"
 RUN_WEB := RUN + " web"
+FOLLOW_LOGS := env("FOLLOW_LOGS", "-f")
 set dotenv-load := false
 # Force just to hand down positional arguments so quoted arguments with spaces are
 # handled appropriately
@@ -42,7 +43,7 @@ down *args:
 
 # Attach logs to all (or the specified) services
 logs *args:
-	{{ DC }} logs -f {{ args }}
+	{{ DC }} logs {{ FOLLOW_LOGS }} {{ args }}
 
 # Pull all docker images
 pull: