amoghrajesh commented on code in PR #73030:
URL: https://github.com/apache/airflow/pull/73030#discussion_r4060513375
##########
airflow-core/src/airflow/ui/src/pages/TaskInstance/Details.tsx:
##########
@@ -131,6 +131,16 @@ export const Details = () => {
return (
<Box p={2}>
+ {taskInstance?.retry_reason === null || taskInstance?.retry_reason ===
undefined ? undefined : (
Review Comment:
Gated the banner on failed/up_for_retry, so a cleared task no longer shows a
reason describing the previous attempt.
Leaving the column itself alone here and doing it as a separate PR, because
the staleness predates both of these PRs and is wider than it first looks:
clear_task_instances leaves retry_delay_override behind too, and that one is
functional rather than cosmetic, next_retry_datetime reads it in preference to
the task's configured retry_delay, so a policy supplied delay from an earlier
run silently controls the timing of the next retry after a clear, even if the
policy has since changed or been removed. That seems worth its own change and
its own test against a function this heavily used, rather than riding along in
a UI PR.
--
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]
