ci: fix mergify matrix integration tests (#8267)

Agoric · Jan 12, 2024 · 20c5f52 · 20c5f52
2 parents 1a281f5 + d1022c4
commit 20c5f52
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 33 deletions.
diff --git a/.github/workflows/integration.yml b/.github/workflows/integration.yml
@@ -19,6 +19,7 @@ on:
       - converted_to_draft
       - ready_for_review
       - labeled
+      - unlabeled
       - auto_merge_enabled
       - auto_merge_disabled
   merge_group:
@@ -253,3 +254,22 @@ jobs:
         timeout-minutes: 4
         with:
           datadog-token: ${{ secrets.DATADOG_API_KEY }}
+
+  integration-test-result:
+    needs:
+      - pre_check
+      - getting-started
+      - deployment-test
+      - test-docker-build
+    if: needs.pre_check.outputs.should_run == 'true' && (success() || failure() || cancelled())
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check job results
+        shell: bash
+        run: |
+          cat <<EOF
+            needs ${{ toJSON(needs) }}
+          EOF
+          [ "${{ needs.getting-started.result }}" = "success" ] || exit 1
+          [ "${{ needs.deployment-test.result }}" = "success" ] || exit 1
+          [ "${{ needs.test-docker-build.result }}" = "success" ] || exit 1
diff --git a/.github/workflows/mergify-ready.yml b/.github/workflows/mergify-ready.yml
@@ -9,6 +9,7 @@ on:
       - converted_to_draft
       - ready_for_review
       - labeled
+      - unlabeled
       - auto_merge_enabled
       - auto_merge_disabled
   merge_group:

diff --git a/.mergify.yml b/.mergify.yml
@@ -5,46 +5,17 @@ queue_rules:
       - base=master
       # Require integration tests before merging only
       - or:
-          - label=bypass:integration
-          - check-failure!=Integration tests
-      - or:
-          - label=bypass:integration
-          - check-success=deployment-test
-          - check-neutral=deployment-test
-          - check-skipped=deployment-test
-      - or:
-          - label=bypass:integration
-          - check-failure!=test-docker-build (main)
-      - or:
-          - label=bypass:integration
-          - check-skipped=test-docker-build
-          - check-success=test-docker-build (main)
-          - check-neutral=test-docker-build (main)
-          - check-skipped=test-docker-build (main)
-      - or:
-          - label=bypass:integration
-          - check-failure!=getting-started (link-cli)
-      - or:
-          - label=bypass:integration
-          - check-skipped=getting-started
-          - check-success=getting-started (link-cli)
-          - check-neutral=getting-started (link-cli)
-          - check-skipped=getting-started (link-cli)
-      # FIXME: Enable this section to validate NPM deploys...
-      #- or:
-      #    - label=bypass:integration
-      #    - check-skipped=getting-started
-      #    - check-success=getting-started (local-npm)
-      #    - check-neutral=getting-started (local-npm)
-      #    - check-skipped=getting-started (local-npm)
+          - check-success=integration-test-result
+          - check-neutral=integration-test-result
+          - check-skipped=integration-test-result
 
 pull_request_rules:
   - name: merge to master
     conditions:
       - base=master
       - label=automerge:no-update
       - or:
-          - "#commits-behind=0"
+          - '#commits-behind=0'
           - label=bypass:linear-history
       - or:
           - check-success=wait-integration-pre-checks