Skip to content

Conversation

@jscheffl
Copy link
Contributor

As I was starting to clean further global variables in settings.py I noticed a couple of constants being defined here which are only used at one place in the codebase. In order to un-clutter the settings.py this PR proposes to remove some fields which in my view are just "lurking" around in the wrong place.

@potiuk
Copy link
Member

potiuk commented Dec 28, 2025

Love it - that might really help with getting rid of the circular dependencies we have :)

@jscheffl jscheffl force-pushed the bugfix/variable-cleanup-in-settings-py branch from b869fec to f548ecb Compare December 28, 2025 20:39
@jscheffl jscheffl added the full tests needed We need to run full set of tests for this PR to merge label Dec 28, 2025
@jscheffl jscheffl marked this pull request as ready for review December 28, 2025 21:47
@jscheffl jscheffl added the all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs label Dec 28, 2025
@jscheffl
Copy link
Contributor Author

Force restart CI

@jscheffl jscheffl closed this Dec 28, 2025
@jscheffl jscheffl reopened this Dec 28, 2025
@jscheffl jscheffl requested a review from potiuk December 28, 2025 22:55
@potiuk potiuk force-pushed the bugfix/variable-cleanup-in-settings-py branch from f548ecb to 7ba0abe Compare December 29, 2025 10:13
@jscheffl jscheffl merged commit b7e909f into apache:main Dec 29, 2025
209 checks passed
Subham-KRLX pushed a commit to Subham-KRLX/airflow that referenced this pull request Jan 2, 2026
* Cleanup of variables in settings.py

* Fix pytest mocks
stegololz pushed a commit to stegololz/airflow that referenced this pull request Jan 9, 2026
* Cleanup of variables in settings.py

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

Labels

all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs area:API Airflow's REST/HTTP API area:CLI area:DAG-processing area:db-migrations PRs with DB migration area:logging area:providers area:Scheduler including HA (high availability) scheduler area:serialization full tests needed We need to run full set of tests for this PR to merge kind:documentation provider:apache-spark provider:celery

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants