Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(482)

Issues Search
    My Issues | Starred     Open | Closed | All

Side by Side Diff: appengine/findit/templates/build_failure.html

Issue 2086113004: [Findit] Show build analysis references in UI for Findit Cross-platform auto-triage (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@build-matching
Patch Set: Rebased on tip of tree Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « appengine/findit/model/wf_analysis.py ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
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 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
69 </style> 69 </style>
70 <link rel="stylesheet" href="https://ajax.googleapis.com/ajax/libs/jqueryui/1. 11.1/themes/smoothness/jquery-ui.css"> 70 <link rel="stylesheet" href="https://ajax.googleapis.com/ajax/libs/jqueryui/1. 11.1/themes/smoothness/jquery-ui.css">
71 <script src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js "></script> 71 <script src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js "></script>
72 <script src="https://ajax.googleapis.com/ajax/libs/jqueryui/1.11.1/jquery-ui.m in.js"></script> 72 <script src="https://ajax.googleapis.com/ajax/libs/jqueryui/1.11.1/jquery-ui.m in.js"></script>
73 <script> 73 <script>
74 var findit = {}; 74 var findit = {};
75 findit.analysisCompleted = '{{analysis_completed}}' == 'True'; 75 findit.analysisCompleted = '{{analysis_completed}}' == 'True';
76 findit.analysisFailed = '{{analysis_failed}}' == 'True'; 76 findit.analysisFailed = '{{analysis_failed}}' == 'True';
77 findit.builderUrl = 'https://build.chromium.org/p/{{master_name}}/builders/{ {builder_name}}'; 77 findit.builderUrl = 'https://build.chromium.org/p/{{master_name}}/builders/{ {builder_name}}';
78 findit.buildUrl = findit.builderUrl + '/builds/{{build_number}}'; 78 findit.buildUrl = findit.builderUrl + '/builds/{{build_number}}';
79 findit.triageReferenceAnalysisUrl = '/waterfall/build-failure?url=https://bu ild.chromium.org/p/{{triage_reference_analysis_master_name}}/builders/{{triage_r eference_analysis_builder_name}}/builds/{{triage_reference_analysis_build_number }}';
79 findit.analysisCorrect = '{{analysis_correct}}'; // Possible values: 'None' , 'True', or 'False'. 80 findit.analysisCorrect = '{{analysis_correct}}'; // Possible values: 'None' , 'True', or 'False'.
81 findit.analysisIsDuplicate = '{{analysis_is_duplicate}}' == 'True';
80 findit.showTriageHelpButton = '{{show_triage_help_button}}' == 'True'; 82 findit.showTriageHelpButton = '{{show_triage_help_button}}' == 'True';
81 findit.showDebugInfo = '{{show_debug_info}}' == 'True'; 83 findit.showDebugInfo = '{{show_debug_info}}' == 'True';
82 findit.analysisResult = {{analysis_result | tojson | safe}}; 84 findit.analysisResult = {{analysis_result | tojson | safe}};
83 findit.tryjobStatusMessageMap = {{status_message_map | tojson | safe}}; 85 findit.tryjobStatusMessageMap = {{status_message_map | tojson | safe}};
84 findit.defaultGitBaseUrl = 'https://chromium.googlesource.com/chromium/src.g it/+/'; 86 findit.defaultGitBaseUrl = 'https://chromium.googlesource.com/chromium/src.g it/+/';
85 87
86 function triageAnalysisResult(e) { 88 function triageAnalysisResult(e) {
87 var target = $(this); 89 var target = $(this);
88 if (target.hasClass('triaged')) 90 if (target.hasClass('triaged'))
89 return; 91 return;
90 92
91 var correct = target.hasClass('thumb-up'); 93 var correct = target.hasClass('thumb-up');
92 $.getJSON('/waterfall/triage-analysis?correct=' + correct + '&url=' + find it.buildUrl, function(data) { 94 $.getJSON('/waterfall/triage-analysis?correct=' + correct + '&url=' + find it.buildUrl, function(data) {
93 if (data['success']) { 95 if (data['success']) {
94 $('.triaged').addClass('triage').removeClass('triaged'); 96 $('.triaged').addClass('triage').removeClass('triaged');
95 target.addClass('triaged').removeClass('triage'); 97 target.addClass('triaged').removeClass('triage');
96 } else { 98 } else {
97 alert('Failed to update datastore. Please refresh and try again.'); 99 alert('Failed to update datastore. Please refresh and try again.');
98 } 100 }
101 {% if show_debug_info %}
102 $('#duplicates_marked').html('Number of matching build analyses that w ere also triaged: ' + data['num_duplicate_analyses']);
103 $('#duplicate_reference').html('');
104 {% endif %}
99 }).error(function(xhr) { 105 }).error(function(xhr) {
100 // Replace the whole page with errors from server side. 106 // Replace the whole page with errors from server side.
101 document.body.outerHTML = xhr.responseText; 107 document.body.outerHTML = xhr.responseText;
102 }); 108 });
103 109
104 e.preventDefault(); 110 e.preventDefault();
105 } 111 }
106 112
107 function triageHelp(e) { 113 function triageHelp(e) {
108 $('#triage_help_button').prop('disabled', true); 114 $('#triage_help_button').prop('disabled', true);
(...skipping 326 matching lines...) Expand 10 before | Expand all | Expand 10 after
435 } else { 441 } else {
436 $('#triage-and-table').text('no failure is found'); 442 $('#triage-and-table').text('no failure is found');
437 } 443 }
438 444
439 if (findit.analysisCorrect == 'True') { 445 if (findit.analysisCorrect == 'True') {
440 $('.thumb-up').addClass('triaged').removeClass('triage'); 446 $('.thumb-up').addClass('triaged').removeClass('triage');
441 } else if (findit.analysisCorrect == 'False') { 447 } else if (findit.analysisCorrect == 'False') {
442 $('.thumb-down').addClass('triaged').removeClass('triage'); 448 $('.thumb-down').addClass('triaged').removeClass('triage');
443 } 449 }
444 450
451 {% if show_debug_info %}
452 if (findit.analysisIsDuplicate) {
453 $('#duplicate_reference').html('This analysis was automatically tr iaged as a duplicate because of another build analysis: ' +
454 '<a href="' + findit.triageReferenc eAnalysisUrl + '">{{triage_reference_analysis_master_name}}/' +
455 '{{triage_reference_analysis_builde r_name}}/{{triage_reference_analysis_build_number}}</a>');
456 }
457 {% endif %}
458
445 if (findit.showTriageHelpButton) { 459 if (findit.showTriageHelpButton) {
446 $('#triage_help_button').click(triageHelp); 460 $('#triage_help_button').click(triageHelp);
447 } 461 }
448 } 462 }
449 } 463 }
450 464
451 $('#score-explanation-dialog').dialog({ 465 $('#score-explanation-dialog').dialog({
452 autoOpen: false, 466 autoOpen: false,
453 modal: true, 467 modal: true,
454 width: 600, 468 width: 600,
(...skipping 190 matching lines...) Expand 10 before | Expand all | Expand 10 after
645 </tbody> 659 </tbody>
646 </table> 660 </table>
647 <br> 661 <br>
648 </div> 662 </div>
649 663
650 <div class="thumbs-up-down"> 664 <div class="thumbs-up-down">
651 <b>Feedback on Findit result:</b><br> 665 <b>Feedback on Findit result:</b><br>
652 <div class="triage thumb-up">Correct <img src="https://www.gstatic.com /images/icons/material/system/1x/thumb_up_black_24dp.png"/></div> 666 <div class="triage thumb-up">Correct <img src="https://www.gstatic.com /images/icons/material/system/1x/thumb_up_black_24dp.png"/></div>
653 <div class="triage thumb-down"><img src="https://www.gstatic.com/image s/icons/material/system/1x/thumb_down_black_24dp.png"/> Incorrect</div> 667 <div class="triage thumb-down"><img src="https://www.gstatic.com/image s/icons/material/system/1x/thumb_down_black_24dp.png"/> Incorrect</div>
654 </div> 668 </div>
669 {% if show_debug_info %}
670 <div id="duplicates_marked"></div>
671 <div id="duplicate_reference"></div>
672 {% endif %}
655 </div> 673 </div>
656 {% endif %} 674 {% endif %}
657 {% endif %} 675 {% endif %}
658 676
659 {% if show_triage_help_button %} 677 {% if show_triage_help_button %}
660 <br><br> 678 <br><br>
661 <div id="triage_help"> 679 <div id="triage_help">
662 <button id="triage_help_button">Triage help</button> 680 <button id="triage_help_button">Triage help</button>
663 <br> 681 <br>
664 <div id="triage_help_result"> 682 <div id="triage_help_result">
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
696 appearing in the failure log. (eg: file.h was changed and 714 appearing in the failure log. (eg: file.h was changed and
697 file_unittest.cc or file_impl.cc appeared in the log.)</li> 715 file_unittest.cc or file_impl.cc appeared in the log.)</li>
698 <li>1: The CL rolled a dependency within src/DEPS and a file of that 716 <li>1: The CL rolled a dependency within src/DEPS and a file of that
699 dependency appears in the failure log. (eg: third_party/dep 717 dependency appears in the failure log. (eg: third_party/dep
700 was changed in src/DEPS and third_party/dep/f.cpp appeared 718 was changed in src/DEPS and third_party/dep/f.cpp appeared
701 in the log.)</li> 719 in the log.)</li>
702 </ul> 720 </ul>
703 (More rules will be added when implemented.) 721 (More rules will be added when implemented.)
704 </div> 722 </div>
705 </body> 723 </body>
OLDNEW
« no previous file with comments | « appengine/findit/model/wf_analysis.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698