Skip to content

Implement DB cleanup in Helm #58585

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
aeroyorch wants to merge 1 commit into from
Closed

Implement DB cleanup in Helm #58585

aeroyorch wants to merge 1 commit into from

Conversation

@aeroyorch
Copy link
Contributor

@aeroyorch aeroyorch commented Nov 22, 2025

closes: #32419

@boring-cyborg boring-cyborg bot added the area:helm-chart Airflow Helm Chart label Nov 22, 2025
@aeroyorch
Copy link
Contributor Author

aeroyorch commented Nov 22, 2025

tests and docs still pending

@aeroyorch aeroyorch force-pushed the dbcleanup-in-helm branch 5 times, most recently from cdec5c1 to 395d55e Compare November 23, 2025 02:34
@jscheffl
Copy link
Contributor

jscheffl commented Nov 23, 2025

If you say some stuff still pending, then putting the PR to "draft", please mark it "ready to review" once we should start reviewing it.

@jscheffl jscheffl marked this pull request as draft November 23, 2025 15:50
@jedcunningham
Copy link
Member

jedcunningham commented Nov 24, 2025

Thanks for the PR @aeroyorch. Unfortunately #58155 just added a cronjob for db clean - it's always worth checking if there is an existing open PR before starting on stuff. If you want to expand on that (e.g, supporting configuring tables), have at it.

@aeroyorch
Copy link
Contributor Author

aeroyorch commented Nov 24, 2025

Thanks for the PR @aeroyorch. Unfortunately #58155 just added a cronjob for db clean - it's always worth checking if there is an existing open PR before starting on stuff. If you want to expand on that (e.g, supporting configuring tables), have at it.

Oh... I actually checked if there was anything already done or open, but I only saw the discussion. In any case, we could probably reuse whatever was implemented for the tables, verbose, and the Skip archive part.

@aeroyorch aeroyorch deleted the dbcleanup-in-helm branch November 25, 2025 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dstandish dstandish Awaiting requested review from dstandish dstandish is a code owner

@jedcunningham jedcunningham Awaiting requested review from jedcunningham jedcunningham is a code owner

@hussein-awala hussein-awala Awaiting requested review from hussein-awala hussein-awala is a code owner

@jscheffl jscheffl Awaiting requested review from jscheffl jscheffl is a code owner

Assignees

No one assigned

Labels

area:helm-chart Airflow Helm Chart

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@aeroyorch @jscheffl @jedcunningham