Skip to content

Commit ebd2ed5

Browse files
Split trigger-tests job and route workflow dispatch through emu-access runner (#1638)
## Summary Fixes automated Integration Tests for PRs on this repo. Splits `trigger-tests` into `create-check` (stays on deco runners, creates the check run on this repo) and `trigger-tests` (moves to `databricks-release-runner-group-emu-access` to perform the cross-org dispatch to `databricks-eng/eng-dev-ecosystem`). ## Why Since the `databricks` org tightened its IP allow list (between 2026-04-17 and 2026-04-20), the deco runner group can no longer call `/repos/databricks-eng/.../installation`, so `create-github-app-token` fails with 403 and no dispatch is sent. Every PR-triggered Integration Tests run on this repo has failed at that step since. Merges only work because `merge_group` auto-approves without running tests. ## What changed Exactly the diff from Hector's #1616, which was already verified green end-to-end on 2026-04-10. Supersedes #1616. ## How is this tested The PR's own Integration Tests run is the test. Confirms: - `create-check`: success (check run appears on PR head) - `trigger-tests`: success (`sdk-go-isolated-pr` dispatch appears on `databricks-eng/eng-dev-ecosystem`, event `workflow_dispatch`) - dispatched eng-dev run completes with `mark-as-pending`/`success`/`failure` green (post-#1250) - the Integration Tests check on the PR transitions `in_progress` → `success` NO_CHANGELOG=true Signed-off-by: Mihai Mitrea <mihai.mitrea@databricks.com>
1 parent b95fd54 commit ebd2ed5

1 file changed

Lines changed: 19 additions & 3 deletions

File tree

.github/workflows/integration-tests.yml

Lines changed: 19 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,8 @@ jobs:
3131
echo "has_token=true" >> $GITHUB_OUTPUT
3232
fi
3333
34-
trigger-tests:
35-
name: Trigger Tests
34+
create-check:
35+
name: Create Check Run
3636

3737
runs-on:
3838
group: databricks-deco-testing-runner-group
@@ -41,6 +41,8 @@ jobs:
4141
needs: check-token
4242
if: github.event_name == 'pull_request' && needs.check-token.outputs.has_token == 'true'
4343
environment: "test-trigger-is"
44+
outputs:
45+
check_run_id: ${{ steps.create-check.outputs.check_run_id }}
4446

4547
steps:
4648
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
@@ -69,6 +71,20 @@ jobs:
6971
check_run_id=$(echo "$response" | jq -r .id)
7072
echo "check_run_id=$check_run_id" >> $GITHUB_OUTPUT
7173
74+
trigger-tests:
75+
name: Trigger Tests
76+
77+
runs-on:
78+
group: databricks-release-runner-group-emu-access
79+
labels: linux-ubuntu-latest-emu-access
80+
81+
needs: [check-token, create-check]
82+
if: github.event_name == 'pull_request' && needs.check-token.outputs.has_token == 'true'
83+
environment: "test-trigger-is"
84+
85+
steps:
86+
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
87+
7288
- name: Generate GitHub App Token for Workflow Trigger
7389
id: generate-token
7490
uses: actions/create-github-app-token@f8d387b68d61c58ab83c6c016672934102569859 # v3.0.0
@@ -86,7 +102,7 @@ jobs:
86102
--ref main \
87103
-f pull_request_number=${{ github.event.pull_request.number }} \
88104
-f commit_sha=${{ github.event.pull_request.head.sha }} \
89-
-f check_run_id=${{ steps.create-check.outputs.check_run_id }}
105+
-f check_run_id=${{ needs.create-check.outputs.check_run_id }}
90106
91107
92108
# The hash for the merge queue may not be the same as the hash for the PR.

0 commit comments

Comments
 (0)