🤦

Author: BYK

_integration-test/test_02_backup.py

@@ -50,9 +50,9 @@ def test_02_import(setup_backup_restore_env_variables):
     # We cannot just rm and create them as they still need the migrations.
     for name in ("postgres", "clickhouse", "kafka"):
         subprocess.run(["docker", "volume", "rm", f"sentry-{name}"], check=True)
+        subprocess.run(["docker", "volume", "create", f"sentry-{name}"], check=True)
         subprocess.run(
             [
-                "sudo",
                 "rsync",
                 "-aW",
                 "--super",
@@ -65,7 +65,6 @@ def test_02_import(setup_backup_restore_env_variables):
             check=True,
             capture_output=True,
         )
-        subprocess.run(["docker", "volume", "create", f"sentry-{name}"], check=True)
 
     subprocess.run(
         ["docker", "compose", "--ansi", "never", "up", "--wait"],

action.yaml

@@ -110,7 +110,8 @@ runs:
     - name: Integration Test
       shell: bash
       run: |
-        sudo rsync -aW --super --numeric-ids --no-compress --mkpath \
+        sudo chown root /usr/bin/rsync && sudo chmod u+s /usr/bin/rsync
+        rsync -aW --super --numeric-ids --no-compress --mkpath \
           /var/lib/docker/volumes/sentry-postgres \
           /var/lib/docker/volumes/sentry-clickhouse \
           /var/lib/docker/volumes/sentry-kafka \