Bug 1995932 - Add detailed log parsing status display in job detail panel

ui/job-view/details/summary/LogUrls.jsx

@@ -10,7 +10,7 @@ import LogItem from './LogItem';
 export default function LogUrls(props) {
   const { logUrls, logViewerUrl, logViewerFullUrl } = props;
   const logUrlsUseful = logUrls.filter(
-    (logUrl) => logUrl.name !== 'perfherder-data-resource-usage.json',
+    (logUrl) => !logUrl.name.includes('perfherder-data'),
   );
 
   return (

ui/job-view/details/summary/SummaryPanel.jsx

@@ -26,12 +26,30 @@ class SummaryPanel extends React.PureComponent {
       classificationMap,
     } = this.props;
 
+    const logs = jobLogUrls.filter(
+      (log) => !log.name.includes('perfherder-data'),
+    );
+    const artifacts = jobLogUrls.filter((artifact) =>
+      artifact.name.includes('perfherder-data'),
+    );
     const logStatus = [
       {
         title: 'Log parsing status',
-        value: !jobLogUrls.length
+        value: !logs.length
           ? 'No logs'
-          : jobLogUrls.map((log) => log.parse_status).join(', '),
+          : logs.map((log) => log.parse_status).join(', '),
+      },
+    ];
+    const artifactStatus = [
+      {
+        title: 'Artifact parsing status',
+        value: !artifacts.length ? 'No artifacts' : null,
+        subfields: artifacts.length
+          ? artifacts.map((artifact) => ({
+              name: artifact.name,
+              value: artifact.parse_status,
+            }))
+          : null,
       },
     ];
 
@@ -50,7 +68,7 @@ class SummaryPanel extends React.PureComponent {
           jobDetails={jobDetails}
           logViewerUrl={logViewerUrl}
           logViewerFullUrl={logViewerFullUrl}
-          jobLogUrls={jobLogUrls}
+          jobLogUrls={logs}
           user={user}
         />
         <div id="summary-panel-content">
@@ -80,7 +98,7 @@ class SummaryPanel extends React.PureComponent {
             <StatusPanel selectedJobFull={selectedJobFull} />
             <JobInfo
               job={selectedJobFull}
-              extraFields={logStatus}
+              extraFields={[...logStatus, ...artifactStatus]}
               currentRepo={currentRepo}
             />
           </ul>

ui/shared/JobInfo.jsx

@@ -39,6 +39,35 @@ const getTimeFields = function getTimeFields(job) {
 };
 
 export default class JobInfo extends React.PureComponent {
+  renderFieldValue(field) {
+    if (field.url) {
+      return (
+        <a
+          title={field.value}
+          href={field.url}
+          target="_blank"
+          rel="noopener noreferrer"
+        >
+          {field.value}
+        </a>
+      );
+    }
+
+    if (field.subfields) {
+      return (
+        <ul className="list-unstyled ml-1">
+          {field.subfields.map((sf) => (
+            <li key={sf.name}>
+              {sf.name}: <span>{sf.value}</span>
+            </li>
+          ))}
+        </ul>
+      );
+    }
+
+    return <span>{field.value}</span>;
+  }
+
   render() {
     const { job, extraFields, showJobFilters, currentRepo } = this.props;
     const {
@@ -107,20 +136,9 @@ export default class JobInfo extends React.PureComponent {
           <Clipboard description="job Name" text={jobTypeName} />
         </li>
         {[...timeFields, ...extraFields].map((field) => (
-          <li className="small" key={`${field.title}${field.value}`}>
+          <li className="small" key={`${field.title}${field.value ?? ''}`}>
             <strong>{field.title}: </strong>
-            {field.url ? (
-              <a
-                title={field.value}
-                href={field.url}
-                target="_blank"
-                rel="noopener noreferrer"
-              >
-                {field.value}
-              </a>
-            ) : (
-              <span>{field.value}</span>
-            )}
+            {this.renderFieldValue(field)}
             {field.clipboard && (
               <Clipboard
                 description={field.clipboard.description}
@@ -153,6 +171,12 @@ JobInfo.propTypes = {
         description: PropTypes.string.isRequired,
         text: PropTypes.string,
       }),
+      subfields: PropTypes.arrayOf(
+        PropTypes.exact({
+          name: PropTypes.string.isRequired,
+          value: PropTypes.string.isRequired,
+        }),
+      ),
     }),
   ),
   showJobFilters: PropTypes.bool,

ui/shared/tabs/failureSummary/FailureSummaryTab.jsx

@@ -184,7 +184,9 @@ class FailureSummaryTab extends React.Component {
       suggestions,
       errors,
     } = this.state;
-    const logs = jobLogUrls;
+    const logs = jobLogUrls.filter(
+      (jlu) => !jlu.name.includes('perfherder-data'),
+    );
     const jobLogsAllParsed =
       logs.length > 0 && logs.every((jlu) => jlu.parse_status !== 'pending');