Skip to content

Clarify FAB and Auth documentation regarding external_db_managers - #69929

Merged
vincbeck merged 3 commits into
apache:mainfrom
stephsi:improve-auth-and-fab-docs
Jul 15, 2026
Merged

Clarify FAB and Auth documentation regarding external_db_managers#69929
vincbeck merged 3 commits into
apache:mainfrom
stephsi:improve-auth-and-fab-docs

Conversation

@stephsi

@stephsi stephsi commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

After upgrading Airflow, I had some issues regarding the FAB migration and started looking into the inner workings more closely. Through this, I discovered that some documentation around FAB and auth managers is a bit misleading and does not fully reflect what happens in the code itself.

This PR clarifies the related documentation by:

  • Replacing the incorrectly used [core] external_db_managers section with [database] external_db_managers.
  • Rewording misleading statements that suggested DB managers are added to the external_db_managers configuration, when they are actually loaded automatically at runtime.

Was generative AI tooling used to co-author this PR?
  • Yes - Codex (gpt-5.5)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

stephsi added 3 commits July 15, 2026 16:25
external_db_managers is part of the database section, not core
FABDBManager is not added to external_db_manager if it is specified
under auth_manager, it is just loaded by the RunDBManager
@vincbeck
vincbeck merged commit 14560e4 into apache:main Jul 15, 2026
70 checks passed
joshuabvarghese pushed a commit to joshuabvarghese/airflow that referenced this pull request Jul 16, 2026
…ache#69929)

This PR clarifies the related documentation by:

- Replacing the incorrectly used [core] external_db_managers section with [database] external_db_managers.
- Rewording misleading statements that suggested DB managers are added to the external_db_managers configuration, when they are actually loaded automatically at runtime.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants