-
Notifications
You must be signed in to change notification settings - Fork 207
197 lines (177 loc) · 8.38 KB
/
Copy pathintegration-kernel.yml
File metadata and controls
197 lines (177 loc) · 8.38 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
# Kernel Integration Tests for dbt-databricks.
#
# Runs the functional suite against the SQL-warehouse profile with the
# databricks-sql-connector Rust kernel backend (use_kernel=True) instead of
# the default Thrift backend, on a weekly Saturday schedule. Like
# integration-spog.yml this is a scheduled smoke, not a PR gate — PR-targeting
# and status-reporting are omitted.
#
# Why warehouse-only: the kernel routes over the SEA (Statement Execution API)
# HTTP transport, which is warehouse-only — the connector rejects an
# all-purpose-cluster http_path ("SEA only works for warehouses"). So the two
# cluster profiles from the main matrix cannot run on the kernel and are not
# included here.
#
# Auth: the kernel supports PAT and Databricks OAuth (M2M/U2M) and has no
# Azure-AD flow. The peco CI credential is an Azure service principal, which the
# kernel cannot authenticate, so tests/profiles.py resolves it to a workspace
# token and hands the kernel a PAT when DBT_DATABRICKS_USE_KERNEL=1 (see
# _resolve_azure_sp_to_token_for_kernel). The adapter's production path forwards
# only raw PAT/Databricks-OAuth creds — see SqlUtils.prepare_connection_arguments.
#
# Concurrency & sharding: unlike the other lanes (-n 10, 3 shards), this one runs
# -n 4 across 5 shards. The kernel evicts SEA sessions under sustained -n 10 load
# (late verification queries fail with "session ... not open"), and the Azure test
# token has a ~1h lifetime — finer shards keep each job well under that. Both are
# kernel-lane workarounds; revisit as the kernel matures. See the run step below.
name: Kernel Integration Tests
on:
workflow_dispatch:
schedule:
- cron: "17 9 * * 6" # Weekly: Saturday 09:17 UTC (14:47 IST). Clear of the
# daily 21:30 UTC nightly and the Sunday SPOG slot,
# which share the peco warehouse.
permissions:
id-token: write
contents: read
concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true
jobs:
prepare-shards:
runs-on:
group: databricks-protected-runner-group
labels: linux-ubuntu-latest
env:
UV_FROZEN: "1"
steps:
- name: Check out the repository
uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
- name: Setup JFrog PyPI Proxy
uses: ./.github/actions/setup-jfrog-pypi
- name: Set up Python
uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065 # v5
with:
python-version: "3.10"
- name: Install uv
uses: astral-sh/setup-uv@38f3f104447c67c051c4a08e39b64a148898af3a # v4
with:
cache-local-path: ~/.cache/uv
- name: Install Hatch
uses: pypa/hatch@257e27e51a6a5616ed08a39a408a21c35c9931bc # install
- name: Collect tests and assign to shards
run: |
set -euo pipefail
mkdir -p shard-assignments
hatch run pytest --collect-only -q --profile databricks_uc_sql_endpoint tests/functional 2>&1 \
| grep "::" \
> "shard-assignments/databricks_uc_sql_endpoint-collected.txt"
python3 scripts/shard_assign.py \
--profile databricks_uc_sql_endpoint \
--num-shards 5 \
--input "shard-assignments/databricks_uc_sql_endpoint-collected.txt" \
--output-dir shard-assignments \
--algo lpt_historical_time \
--timings .github/test_timings.json
- name: Upload shard assignments
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4
with:
name: shard-assignments-kernel
path: shard-assignments/
retention-days: 5
uc-sql-endpoint:
needs: prepare-shards
strategy:
fail-fast: false
matrix:
shard: [0, 1, 2, 3, 4]
runs-on:
group: databricks-protected-runner-group
labels: linux-ubuntu-latest
environment: azure-prod
env:
DBT_DATABRICKS_HOST_NAME: ${{ secrets.DATABRICKS_HOST }}
DBT_DATABRICKS_CLIENT_ID: ${{ secrets.TEST_PECO_SP_ID }}
DBT_DATABRICKS_CLIENT_SECRET: ${{ secrets.TEST_PECO_SP_SECRET }}
DBT_DATABRICKS_HTTP_PATH: ${{ secrets.TEST_PECO_WAREHOUSE_HTTP_PATH }}
DBT_DATABRICKS_UC_INITIAL_CATALOG: peco
DBT_DATABRICKS_LOCATION_ROOT: ${{ secrets.TEST_PECO_EXTERNAL_LOCATION }}test
# Wire the UC cluster id so build_cluster_http_path.py can populate
# DBT_DATABRICKS_UC_CLUSTER_HTTP_PATH. The kernel-incompatible tests that
# pin a model to that cluster path are @pytest.mark.skip_kernel and skip
# here; this keeps the env var resolvable so their fixtures still parse.
TEST_PECO_UC_CLUSTER_ID: ${{ secrets.TEST_PECO_UC_CLUSTER_ID }}
TEST_PECO_SPOG_HOST: ${{ secrets.TEST_PECO_SPOG_HOST }}
TEST_PECO_SPOG_WORKSPACE_ID: ${{ secrets.TEST_PECO_SPOG_WORKSPACE_ID }}
# Route the whole suite through the connector's Rust kernel (SEA) backend.
DBT_DATABRICKS_USE_KERNEL: "1"
UV_FROZEN: "1"
steps:
- name: Check out repository
uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
- name: Setup JFrog PyPI Proxy
uses: ./.github/actions/setup-jfrog-pypi
- name: Set up python
uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065 # v5
with:
python-version: "3.10"
- name: Get http path from environment
run: python .github/workflows/build_cluster_http_path.py
shell: sh
- name: Install uv
uses: astral-sh/setup-uv@38f3f104447c67c051c4a08e39b64a148898af3a # v4
with:
cache-local-path: ~/.cache/uv
- name: Install Hatch
uses: pypa/hatch@257e27e51a6a5616ed08a39a408a21c35c9931bc # install
- name: Install the connector kernel extra
# The kernel backend ships in the optional databricks-sql-connector
# [kernel] extra (a PyO3 wheel), which the frozen lock does not include.
# Add it into the hatch env so use_kernel=True can import the kernel.
run: |
set -euo pipefail
HATCH_PY="$(hatch run python -c 'import sys; print(sys.executable)')"
hatch run python -c "import databricks.sql as s; print('connector version:', s.__version__)"
uv pip install --python "$HATCH_PY" "databricks-sql-connector[kernel]"
hatch run python -c "import databricks_sql_kernel; print('kernel extra installed')"
- name: Download shard assignments
uses: actions/download-artifact@65a9edc5881444af0b9093a5e628f2fe47ea3b2e # v4.1.7
with:
name: shard-assignments-kernel
path: shard-assignments/
- name: Resolve test list for this shard
run: |
set -euo pipefail
SHARD_FILE="shard-assignments/databricks_uc_sql_endpoint-shard-${{ matrix.shard }}.txt"
if [ ! -s "$SHARD_FILE" ]; then
echo "::error::Shard file missing or empty: $SHARD_FILE"
exit 1
fi
echo "SHARD_TESTS_FILE=$SHARD_FILE" >> "$GITHUB_ENV"
echo "Files in shard ${{ matrix.shard }}: $(wc -l < "$SHARD_FILE")"
- name: Run Sql Endpoint Functional Tests on kernel (shard ${{ matrix.shard }})
# Runs at -n 4 rather than the -n 10 the other lanes use: under sustained
# -n 10 the kernel's SEA sessions get evicted server-side, so a test's
# post-build verification query intermittently fails with "Cannot
# execute_command without an open session". -n 4 keeps the kernel session
# count low enough to avoid that. This lane is also sharded more finely
# (5 shards, above) so no shard runs long enough to cross an auth token's
# ~1h lifetime. Revisit once the kernel handles session lifecycle under
# higher concurrency.
run: |
mkdir -p logs
DBT_TEST_USER=notnecessaryformosttests@example.com \
xargs -r hatch -v run pytest \
--color=yes -v \
--profile databricks_uc_sql_endpoint \
-n 4 --dist=loadfile \
--reruns 2 --reruns-delay 120 \
--junitxml=logs/junit-kernel-sql-endpoint-shard-${{ matrix.shard }}.xml \
< "$SHARD_TESTS_FILE"
- name: Upload Kernel SQL Endpoint Test Logs
if: always()
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4
with:
name: kernel-sql-endpoint-logs-shard-${{ matrix.shard }}
path: logs/
retention-days: 14