-
Notifications
You must be signed in to change notification settings - Fork 495
feat: add airflow.kubernetesPodTemplate.extraContainers
#456
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
Conversation
thesuperzapper
left a comment
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.
@jonashartwig thanks for the PR!
I have left a few small comments so we can get this merged!
Signed-off-by: cvv556 <[email protected]>
|
Thank you for taking the time and giving feedback. I adjusted my PR to accompany your requests. |
airflow.kubernetesPodTemplate.extraContainers
|
@jonashartwig thanks for your work, I have added this to the |
|
Happy to hear :) |
thesuperzapper
left a comment
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.
This is ready for the 8.6.0 release, and will be merged before we cut it.
What issues does your PR fix?
What does your PR do?
airflow.kubernetesPodTemplate.extraContainersvalue to specify extra containers in the KubernetesExecutor pod_template.Checklist
For all Pull Requests
For releasing ONLY