csm10495 commented on code in PR #28808:
URL: https://github.com/apache/airflow/pull/28808#discussion_r1065007142
##########
airflow/providers/cncf/kubernetes/operators/kubernetes_pod.py:
##########
@@ -206,9 +206,10 @@ class KubernetesPodOperator(BaseOperator):
to populate the environment variables with. The contents of the target
ConfigMap's Data field will represent the key-value pairs as
environment variables.
Extends env_from.
+ :param base_container_name: The name of the base container in the pod.
This container's logs
Review Comment:
I don't think the concept of 'base container' is mentioned anywhere already.
It seems to just be the name given to the first container.. and is also the one
that can be logged via get_logs=True. That info is now in this docstr. Anywhere
else where we should mention this?
--
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]
