reporting failed status if action result has error

elastic · Jan 26, 2023 · cea835f · cea835f
1 parent 7ff346d
commit cea835f
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/x-pack/plugins/fleet/common/types/models/agent.ts b/x-pack/plugins/fleet/common/types/models/agent.ts
@@ -161,7 +161,7 @@ export interface AgentDiagnostics {
   name: string;
   createTime: string;
   filePath: string;
-  status: 'READY' | 'AWAITING_UPLOAD' | 'DELETED' | 'IN_PROGRESS';
+  status: 'READY' | 'AWAITING_UPLOAD' | 'DELETED' | 'IN_PROGRESS' | 'FAILED';
   actionId: string;
 }
 

diff --git a/...lications/fleet/sections/agents/agent_details_page/components/agent_diagnostics/index.tsx b/...lications/fleet/sections/agents/agent_details_page/components/agent_diagnostics/index.tsx
@@ -132,7 +132,7 @@ export const AgentDiagnosticsTab: React.FunctionComponent<AgentDiagnosticsProps>
           </EuiText>
         ) : (
           <EuiText color="subdued">
-            <EuiToolTip content={`File status: ${currentItem?.status}`}>
+            <EuiToolTip content={`Diagnostics status: ${currentItem?.status}`}>
               <EuiIcon type="alert" color="red" />
             </EuiToolTip>
             &nbsp;

diff --git a/x-pack/plugins/fleet/server/services/agents/uploads.ts b/x-pack/plugins/fleet/server/services/agents/uploads.ts
@@ -72,7 +72,7 @@ export async function getAgentUploads(
     const result = {
       actionId: action.actionId,
       id: file?.id ?? action.actionId,
-      status: file?.Status ?? 'IN_PROGRESS',
+      status: file?.Status ?? (action.error ? 'FAILED' : 'IN_PROGRESS'),
       name: fileName,
       createTime: action.timestamp!,
       filePath,
@@ -86,7 +86,7 @@ export async function getAgentUploads(
 async function _getRequestDiagnosticsActions(
   esClient: ElasticsearchClient,
   agentId: string
-): Promise<Array<{ actionId: string; timestamp?: string; fileId?: string }>> {
+): Promise<Array<{ actionId: string; timestamp?: string; fileId?: string; error?: string }>> {
   const agentActionRes = await esClient.search<any>({
     index: AGENT_ACTIONS_INDEX,
     ignore_unavailable: true,
@@ -144,13 +144,15 @@ async function _getRequestDiagnosticsActions(
       actionId: hit._source?.action_id as string,
       timestamp: hit._source?.['@timestamp'],
       fileId: hit._source?.data?.file_id as string,
+      error: hit._source?.error,
     }));
     return agentActions.map((action) => {
       const actionResult = actionResults.find((result) => result.actionId === action.actionId);
       return {
         actionId: action.actionId,
         timestamp: actionResult?.timestamp ?? action.timestamp,
         fileId: actionResult?.fileId,
+        error: actionResult?.error,
       };
     });
   } catch (err) {