Skip to content

Conversation

@varundeepsaini
Copy link
Contributor

Closes #4131

Changes

Pass authentication environment variables to the Python mutator subprocess, allowing WorkspaceClient() to function without explicit authentication configuration.

Why

Fixes issue where Python mutators couldn't use WorkspaceClient() without manually passing auth arguments. The CLI now passes DATABRICKS_HOST, DATABRICKS_TOKEN, and other environment variables to the Python subprocess.

Tests

  • Unit test verifying auth env vars are passed to subprocess
  • Acceptance test (bundle/python/workspace-client-auth) that creates a WorkspaceClient() and calls current_user.me()

@varundeepsaini
Copy link
Contributor Author

@denik @kanterov could one of you take a look and merge if it looks good?

@denik denik requested a review from kanterov December 30, 2025 15:13
Copy link
Contributor

@shreyas-goenka shreyas-goenka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Blocking merge until we use auth.ProcessEnv.

@varundeepsaini varundeepsaini force-pushed the pass-auth-to-python-mutator branch from b5aad29 to 5216e5c Compare January 8, 2026 16:12
@github-actions
Copy link

github-actions bot commented Jan 8, 2026

An authorized user can trigger integration tests manually by following the instructions below:

Trigger:
go/deco-tests-run/cli

Inputs:

  • PR number: 4172
  • Commit SHA: 5216e5c8cdc9cb681edf22cf8adb88ee15562bc7

Checks will be approved automatically on success.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pass AuthContext to Python Mutator Functions / Create WorkspaceClient without additional config

3 participants