Skip to content

[Internal] Migrate to Checks API for integration tests - #557

Merged
rauchy merged 1 commit into
mainfrom
checks-api-migration
Nov 28, 2025
Merged

rauchy merged 1 commit into
mainfrom
checks-api-migration

Conversation

@rauchy

@rauchy rauchy commented Nov 27, 2025

Copy link
Copy Markdown
Contributor

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 added this pull request to the merge queue Nov 27, 2025
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Nov 27, 2025
@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 6d37c2b to 4e4a443 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 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 4e4a443 to ba13214 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
Copy Markdown
Contributor

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-java

Inputs:

  • PR number: 557
  • Commit SHA: ba13214ee5c335198c765259227dd3a887b3c210

Checks will be approved automatically on success.

@rauchy
rauchy added this pull request to the merge queue Nov 28, 2025
Merged via the queue into main with commit 32fc1a2 Nov 28, 2025
16 checks passed
@rauchy
rauchy deleted the checks-api-migration branch November 28, 2025 12:48

This branch was previously deployed

1 inactive deployment
test-trigger-is — ba13214e Deployed Nov 28, 2025 by rauchy via Trigger Tests #732
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.

2 participants