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.

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

Use an init process as container entrypoint #176

Open
Kytha opened this issue Nov 15, 2024 · 0 comments
Open

Use an init process as container entrypoint #176

Kytha opened this issue Nov 15, 2024 · 0 comments

Comments

@Kytha
Copy link
Contributor

Kytha commented Nov 15, 2024

Overview

Our image runs the entry point script directly as the container entry point. This is against docker best practices because it omits the use of an init process. This will mean that no zombie processes will get properly reaped and will accumulate until system hits process limit. This is a problematic issue for a software like airflow, that dynamically creates subprocesses regularly.

Instead, our image should follow airflow and docker best practices and make use of a lightweight init process like dumb-init or tini.

This issue may be the reason why we sometimes see issues related to delays in worker task shutdown, or schedulers stalling after long lifetimes.

Acceptance Criteria

  • Integrate a lightweight init process to all airflow images

Additional Info

Also see related discussion in github airflow issue: apache/airflow#25225

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant