Skip to content

Commit aeb655c

Browse files
docs(kernel-e2e): refresh workflow header — setup is done, queue is live (#809)
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 <vikrant.puppala@databricks.com>
1 parent 34d4233 commit aeb655c

1 file changed

Lines changed: 11 additions & 12 deletions

File tree

.github/workflows/kernel-e2e.yml

Lines changed: 11 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -20,18 +20,17 @@ name: Kernel E2E Tests
2020
# - merge_group fires the real gate — dispatches when kernel-relevant
2121
# files changed, auto-passes otherwise.
2222
#
23-
# Required external setup:
24-
# 1. `kernel-e2e` label exists in this repo.
25-
# 2. `INTEGRATION_TEST_APP_ID` / `INTEGRATION_TEST_PRIVATE_KEY`
26-
# secrets exist (already installed for the proxy-tests workflow).
27-
# The GitHub App's repo allowlist must include
28-
# `databricks/databricks-sql-kernel` — extend the existing App
29-
# config; do not create a new App.
30-
# 3. `KERNEL_REV` file at the repo root containing a 40-char kernel
31-
# commit SHA.
32-
# 4. `azure-prod` environment exposes DATABRICKS_HOST /
33-
# TEST_PECO_WAREHOUSE_HTTP_PATH / DATABRICKS_TOKEN
34-
# (already configured for code-coverage.yml).
23+
# External setup that this workflow depends on (already in place,
24+
# documented here for future debugging):
25+
# - `kernel-e2e` label exists in this repo.
26+
# - `INTEGRATION_TEST_APP_ID` / `INTEGRATION_TEST_PRIVATE_KEY`
27+
# secrets are installed and the App's repo allowlist includes
28+
# `databricks/databricks-sql-kernel`.
29+
# - `KERNEL_REV` file at the repo root pins the kernel commit SHA.
30+
# - `azure-prod` environment exposes DATABRICKS_HOST /
31+
# TEST_PECO_WAREHOUSE_HTTP_PATH / DATABRICKS_TOKEN.
32+
# - `Kernel E2E` is listed as a required status check on the
33+
# `main` ruleset, so merge queue waits for it.
3534

3635
on:
3736
pull_request:

0 commit comments

Comments
 (0)