Skip to content

ci: fix dead driver-test workflow link in dispatch comment#880

Closed
eric-wang-1990 wants to merge 1 commit into
mainfrom
ci/fix-dispatch-comment-link
Closed

ci: fix dead driver-test workflow link in dispatch comment#880
eric-wang-1990 wants to merge 1 commit into
mainfrom
ci/fix-dispatch-comment-link

Conversation

@eric-wang-1990

Copy link
Copy Markdown
Contributor

Summary

The cross-repo dispatch sender comments "Integration tests triggered" on the PR with a link to the driver-test receiver workflow. The link (and two header comments) pointed to python-proxy-tests.yml, which does not exist (404) — the actual receiver file is databricks-sql-python-proxy-tests.yml.

Fixed the filename in all three spots and clarified that the authoritative results are the "Python Proxy Tests / thrift" and "Python Proxy Tests / kernel" checks posted back on the PR (the sender can't link the specific driver-test run since repository_dispatch returns no run ID).

Test Plan

  • Corrected target databricks-sql-python-proxy-tests.yml resolves 200 on driver-test main
  • Workflow YAML parses; no stale python-proxy-tests.yml reference remains

This pull request and its description were written by Isaac.

The "Integration tests triggered" PR comment (and two header comments) linked
to python-proxy-tests.yml, which does not exist (404) — the receiver file is
databricks-sql-python-proxy-tests.yml. Point to the correct workflow and note
that the authoritative results are the "Python Proxy Tests / thrift" and
"Python Proxy Tests / kernel" checks posted back on the PR.

Co-authored-by: Isaac
Copilot AI review requested due to automatic review settings July 24, 2026 07:05

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@eric-wang-1990 eric-wang-1990 added the engineer-bot Maintainer-applied gate: triggers engineer-bot (bug-fix on issue / take-over on PR). label Jul 24, 2026
@eric-wang-1990

Copy link
Copy Markdown
Contributor Author

Folded into #879 (same file, trigger-integration-tests.yml). The comment-link 404 fix now rides on that PR's branch; closing this standalone PR.

@eric-wang-1990
eric-wang-1990 deleted the ci/fix-dispatch-comment-link branch July 24, 2026 07:07

@peco-review-bot peco-review-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No issues identified by the review bot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

engineer-bot Maintainer-applied gate: triggers engineer-bot (bug-fix on issue / take-over on PR).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants