Skip to content

Commit cffa7fd

Browse files
eakmanrqclaude
andcommitted
Use OAuth for Databricks CLI, remove ACCESS_TOKEN
The integration tests use OAuth (client_id + client_secret), not PAT. Revert the CLI workaround and remove DATABRICKS_ACCESS_TOKEN so the CLI uses the same OAuth credentials. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: eakmanrq <6326532+eakmanrq@users.noreply.github.com>
1 parent cfba57e commit cffa7fd

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

.github/scripts/manage-test-db.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,11 +43,11 @@ databricks_init() {
4343
}
4444

4545
databricks_up() {
46-
DATABRICKS_CLIENT_ID= DATABRICKS_CLIENT_SECRET= DATABRICKS_TOKEN=$DATABRICKS_ACCESS_TOKEN databricks catalogs create $1 || true
46+
databricks catalogs create $1 || true
4747
}
4848

4949
databricks_down() {
50-
DATABRICKS_CLIENT_ID= DATABRICKS_CLIENT_SECRET= DATABRICKS_TOKEN=$DATABRICKS_ACCESS_TOKEN databricks catalogs delete $1 --force || true
50+
databricks catalogs delete $1 --force || true
5151
}
5252

5353
# Redshift

.github/workflows/pr.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -315,7 +315,6 @@ jobs:
315315
DATABRICKS_SERVER_HOSTNAME: ${{ secrets.DATABRICKS_SERVER_HOSTNAME }}
316316
DATABRICKS_HOST: ${{ secrets.DATABRICKS_SERVER_HOSTNAME }}
317317
DATABRICKS_HTTP_PATH: ${{ secrets.DATABRICKS_HTTP_PATH }}
318-
DATABRICKS_ACCESS_TOKEN: ${{ secrets.DATABRICKS_ACCESS_TOKEN }}
319318
DATABRICKS_CLIENT_ID: ${{ secrets.DATABRICKS_CLIENT_ID }}
320319
DATABRICKS_CLIENT_SECRET: ${{ secrets.DATABRICKS_CLIENT_SECRET }}
321320
DATABRICKS_CONNECT_VERSION: ${{ secrets.DATABRICKS_CONNECT_VERSION }}

0 commit comments

Comments
 (0)