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 07:37:48 UTC

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

Swalloow commented on a change in pull request #13735:
URL: https://github.com/apache/airflow/pull/13735#discussion_r560734481



##########
File path: chart/README.md
##########
@@ -366,6 +366,17 @@ helm install airflow . \
     --set data.brokerUrl=redis://redis-user:password@redis-host:6379/0
 ```
 
+## Using additional containers
+
+If you are using your own sidecar container, you can add it through the `extraContainers` value. You can define different containers for scheduler, webserver and worker pods. For example, a sidecar that syncs DAGs from object storage.
+
+```bash

Review comment:
       @gardnerdev `extraContainers` are defined as properties of scheduler, webserver, and worker.
   
   ```
   scheduler:
     ...
     extraContainers: []
   
   webserver:
     ...
     extraContainers: []
   
   workers:
     ...
     extraContainers: []
   ```




----------------------------------------------------------------
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