seanghaeli commented on code in PR #75251: URL: https://github.com/apache/airflow/pull/75251#discussion_r4356828787
########## providers/cncf/kubernetes/src/airflow/providers/cncf/kubernetes/cli/kubernetes_command.py: ########## @@ -148,7 +149,8 @@ def _get_pod_completion_time(pod): @providers_configuration_loaded def cleanup_pods(args): Review Comment: Funny enough, this concern is mostly subsumed by #75255, which merged earlier today, so I'm reusing that work here, and this should now be addressed. `cleanup-pods --team` now resolves the team's executor the same way the scheduler does and builds its client through that executor's client factory, so a team on a Kubernetes executor cleans its own cluster. If a team has more than one Kubernetes executor, `--executor` picks which one, and it refuses to run when another team or the global executors use the same namespace. -- 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]
