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

Unified Diff: scripts/slave/recipes/bisect.expected/broken_bad_revision_test.json

Issue 1573293002: Change auto_bisect to post results to perf dashboard. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: . Created 4 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: scripts/slave/recipes/bisect.expected/broken_bad_revision_test.json
diff --git a/scripts/slave/recipes/bisect.expected/broken_bad_revision_test.json b/scripts/slave/recipes/bisect.expected/broken_bad_revision_test.json
index 269f9a6014ad104692a1878cc8a026bf15384d72..5c6b42d4f5055f3de58d30eef592b3114b223843 100644
--- a/scripts/slave/recipes/bisect.expected/broken_bad_revision_test.json
+++ b/scripts/slave/recipes/bisect.expected/broken_bad_revision_test.json
@@ -707,11 +707,11 @@
"@@@STEP_LOG_LINE@Debug Info@bisector.lkgr: PerfRevisionState(cp=306475, values=[70, 70, 80, 90, 90], mean_value=80.0, std_dev=10.0)@@@",
"@@@STEP_LOG_LINE@Debug Info@bisector.fkbr: PerfRevisionState(cp=306478, values=[], mean_value=None, std_dev=None)@@@",
"@@@STEP_LOG_LINE@Debug Info@@@@",
- "@@@STEP_LOG_LINE@Debug Info@Revision Values @@@",
- "@@@STEP_LOG_LINE@Debug Info@306475 [70, 70, 80, 90, 90] @@@",
- "@@@STEP_LOG_LINE@Debug Info@306476 [] @@@",
- "@@@STEP_LOG_LINE@Debug Info@306477 [] @@@",
- "@@@STEP_LOG_LINE@Debug Info@306478 [] @@@",
+ "@@@STEP_LOG_LINE@Debug Info@Revision Values @@@",
+ "@@@STEP_LOG_LINE@Debug Info@306475 [70, 70, 80, 90, 90]@@@",
+ "@@@STEP_LOG_LINE@Debug Info@306476 [] @@@",
+ "@@@STEP_LOG_LINE@Debug Info@306477 [] @@@",
+ "@@@STEP_LOG_LINE@Debug Info@306478 [] @@@",
"@@@STEP_LOG_END@Debug Info@@@"
]
},
@@ -719,17 +719,17 @@
"cmd": [
"python",
"-u",
- "\nimport shutil\nimport sys\nshutil.copyfileobj(open(sys.argv[1]), sys.stdout)\n",
- "---bisect results start here---\n\n===== BISECT JOB RESULTS =====\nStatus: Negative: Failed to bisect.\n\nTest Command: tools/perf/run_benchmark -v --browser=release page_cycler.intl_ar_fa_he\nTest Metric: warm_times/page_load_time\nRelative Change: None\nScore: None\nRetested CL with revert: Not Implemented.\n\n\n===== TESTED REVISIONS =====\nDepot Revision Mean Value Std. Dev. Num Values Good? \nchromium r306475 80.0 10.0 5 good \n\n\n\n| O O | Visit http://www.chromium.org/developers/speed-infra/perf-bug-faq\n| X | for more information addressing perf regression bugs. For feedback,\n| / \\ | file a bug with label Cr-Tests-AutoBisect. Thank you!"
+ "RECIPE_MODULE[build::perf_dashboard]/resources/post_json.py"
],
"cwd": "[SLAVE_BUILD]",
- "name": "Results",
+ "name": "Post bisect results",
+ "stdin": "{\"data\": {\"bad_revision\": \"a6298e4afedbf2cd461755ea6f45b0ad64222222\", \"bisect_bot\": \"linux_perf_tester\", \"bug_id\": \"425582\", \"buildbot_log_url\": \"http://c.org/p/cr/builders/linux_perf_bisector/builds/571\", \"change\": null, \"command\": \"tools/perf/run_benchmark -v --browser=release page_cycler.intl_ar_fa_he\", \"culprit_data\": null, \"fail_reason\": null, \"good_revision\": \"e28dc0d49c331def2a3bbf3ddd0096eb51551155\", \"metric\": \"warm_times/page_load_time\", \"revision_data\": [{\"commit_pos\": 306475, \"depot_name\": \"chromium\", \"deps_revision\": null, \"mean_value\": 80.0, \"result\": \"good\", \"std_dev\": 10.0, \"values\": [70, 70, 80, 90, 90]}], \"score\": 0, \"status\": \"failed\", \"test_type\": \"perf\", \"try_job_id\": null, \"warnings\": []}, \"url\": \"https://chromeperf.appspot.com/post_bisect_results\"}",
+ "stdout": "/path/to/tmp/json",
"~followup_annotations": [
- "@@@STEP_LOG_LINE@python.inline@@@@",
- "@@@STEP_LOG_LINE@python.inline@import shutil@@@",
- "@@@STEP_LOG_LINE@python.inline@import sys@@@",
- "@@@STEP_LOG_LINE@python.inline@shutil.copyfileobj(open(sys.argv[1]), sys.stdout)@@@",
- "@@@STEP_LOG_END@python.inline@@@"
+ "@@@STEP_LOG_LINE@json.output@{@@@",
+ "@@@STEP_LOG_LINE@json.output@ \"status_code\": 200@@@",
+ "@@@STEP_LOG_LINE@json.output@}@@@",
+ "@@@STEP_LOG_END@json.output@@@"
]
},
{

Powered by Google App Engine
This is Rietveld 408576698