Skip to content

Conversation

@jonashartwig
Copy link
Contributor

@jonashartwig jonashartwig commented Nov 2, 2021

What issues does your PR fix?

What does your PR do?

  • Adds the airflow.kubernetesPodTemplate.extraContainers value to specify extra containers in the KubernetesExecutor pod_template.

Checklist

For all Pull Requests

For releasing ONLY

@jonashartwig jonashartwig changed the title #455 - added extra containers for pod template feat: #455 - added extra containers for pod template Nov 2, 2021
@jonashartwig jonashartwig reopened this Nov 2, 2021
Copy link
Member

@thesuperzapper thesuperzapper left a 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!

@thesuperzapper thesuperzapper changed the title feat: #455 - added extra containers for pod template feat: add extra containers to kubernetesPodTemplate Nov 11, 2021
@jonashartwig
Copy link
Contributor Author

jonashartwig commented Nov 12, 2021

Thank you for taking the time and giving feedback. I adjusted my PR to accompany your requests.

@thesuperzapper thesuperzapper changed the title feat: add extra containers to kubernetesPodTemplate feat: add airflow.kubernetesPodTemplate.extraContainers Dec 14, 2021
@thesuperzapper
Copy link
Member

@jonashartwig thanks for your work, I have added this to the 8.6.0 milestone.

@jonashartwig
Copy link
Contributor Author

Happy to hear :)

thesuperzapper
thesuperzapper previously approved these changes Feb 17, 2022
Copy link
Member

@thesuperzapper thesuperzapper left a 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.

@thesuperzapper thesuperzapper added the status/ready-to-merge status - this will be merged into next release label Mar 22, 2022
@thesuperzapper thesuperzapper merged commit 194834d into airflow-helm:main Apr 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/ready-to-merge status - this will be merged into next release

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Allow users to provide extra Containers in kubernetesPodTemplate

2 participants