dstandish commented on a change in pull request #19339:
URL: https://github.com/apache/airflow/pull/19339#discussion_r739753871
##########
File path: docs/apache-airflow/executor/kubernetes.rst
##########
@@ -21,22 +21,22 @@
Kubernetes Executor
===================
-The kubernetes executor is introduced in Apache Airflow 1.10.0. The Kubernetes
executor will create a new pod for every task instance.
+The Kubernetes executor runs each task instance in its own pod on a Kubernetes
cluster.
-Example kubernetes files are available at
``scripts/in_container/kubernetes/app/{secrets,volumes,postgres}.yaml`` in the
source distribution (please note that these examples are not ideal for
production environments).
-The volumes are optional and depend on your configuration. There are two
volumes available:
+Use of persistent volumes is optional and depends on your configuration. There
are two types of volumes you may configure:
- **Dags**:
- - By storing dags onto persistent disk, it will be made available to all
workers
+ - By storing dags on a persistent volume, it will be made available to all
workers
Review comment:
it's a little unclear to me what this line is actually meant to help the
user understand. in order to use a PV, wouldn't you also need to have set up
git sync, so that you actually get dags onto the volume? but here it's sort of
represented as an alternative to git sync?
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
