Re: [PR] Fix Scheduler restarting due to too many completed pods in cluster [airflow]

2024-06-13 Thread via GitHub
ephraimbuddy merged PR #40183: URL: https://github.com/apache/airflow/pull/40183 -- 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:

Re: [PR] Fix Scheduler restarting due to too many completed pods in cluster [airflow]

2024-06-13 Thread via GitHub
jedcunningham commented on code in PR #40183: URL: https://github.com/apache/airflow/pull/40183#discussion_r1638413499 ## airflow/providers/cncf/kubernetes/executors/kubernetes_executor_utils.py: ## @@ -271,14 +273,9 @@ def process_status( elif status == "Succeeded":

Re: [PR] Fix Scheduler restarting due to too many completed pods in cluster [airflow]

2024-06-13 Thread via GitHub
ephraimbuddy commented on code in PR #40183: URL: https://github.com/apache/airflow/pull/40183#discussion_r1637720232 ## airflow/providers/cncf/kubernetes/executors/kubernetes_executor_utils.py: ## @@ -269,21 +271,6 @@ def process_status( (pod_name, namespace,

Re: [PR] Fix Scheduler restarting due to too many completed pods in cluster [airflow]

2024-06-13 Thread via GitHub
ephraimbuddy commented on code in PR #40183: URL: https://github.com/apache/airflow/pull/40183#discussion_r1637718476 ## airflow/providers/cncf/kubernetes/executors/kubernetes_executor_utils.py: ## @@ -139,7 +139,9 @@ def _run( ) -> str | None:

Re: [PR] Fix Scheduler restarting due to too many completed pods in cluster [airflow]

2024-06-12 Thread via GitHub
jedcunningham commented on code in PR #40183: URL: https://github.com/apache/airflow/pull/40183#discussion_r1636830772 ## airflow/providers/cncf/kubernetes/executors/kubernetes_executor_utils.py: ## @@ -269,21 +271,6 @@ def process_status( (pod_name, namespace,

Re: [PR] Fix Scheduler restarting due to too many completed pods in cluster [airflow]

2024-06-12 Thread via GitHub
jedcunningham commented on code in PR #40183: URL: https://github.com/apache/airflow/pull/40183#discussion_r1636824574 ## airflow/providers/cncf/kubernetes/pod_generator.py: ## @@ -56,6 +56,16 @@ from airflow.utils.hashlib_wrapper import md5 from airflow.version import

Re: [PR] Fix Scheduler restarting due to too many completed pods in cluster [airflow]

2024-06-12 Thread via GitHub
jedcunningham commented on code in PR #40183: URL: https://github.com/apache/airflow/pull/40183#discussion_r1636470973 ## airflow/providers/cncf/kubernetes/executors/kubernetes_executor_utils.py: ## @@ -139,7 +139,9 @@ def _run( ) -> str | None: