Merge pull request #243 from protocol/clean-concurrency

galargh · web-flow · commit b69bd2ada022 · 2021-12-10T12:39:57.000+01:00
Replace concurrency setup with max-parallel=1 strategy in copy-workflow
diff --git a/.github/workflows/copy-workflow.yml b/.github/workflows/copy-workflow.yml
@@ -14,9 +14,6 @@ on:
       targets:
         description: "List of repositories to deploy to"
         required: true
-      concurrency_group:
-        description: "Concurrency Group"
-        required: true
 
 jobs:
   copy:
@@ -25,7 +22,7 @@ jobs:
       fail-fast: false
       matrix:
         cfg: ${{ fromJson(github.event.inputs.targets) }}
-    concurrency: copy-${{ github.event.inputs.concurrency_group }}
+      max-parallel: 1
     env:
       TARGET_REPO_DIR: "target-repo"
       TEMPLATE_REPO_DIR: "template-repo"
diff --git a/.github/workflows/dispatch.yml b/.github/workflows/dispatch.yml
@@ -8,8 +8,7 @@ on:
 
 env:
   # Number of repositories in a batch.
-  # Deployment jobs in the same batch use the same concurrency group,
-  # and are therefore run sequentially.
+  # Deployment jobs in the same batch are run sequentially.
   # With ~180 repos, this means we'll run around 9 instances of the copy workflow in parallel.
   # This is (hopefully) sufficient to prevent us from triggering GitHub Action's abuse detection mechanism.
   MAX_REPOS_PER_WORKFLOW: 20
@@ -52,4 +51,4 @@ jobs:
         with:
           workflow: "Deploy" # "name" attribute of copy-workflow.yml
           token: ${{ secrets.WEB3BOT_GITHUB_TOKEN }}
-          inputs: '{ "head_commit_url": ${{ toJson(github.event.head_commit.url) }}, "files": ${{ toJson(env.FILES) }}, "targets": ${{ toJson(toJson(matrix.cfg.value)) }}, "concurrency_group": "${{ matrix.cfg.key }}" }'
+          inputs: '{ "head_commit_url": ${{ toJson(github.event.head_commit.url) }}, "files": ${{ toJson(env.FILES) }}, "targets": ${{ toJson(toJson(matrix.cfg.value)) }} }'
