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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org