Skip to content
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

Add startupProbe to airflow helm charts #33099

Closed
2 tasks done
amoghrajesh opened this issue Aug 4, 2023 · 1 comment · Fixed by #33107
Closed
2 tasks done

Add startupProbe to airflow helm charts #33099

amoghrajesh opened this issue Aug 4, 2023 · 1 comment · Fixed by #33107
Assignees
@amoghrajesh
Labels
area:helm-chart Airflow Helm Chart kind:feature Feature Requests

Comments

@amoghrajesh
Copy link
Contributor

Description

Introducing a startupProbe onto the airflow services would be useful for slow starting container and most of all it doesn't have side effects.

Use case/motivation

We have an internal feature where we perform a copy of venv from airflow services to cloud storages which can sometimes take a few minutes. Copying of a venv is a metadata heavy load: https://learn.microsoft.com/en-us/troubleshoot/azure/azure-storage/files-troubleshoot-performance?tabs=linux#cause-2-metadata-or-namespace-heavy-workload.

Introducing a startupProbe onto the airflow services would be useful for slow starting container and most of all it doesn't have side effects.

Related issues

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct
@amoghrajesh amoghrajesh added kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet area:helm-chart Airflow Helm Chart and removed needs-triage label for new issues that we didn't triage yet labels Aug 4, 2023
@amoghrajesh
Copy link
Contributor Author

Self assigning this as I will work on this soon

@amoghrajesh amoghrajesh self-assigned this Aug 4, 2023
@amoghrajesh amoghrajesh mentioned this issue Aug 4, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@amoghrajesh amoghrajesh

Labels
area:helm-chart Airflow Helm Chart kind:feature Feature Requests
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adding startupProbe to airflow services charts amoghrajesh/airflow
1 participant
@amoghrajesh