You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/01/20 09:04:52 UTC

[GitHub] [airflow] potiuk commented on pull request #13735: Support extraContainers configuration in Helm Chart

potiuk commented on pull request #13735:
URL: https://github.com/apache/airflow/pull/13735#issuecomment-763451530


   > @Swalloow Thanks for the doc update.
   > 
   > From what I can see so far, I'm not sure yet if this implementation itself is flexible enough or sufficient for most side car use cases. I would like to have our K8S guru @dimberman to take a look before we proceed.
   
   I approved it from my part - it looks great witth the docs and unit tests, but @dimberman comments here are needed (at lest for now - I have a feeling that we should not get into the situation we have in CI now where I am mostly Single Point of Failure! I hope at today's call we can talk about our approach to make helm chart relasable.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org