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 2022/06/02 06:47:13 UTC

[GitHub] [airflow] mariasumedre-3pg commented on a diff in pull request #24054: Add param docs to KubernetesHook and KubernetesPodOperator (#23955)

mariasumedre-3pg commented on code in PR #24054:
URL: https://github.com/apache/airflow/pull/24054#discussion_r887615585


##########
airflow/providers/cncf/kubernetes/operators/kubernetes_pod.py:
##########
@@ -112,6 +112,12 @@ class KubernetesPodOperator(BaseOperator):
     :param labels: labels to apply to the Pod. (templated)
     :param startup_timeout_seconds: timeout in seconds to startup the pod.
     :param get_logs: get the stdout of the container as logs of the tasks.
+    :param env_from: (Optional) List of sources to populate environment variables in the container.
+        The keys defined within a source must be a C_IDENTIFIER.
+        All invalid keys will be reported as an event when the container is starting.
+        When a key exists in multiple sources, the value associated with the last source
+        will take precedence. Values defined by an Env with a duplicate key will take precedence.
+        Cannot be updated.

Review Comment:
   Done, also moved `env_from` right after `env_vars`. Thank you.



-- 
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: commits-unsubscribe@airflow.apache.org

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