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

helm chart doesn't support securityContext for airflow-run-airflow-migration and dag-processor init container #34595

Closed
1 of 2 tasks
rahulgoyal2987 opened this issue Sep 25, 2023 · 3 comments · Fixed by #35593
Closed
1 of 2 tasks

helm chart doesn't support securityContext for airflow-run-airflow-migration and dag-processor init container #34595

rahulgoyal2987 opened this issue Sep 25, 2023 · 3 comments · Fixed by #35593
Labels
area:helm-chart Airflow Helm Chart good first issue kind:bug This is a clearly a bug

Comments

@rahulgoyal2987
Copy link
Contributor

rahulgoyal2987 commented Sep 25, 2023 •
edited

Official Helm Chart version

1.10.0 (latest released)

Apache Airflow version

2.6.3

Kubernetes Version

1.27

Helm Chart configuration

helm chart doesn't support securityContext for airflow-run-airflow-migration and dag-processor init container

Docker Image customizations

No response

What happened

No response

What you think should happen instead

No response

How to reproduce

helm chart doesn't support securityContext for airflow-run-airflow-migration and dag-processor init container.

Anything else

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct
@rahulgoyal2987 rahulgoyal2987 added area:helm-chart Airflow Helm Chart kind:bug This is a clearly a bug needs-triage label for new issues that we didn't triage yet labels Sep 25, 2023
@rahulgoyal2987 rahulgoyal2987 changed the title helm chart doesn't support security context for airflow-run-airflow-migration and dag-processor init container helm chart doesn't support securityContext for airflow-run-airflow-migration and dag-processor init container Sep 25, 2023
@jscheffl
Copy link
Contributor

As there was a parallel BugReport for security context in OpenShift in #34599 does this belong together?

As you started analyzing already, would you be able to provide an improvement as a PR?

@jscheffl jscheffl added good first issue and removed needs-triage label for new issues that we didn't triage yet labels Sep 25, 2023
@eladkal
Copy link
Contributor

eladkal commented Nov 2, 2023

As there was a parallel BugReport for security context in OpenShift in #34599 does this belong together?

It was closed as invalid
#34599 (comment)

@shohamy7 shohamy7 mentioned this issue Nov 12, 2023
Merged
@MPParsley
Copy link

Indeed, it was about redis and statsd but the problem with airflow-run-airflow-migrations remains.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:helm-chart Airflow Helm Chart good first issue kind:bug This is a clearly a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add support for adding securityContext in the initContainer of the da… shohamy7/airflow
4 participants
@MPParsley @rahulgoyal2987 @eladkal @jscheffl