-
Notifications
You must be signed in to change notification settings - Fork 476
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
feat: add airflow.defaultContainerSecurityContext
value
#624
Conversation
Signed-off-by: Philipp Hitzler <[email protected]>
Signed-off-by: Philipp Hitzler <[email protected]>
This issue has been automatically marked as stale because it has not had activity in 60 days. Thank you for your contributions. Issues never become stale if any of the following is true:
|
seems like a good feature to use, will this approved? |
Signed-off-by: Mathew Wicks <[email protected]>
airflow.defaultContainerSecurityContext
value
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zartstrom sorry about the extreme delay on this, but thanks for the PR.
I have pushed a small cleanup in abe66e4.
But its otherwise good to go, and should be in version 8.9.0 of the chart!
Signed-off-by: Philipp Hitzler [email protected]
What issues does your PR fix?
airflow.defaultContainerSecurityContext
value #578 (except for Postgres and Redis)What does your PR do?
This PR adds the following values:
airflow.defaultContainerSecurityContext
(default:{}
)runAsUser
andrunAsGroup
(those are set per-image with the*.image.uid
and*.image.gid
values)stable/postgres
chart #303stable/redis
chart #304@thesuperzapper all code is taken from your #578 (comment)
Checklist
For all Pull Requests
For releasing ONLY