docs(kernel-e2e): refresh workflow header — setup is done, queue is live#809
Merged
Merged
Conversation
The "Required external setup" block read like a TODO list, but every item on it is in place after #808 landed and the main ruleset was updated with required_status_checks + merge_queue. Reword as a "things this depends on (debugging crib)" and add the ruleset note so future readers know why the merge_group path matters. Also serves as a low-risk pilot PR for exercising the new merge queue end-to-end — comments-only diff, no behavioral change. Co-authored-by: Isaac Signed-off-by: Vikrant Puppala <[email protected]>
msrathore-db
approved these changes
May 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Comments-only refresh of the
kernel-e2e.ymlheader. The old block read like a TODO list ("Required external setup: 1. … 2. … 3. …") but every item is now in place:kernel-e2elabel exists.databricks/databricks-sql-kernel.KERNEL_REVpins the kernel commit.azure-prodenv exposes the dogfood warehouse creds.Kernel E2Eis now a required status check onmain's ruleset.Rewords the block as a "what this depends on" debugging crib and adds the ruleset note so future readers understand why the
merge_grouppath matters.Why this PR
Also serving as the merge-queue pilot. The
mainruleset was just updated to requirePython Proxy Tests+Kernel E2Eand enable merge queue withALLGREENgrouping. This is a comments-only diff with no behavioral change — the lowest-risk way to exercise the new queue end-to-end.Expected behavior when added to the queue:
merge_groupevent fires.Kernel E2Eworkflow auto-passes (detect-changessees no kernel-relevant files changed).Python Proxy Testsworkflow auto-passes (no driver source changed).If anything in that chain hangs, the queue's
check_response_timeout_minutes: 60will release the PR after an hour rather than blocking forever.Test plan
CoPilotReviewruleset to remove the required-checks rule temporarily, debug, re-add.This pull request and its description were written by Isaac.