-
Notifications
You must be signed in to change notification settings - Fork 474
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.kubernetesPodTemplate.extraInitContainers
#446
Conversation
68a2c81
to
2ef8cf8
Compare
Hey @thesuperzapper @gsemet , is there anything else I need to do to get this merged? |
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.
@jmcgrath207 thanks for the PR, I have left a small fix that is needed before we can merge.
kubernetesPodTemplate.extraInitContainers
kubernetesPodTemplate.extraInitContainers
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.
changes complete. Waiting for approval
@jmcgrath207 thanks for your work! I will put this in the 8.6.0 milestone (as it adds a new value). |
airflow.kubernetesPodTemplate.extraInitContainers
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.
@jmcgrath207 I just realized you were not correctly referencing the value you created.
The value is airflow.kubernetesPodTemplate.extraInitContainers
, not kubernetesPodTemplate.extraInitContainers
.
@jmcgrath207 do you still have time to apply the changes needed to address comments in #446 (review)? |
Signed-off-by: John McGrath <john.mcgrath207@gmail.com>
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.
@jmcgrath207 I have gone ahead and fixed your PR, so it can be merged for 8.6.0.
What issues does your PR fix?
N/A
What does your PR do?
airflow.kubernetesPodTemplate.extraInitContainers
value to specify extra init-containers in the KubernetesExecutor pod_template.Checklist
For all Pull Requests
For releasing ONLY