Skip to content

Document all valid auth_type values in README#1121

Closed
chrisst wants to merge 7 commits into
databricks:mainfrom
chrisst:update-auth-type-docs
Closed

Document all valid auth_type values in README#1121
chrisst wants to merge 7 commits into
databricks:mainfrom
chrisst:update-auth-type-docs

Conversation

@chrisst

@chrisst chrisst commented Nov 20, 2025

Copy link
Copy Markdown

Summary

  • Document all 18 valid auth_type values for the SDK
  • Add a new Authentication Types Reference page listing each auth type with its required/optional parameters, environment variables, and a usage example, plus the default authentication priority order
  • Link to it from the README and docs/authentication.md auth_type rows, and add it to the docs toctree

Motivation

The documentation mentions the auth_type parameter but doesn't state which values are acceptable, which leads to confusion when configuring authentication explicitly.

Valid values documented: pat, basic, oauth-m2m, external-browser, databricks-cli, azure-client-secret, azure-cli, github-oidc, github-oidc-azure, azure-devops-oidc, google-credentials, google-id, metadata-service, runtime, runtime-oauth, model-serving, env-oidc, file-oidc.

Test plan

Documentation change only — no functional changes. Content verified against the credential strategies registered in databricks/sdk/credentials_provider.py and the config attributes in databricks/sdk/config.py as of the current main.

NO_CHANGELOG=true

🤖 Generated with Claude Code

Explicitly list all 18 valid auth_type values in the
"Additional authentication configuration options" section
to improve developer experience and reduce confusion about
which authentication types are supported.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

Please ensure that the NEXT_CHANGELOG.md file is updated with any relevant changes.
If this is not necessary for your PR, please include the following in your PR description:
NO_CHANGELOG=true
and rerun the job.

Added the same comprehensive list of valid auth_type values
to the docs/authentication.md file to ensure consistency
across all documentation.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Created a new detailed reference document (docs/auth-types-reference.md)
that provides:
- Complete table of all 18 auth types with descriptions
- Required and optional parameters for each auth type
- Environment variables for all parameters
- Usage examples for common auth types
- Authentication priority order
- Cross-references and notes

Also added links from README.md and docs/authentication.md to the
new reference guide for easy discoverability.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@chrisst chrisst marked this pull request as draft November 20, 2025 16:45
- Add auth_type parameter to all examples in auth-types-reference.md
- Clarify that explicit auth_type short-circuits automatic detection
- Update README.md and authentication.md to link to auth-types-reference
- Add auth-types-reference to docs/index.rst toctree for readthedocs
- Fix broken azure-ad.md link in README.md
- Add new examples: basic, azure-cli, github-oidc-azure, azure-devops-oidc,
  google-id, metadata-service, runtime-oauth, env-oidc, file-oidc, model-serving
@renaudhartert-db renaudhartert-db removed their request for review March 11, 2026 16:30
@chrisst chrisst temporarily deployed to test-trigger-is June 12, 2026 22:49 — with GitHub Actions Inactive
@github-actions

Copy link
Copy Markdown

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-py

Inputs:

  • PR number: 1121
  • Commit SHA: ab3f4dd2b7b467e2e5ca005a15878f4ad1f2d369

Checks will be approved automatically on success.

@chrisst

chrisst commented Jun 12, 2026

Copy link
Copy Markdown
Author

Closing this PR: this repository is now a read-only mirror, and changes are applied in the internal source repository instead (see CONTRIBUTING.md). This change has been re-applied internally and will ship to this mirror with an upcoming release.

@chrisst chrisst closed this Jun 12, 2026
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.

1 participant