Re: [PR] retry for failure in the queue [airflow]

2024-08-05 Thread via GitHub
potiuk commented on PR #39398: URL: https://github.com/apache/airflow/pull/39398#issuecomment-2268583249 I am not particularly experienced in kubernetes APIS there :( - would have to take a deeper look - but maybe others (@RNHTTR ? @Taragolis can help)? -- This is an automated message fr

Re: [PR] retry for failure in the queue [airflow]

2024-08-04 Thread via GitHub
Bowrna commented on PR #39398: URL: https://github.com/apache/airflow/pull/39398#issuecomment-2268244502 > > I have been working on this PR after taking a long break due to personal commitments. I have been struck in this PR. From whatever I have learnt so far, I can see that. > > >

Re: [PR] retry for failure in the queue [airflow]

2024-07-19 Thread via GitHub
RNHTTR commented on code in PR #39398: URL: https://github.com/apache/airflow/pull/39398#discussion_r1684729030 ## airflow/settings.py: ## @@ -98,6 +98,7 @@ STATE_COLORS = { "deferred": "mediumpurple", "failed": "red", +"failed_in_queue": "reddishorange", Review

Re: [PR] retry for failure in the queue [airflow]

2024-07-16 Thread via GitHub
collinmcnulty commented on PR #39398: URL: https://github.com/apache/airflow/pull/39398#issuecomment-2231761533 @RNHTTR Any insights here? -- 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 spec

Re: [PR] retry for failure in the queue [airflow]

2024-07-15 Thread via GitHub
Bowrna commented on PR #39398: URL: https://github.com/apache/airflow/pull/39398#issuecomment-2228569427 > I have been working on this PR after taking a long break due to personal commitments. I have been struck in this PR. From whatever I have learnt so far, I can see that. > > 1. t

Re: [PR] retry for failure in the queue [airflow]

2024-07-15 Thread via GitHub
Bowrna commented on PR #39398: URL: https://github.com/apache/airflow/pull/39398#issuecomment-2227829264 I have been working on this PR after taking a long break due to personal commitments. I have been struck in this PR. From whatever I have learnt so far, I can see that. 1. there is a

Re: [PR] retry for failure in the queue [airflow]

2024-06-01 Thread via GitHub
Bowrna commented on code in PR #39398: URL: https://github.com/apache/airflow/pull/39398#discussion_r1623177973 ## airflow/providers/celery/executors/celery_executor.py: ## @@ -469,7 +469,7 @@ def cleanup_stuck_queued_tasks(self, tis: list[TaskInstance]) -> list[str]:

Re: [PR] retry for failure in the queue [airflow]

2024-05-10 Thread via GitHub
Bowrna commented on code in PR #39398: URL: https://github.com/apache/airflow/pull/39398#discussion_r1596877477 ## airflow/providers/celery/executors/celery_executor.py: ## @@ -469,7 +469,7 @@ def cleanup_stuck_queued_tasks(self, tis: list[TaskInstance]) -> list[str]:

Re: [PR] retry for failure in the queue [airflow]

2024-05-04 Thread via GitHub
Taragolis commented on code in PR #39398: URL: https://github.com/apache/airflow/pull/39398#discussion_r1590159486 ## airflow/providers/celery/executors/celery_executor.py: ## @@ -469,7 +469,7 @@ def cleanup_stuck_queued_tasks(self, tis: list[TaskInstance]) -> list[str]:

[PR] retry for failure in the queue [airflow]

2024-05-03 Thread via GitHub
Bowrna opened a new pull request, #39398: URL: https://github.com/apache/airflow/pull/39398 related: #38304 I think to have a retry option specifically for the failure happening in queue, we need to have a separate state where we need to indicate that failure is happening due to lon