diff --git a/public/app/features/provisioning/Job/FinishedJobStatus.tsx b/public/app/features/provisioning/Job/FinishedJobStatus.tsx index 60b5ce73917..c8b4dd037fa 100644 --- a/public/app/features/provisioning/Job/FinishedJobStatus.tsx +++ b/public/app/features/provisioning/Job/FinishedJobStatus.tsx @@ -36,6 +36,20 @@ export function FinishedJobStatus({ jobUid, repositoryName, jobType, onStatusCha }, 1000); } + if (retryFailed) { + onStatusChange?.({ + status: 'error', + error: { + title: t('provisioning.job-status.no-job-found', 'No job found'), + message: t( + 'provisioning.job-status.no-job-found-message', + 'The job may have been deleted or could not be retrieved. Cancel the current process and start again.' + ), + }, + }); + return; + } + if (finishedQuery.isSuccess && job?.status) { const { state, message, errors } = job.status; @@ -70,21 +84,7 @@ export function FinishedJobStatus({ jobUid, repositoryName, jobType, onStatusCha clearTimeout(timeoutId); } }; - }, [finishedQuery, job, onStatusChange]); - - if (retryFailed) { - onStatusChange?.({ - status: 'error', - error: { - title: t('provisioning.job-status.no-job-found', 'No job found'), - message: t( - 'provisioning.job-status.no-job-found-message', - 'The job may have been deleted or could not be retrieved. Cancel the current process and start again.' - ), - }, - }); - return null; - } + }, [finishedQuery, job, onStatusChange, retryFailed]); if (!job || finishedQuery.isLoading || finishedQuery.isFetching) { return ( diff --git a/public/app/features/provisioning/Job/JobContent.tsx b/public/app/features/provisioning/Job/JobContent.tsx index 9b58c2f8143..8234a1b4378 100644 --- a/public/app/features/provisioning/Job/JobContent.tsx +++ b/public/app/features/provisioning/Job/JobContent.tsx @@ -76,15 +76,13 @@ export function JobContent({ jobType, job, isFinishedJob = false, onStatusChange {['working', 'pending'].includes(state ?? '') && ( - - - - {message ?? state ?? t('provisioning.job-status.starting', 'Starting...')} - - - )} - {state && !['success', 'error'].includes(state) && ( - + + + + + {message ?? state ?? t('provisioning.job-status.starting', 'Starting...')} + + )}