Skip to content

ci: add DBR LTS install check to catch ES-1960554-class regressions #843

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
vikrantpuppala merged 4 commits into from
Jul 3, 2026
Merged

ci: add DBR LTS install check to catch ES-1960554-class regressions #843

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
14149d2
ci: add DBR LTS install check + fix pyarrow-compat it caught (ES-1960…
vikrantpuppala Jul 2, 2026
4cd5fda
test(e2e): authenticate e2e via OAuth M2M so staging tests match DATA…
vikrantpuppala Jul 3, 2026
4821823
test(e2e): add databricks-sdk dev dependency for OAuth M2M auth
vikrantpuppala Jul 3, 2026
e679214
Revert e2e OAuth M2M; use SP PAT instead (keep DBR LTS check on M2M)
vikrantpuppala Jul 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
test(e2e): add databricks-sdk dev dependency for OAuth M2M auth 
The e2e M2M auth (conftest.auth_connect_kwargs) imports
databricks.sdk.core.oauth_service_principal, but databricks-sdk was not a
project dependency, so `poetry install` in code-coverage.yml didn't provide
it -- every e2e connection failed with `ModuleNotFoundError: No module named
'databricks.sdk'`. Add it to the dev group (test-only; not a runtime dep of
the connector). CI's setup-poetry runs `poetry lock` before install, so the
lockfile is regenerated on the runner.

Co-authored-by: Isaac
Signed-off-by: Vikrant Puppala <vikrant.puppala@databricks.com>
  • Loading branch information
@vikrantpuppala
vikrantpuppala committed Jul 3, 2026
commit 48218237bf466bf6f9e5cc45a1b604f4d0091e33
7 changes: 7 additions & 0 deletions pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,13 @@ black = "^22.3.0"
pytest-dotenv = "^0.5.2"
pytest-cov = "^4.0.0"
pytest-xdist = "^3.0.0"
# Test-only: the e2e suite authenticates to the warehouse via OAuth M2M
# (service principal) using databricks.sdk's oauth_service_principal — see
# conftest.auth_connect_kwargs. The connector itself has no native
# Databricks-OAuth M2M path (its azure-sp-m2m targets Azure AD; its
# databricks-oauth path is interactive U2M), so the SDK provides the
# client-credentials flow against /oidc/v1/token. Not a runtime dependency.
databricks-sdk = ">=0.20"
numpy = [
{ version = ">=1.16.6", python = ">=3.8,<3.11" },
{ version = ">=1.23.4", python = ">=3.11" },
Expand Down
Loading