Skip to content

Conversation

@samuelkhtu
Copy link
Contributor

Description

This PR addresses #53333 by adding support for the account_host field in the ADLS get_fs() method. The current implementation misses this field, even though it's commonly needed for custom Azure endpoint configurations.

What is the purpose of this change?

To ensure users can configure account_host via Airflow's Azure connection extras, aligning the behavior with other fields like account_name and account_key.

Related Issue

Closes #53333

Changes

  • Added account_host to the fields list in airflow/providers/microsoft/azure/fs/adls.py.
  • Verified via unit tests that the account_host is picked up correctly.
  • Updated document

Testing

  • Ran pytest for affected provider files.
  • Verified pre-commit hooks pass after applying required changes.

Backport

Please consider backporting this fix to the v2-10-stable branch. This bug affects users on Airflow 2.10 who rely on custom ADLS account_host values for enterprise or private Azure environments, and there is no workaround without this patch.

Checklist

  • My PR addresses the linked issue
  • Unit tests pass locally with pytest
  • I ran pre-commit run --all-files and fixed all issues
  • My code follows Airflow's contributing guidelines

@shahar1 shahar1 merged commit ceee7a2 into apache:main Jul 19, 2025
75 checks passed
@samuelkhtu samuelkhtu deleted the fix/azureblob-custom-host-not-working branch August 1, 2025 06:44
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.

account_host missing in get_fs() in adls.py for AzureBlobFileSystem Body

2 participants