jedcunningham commented on code in PR #23685: URL: https://github.com/apache/airflow/pull/23685#discussion_r871808287
########## docs/apache-airflow/executor/kubernetes.rst: ########## @@ -214,7 +214,9 @@ To get task logs out of the workers, you can: Comparison with CeleryExecutor ------------------------------ -In contrast to CeleryExecutor, KubernetesExecutor does not require additional components such as Redis and Flower, but does require access to Kubernetes cluster. +In contrast to CeleryExecutor, KubernetesExecutor does not require additional components such as Redis, but does require access to Kubernetes cluster. + +Also monitoring the Pods can be done with the built-in Kubernetes monitoring. For Celery you might want to use tools like Flower to monitor running Celery workers. Review Comment: ```suggestion Also monitoring the Pods can be done with the built-in Kubernetes monitoring. ``` Probably better to add that to the celery page instead? Seems out of place here to me. Maybe even bring this into the paragraph above? -- 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]
