Skip to content

Conversation

@rauchy
Copy link
Contributor

@rauchy rauchy commented Nov 27, 2025

Summary

Migrate integration test status reporting from GitHub Statuses API to Checks API, enabling the use of GitHub App authentication and eliminating the need for monthly PAT rotation.

Changes

  • Add validation for DECO_TEST_APPROVAL_APP_ID secret in addition to existing DECO_WORKFLOW_TRIGGER_APP_ID
  • Create check run using DECO_TEST_APPROVAL_APP before triggering tests in eng-dev-ecosystem
  • Pass check_run_id to eng-dev-ecosystem workflow for check updates
  • Update merge queue auto-approve to use Checks API instead of Statuses API

Testing

Tested successfully on databricks-sdk-go with full end-to-end flow working.

NO_CHANGELOG=true

@rauchy rauchy enabled auto-merge November 27, 2025 15:08
@rauchy rauchy added this pull request to the merge queue Nov 28, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Nov 28, 2025
@rauchy rauchy added this pull request to the merge queue Nov 28, 2025
@rauchy rauchy removed this pull request from the merge queue due to a manual request Nov 28, 2025
@rauchy rauchy force-pushed the checks-api-migration branch from 26645c6 to 98897f8 Compare November 28, 2025 12:05
@rauchy rauchy temporarily deployed to test-trigger-is November 28, 2025 12:05 — with GitHub Actions Inactive
@rauchy rauchy temporarily deployed to test-trigger-is November 28, 2025 12:06 — with GitHub Actions Inactive
@rauchy rauchy force-pushed the checks-api-migration branch from 98897f8 to 1604392 Compare November 28, 2025 12:35
@rauchy rauchy temporarily deployed to test-trigger-is November 28, 2025 12:35 — with GitHub Actions Inactive
@rauchy rauchy temporarily deployed to test-trigger-is November 28, 2025 12:36 — with GitHub Actions Inactive
@github-actions
Copy link

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-py

Inputs:

  • PR number: 1130
  • Commit SHA: 1604392362e47a2a2ba30f97dd613d1514fb92f5

Checks will be approved automatically on success.

@rauchy rauchy enabled auto-merge November 28, 2025 12:50
@rauchy rauchy added this pull request to the merge queue Nov 28, 2025
Merged via the queue into main with commit 24cd018 Nov 28, 2025
17 checks passed
@rauchy rauchy deleted the checks-api-migration branch November 28, 2025 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants