fix: get logs from artifact when workflow deleted instead of archived. …

…Fixes: #14083 (#14087)

Signed-off-by: shuangkun <[email protected]>

ui/src/shared/services/workflows-service.ts

@@ -4,7 +4,7 @@ import {catchError, filter, map, mergeMap, switchMap} from 'rxjs/operators';
 import {NameFilterKeys} from '../../workflows/components/workflow-filters/workflow-filters';
 import {uiUrl} from '../base';
 import * as models from '../models';
-import {Event, LogEntry, NodeStatus, Workflow, WorkflowList, WorkflowPhase} from '../models';
+import {Event, isWorkflowInCluster, LogEntry, NodeStatus, Workflow, WorkflowList, WorkflowPhase} from '../models';
 import {ResubmitOpts, RetryOpts} from '../models';
 import {SubmitOpts} from '../models/submit-opts';
 import {Pagination} from '../pagination';
@@ -253,9 +253,8 @@ export const WorkflowsService = {
         const getLogsFromArtifact = () => this.getContainerLogsFromArtifact(workflow, nodeId, container, grep, archived);
         const getLogsFromCluster = () => this.getContainerLogsFromCluster(workflow, podName, container, grep);
 
-        // If our workflow is archived, don't even bother inspecting the cluster for logs since it's likely
-        // that the Workflow and associated pods have been deleted
-        if (archived) {
+        // If our workflow was deleted, try to get logs from artifacts.
+        if (!isWorkflowInCluster(workflow)) {
             return getLogsFromArtifact();
         }