| OLD | NEW |
|---|
| 1 <!DOCTYPE html> | 1 <!DOCTYPE html> |
| 2 <head> | 2 <head> |
| 3 <title>Build Failure</title> | 3 <title>Build Failure</title> |
| 4 <meta charset="utf-8"> | 4 <meta charset="utf-8"> |
| 5 <link rel="stylesheet" href="/common.css"> | 5 <link rel="stylesheet" href="/common.css"> |
| 6 <style> | 6 <style> |
| 7 .running { | 7 .running { |
| 8 color: #666666; | 8 color: #666666; |
| 9 background-color: #fffc6c; | 9 background-color: #fffc6c; |
| 10 border-color: #c5c56d; | 10 border-color: #c5c56d; |
| (...skipping 316 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 327 } | 327 } |
| 328 | 328 |
| 329 function displayUnclassifiedFailures(step_name, results) { | 329 function displayUnclassifiedFailures(step_name, results) { |
| 330 if (!jQuery.isEmptyObject(results)) { | 330 if (!jQuery.isEmptyObject(results)) { |
| 331 var tableString = ''; | 331 var tableString = ''; |
| 332 $.each(results, function(index, result) { | 332 $.each(results, function(index, result) { |
| 333 tableString += '<tr>'; | 333 tableString += '<tr>'; |
| 334 var rowspan = result.heuristic_analysis.suspected_cls.length > 0 ? res
ult.heuristic_analysis.suspected_cls.length : 1; | 334 var rowspan = result.heuristic_analysis.suspected_cls.length > 0 ? res
ult.heuristic_analysis.suspected_cls.length : 1; |
| 335 tableString += generateCommonCellsForAllCategories(step_name, 'undeter
mined', index, result.tests, result.first_failure, result.last_pass, rowspan); | 335 tableString += generateCommonCellsForAllCategories(step_name, 'undeter
mined', index, result.tests, result.first_failure, result.last_pass, rowspan); |
| 336 | 336 |
| 337 tableString += '<td rowspan="' + rowspan + '">' + findit.tryjobStatusM
essageMap[result.try_job.status] + '</td>'; | 337 tableString += '<td id="unclassified_reason_td" rowspan="' + rowspan +
'">' + findit.tryjobStatusMessageMap[result.try_job.status] + '</td>'; |
| 338 | |
| 339 tableString += generateHeuristicCulpritCls(result.heuristic_analysis.s
uspected_cls, result.supported); | 338 tableString += generateHeuristicCulpritCls(result.heuristic_analysis.s
uspected_cls, result.supported); |
| 340 tableString += '</tr>'; | 339 tableString += '</tr>'; |
| 341 }); | 340 }); |
| 342 | 341 |
| 343 $('#unclassified_failures_table tbody').append(tableString); | 342 $('#unclassified_failures_table tbody').append(tableString); |
| 344 $('#unclassified_failures').removeClass('not-display'); | 343 $('#unclassified_failures').removeClass('not-display'); |
| 344 |
| 345 // Do not display Reason to non-admin users. |
| 346 if (! findit.showDebugInfo) { |
| 347 $('#unclassified_reason_th').addClass('not-display'); |
| 348 $('#unclassified_reason_td').addClass('not-display'); |
| 349 } |
| 345 } | 350 } |
| 346 } | 351 } |
| 347 | 352 |
| 348 function displayUnclassifiedFailuresToAdmins(step_name, results) { | 353 function displayUnclassifiedFailuresToAdmins(step_name, results) { |
| 349 if (!jQuery.isEmptyObject(results)) { | 354 if (!jQuery.isEmptyObject(results)) { |
| 350 var tableString = ''; | 355 var tableString = ''; |
| 351 $.each(results, function(index, result) { | 356 $.each(results, function(index, result) { |
| 352 tableString += '<tr>'; | 357 tableString += '<tr>'; |
| 353 var rowspan = result.heuristic_analysis.suspected_cls.length > 0 ? res
ult.heuristic_analysis.suspected_cls.length : 1; | 358 var rowspan = result.heuristic_analysis.suspected_cls.length > 0 ? res
ult.heuristic_analysis.suspected_cls.length : 1; |
| 354 | 359 |
| (...skipping 229 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 584 | 589 |
| 585 <div id="unclassified_failures" class='not-display'> | 590 <div id="unclassified_failures" class='not-display'> |
| 586 <b>Unclassified failures</b> | 591 <b>Unclassified failures</b> |
| 587 <table id="unclassified_failures_table"> | 592 <table id="unclassified_failures_table"> |
| 588 <thead> | 593 <thead> |
| 589 <tr> | 594 <tr> |
| 590 <th rowspan="3" title="Failed step name" width="150px">Step</t
h> | 595 <th rowspan="3" title="Failed step name" width="150px">Step</t
h> |
| 591 <th rowspan="3" title="Failed test name" width="300px">Test(s)
</th> | 596 <th rowspan="3" title="Failed test name" width="300px">Test(s)
</th> |
| 592 <th rowspan="3" title="The build cycle in which the step start
ed to fail">First Failure</th> | 597 <th rowspan="3" title="The build cycle in which the step start
ed to fail">First Failure</th> |
| 593 <th rowspan="3" title="The last build cycle in which the step
passed">Last Pass<br>Before Failure</th> | 598 <th rowspan="3" title="The last build cycle in which the step
passed">Last Pass<br>Before Failure</th> |
| 594 <th rowspan="3" title="The reason we're not sure about the res
ult">Reason</th> | 599 <th id="unclassified_reason_th" rowspan="3" title="The reason
we're not sure about the result">Reason</th> |
| 595 <th colspan="4">Suspected CLs</th> | 600 <th colspan="4">Suspected CLs</th> |
| 596 </tr> | 601 </tr> |
| 597 <tr> | 602 <tr> |
| 598 <th colspan="4">Heuristic Analysis Result</th> | 603 <th colspan="4">Heuristic Analysis Result</th> |
| 599 </tr> | 604 </tr> |
| 600 <tr> | 605 <tr> |
| 601 <th title="The build cycle in which the CL was built or tested
for the first time">Build Number</th> | 606 <th title="The build cycle in which the CL was built or tested
for the first time">Build Number</th> |
| 602 <th title="Git commit position/hash">Commit</th> | 607 <th title="Git commit position/hash">Commit</th> |
| 603 <th title="The higher score, the more suspected">Score (<a cla
ss="score-info" href="javascript:">?</a>)</th> | 608 <th title="The higher score, the more suspected">Score (<a cla
ss="score-info" href="javascript:">?</a>)</th> |
| 604 <th title="Why this CL is related to the failure">Hints</th> | 609 <th title="Why this CL is related to the failure">Hints</th> |
| (...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 691 appearing in the failure log. (eg: file.h was changed and | 696 appearing in the failure log. (eg: file.h was changed and |
| 692 file_unittest.cc or file_impl.cc appeared in the log.)</li> | 697 file_unittest.cc or file_impl.cc appeared in the log.)</li> |
| 693 <li>1: The CL rolled a dependency within src/DEPS and a file of that | 698 <li>1: The CL rolled a dependency within src/DEPS and a file of that |
| 694 dependency appears in the failure log. (eg: third_party/dep | 699 dependency appears in the failure log. (eg: third_party/dep |
| 695 was changed in src/DEPS and third_party/dep/f.cpp appeared | 700 was changed in src/DEPS and third_party/dep/f.cpp appeared |
| 696 in the log.)</li> | 701 in the log.)</li> |
| 697 </ul> | 702 </ul> |
| 698 (More rules will be added when implemented.) | 703 (More rules will be added when implemented.) |
| 699 </div> | 704 </div> |
| 700 </body> | 705 </body> |
| OLD | NEW |
|---|
Chromium Code Reviews
Issue 2129073002:
[Findit] don't display reason for unclassified ressults to non-admin users. (Closed)
Base URL: https://chromium.googlesource.com/infra/infra.git@master