Skip to content

Conversation

@Aakcht
Copy link
Contributor

@Aakcht Aakcht commented Dec 25, 2025

Added possibility to pass custom envs to database-cleanup cronjob as it is done for create-user-job. Custom envs can affect configuration and since databaseCleanup container runs airflow ... command, I think it would be useful to have custom envs in databaseCleanup container too.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@boring-cyborg boring-cyborg bot added the area:helm-chart Airflow Helm Chart label Dec 25, 2025
@Aakcht Aakcht force-pushed the custom_envs_database_cleanup branch from 0b51b8e to c896f86 Compare December 25, 2025 14:25
@Aakcht Aakcht force-pushed the custom_envs_database_cleanup branch from c896f86 to 27d451e Compare December 25, 2025 14:50
Copy link
Contributor

@jscheffl jscheffl left a comment

Choose a reason for hiding this comment

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

That makes sense for consistency, thanks for the extension!

@jscheffl jscheffl added this to the Airflow Helm Chart 1.19.0 milestone Dec 25, 2025
@jscheffl jscheffl merged commit 3314d54 into apache:main Dec 25, 2025
93 checks passed
@Aakcht Aakcht deleted the custom_envs_database_cleanup branch December 25, 2025 18:57
amoghrajesh pushed a commit to astronomer/airflow that referenced this pull request Dec 29, 2025
Subham-KRLX pushed a commit to Subham-KRLX/airflow that referenced this pull request Jan 2, 2026
stegololz pushed a commit to stegololz/airflow that referenced this pull request Jan 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:helm-chart Airflow Helm Chart

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants