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

Unified Diff: scripts/slave/recipes/bisect_test/example_bisect_results.expected/basic_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_test/example_bisect_results.expected/basic_test.json
diff --git a/scripts/slave/recipes/bisect_test/example_bisect_results.expected/basic_test.json b/scripts/slave/recipes/bisect_test/example_bisect_results.expected/basic_test.json
index 2b0ff67e0507f7ac0ebca4a7575545a903b3ed22..bd26b86d90668e46a0615dfcc81f033715e6466a 100644
--- a/scripts/slave/recipes/bisect_test/example_bisect_results.expected/basic_test.json
+++ b/scripts/slave/recipes/bisect_test/example_bisect_results.expected/basic_test.json
@@ -166,25 +166,24 @@
"@@@STEP_LOG_LINE@json.output@ \"email\": \"dummy@nowhere.com\", @@@",
"@@@STEP_LOG_LINE@json.output@ \"subject\": \"Some random CL\"@@@",
"@@@STEP_LOG_LINE@json.output@}@@@",
- "@@@STEP_LOG_END@json.output@@@",
- "@@@SET_BUILD_PROPERTY@extra_result_code@[\"B4T_LO_FINAL_CONF\"]@@@"
+ "@@@STEP_LOG_END@json.output@@@"
]
},
{
"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: Completed, but no culprit was found with high confidence.\n\nTest Command: src/tools/perf/run_benchmark -v --browser=release smoothness.tough_scrolling_cases\nTest Metric: mean_input_event_latency/mean_input_event_latency\nRelative Change: None\nScore: 0.0\nRetested CL with revert: Not Implemented.\n\n\n===== WARNINGS =====\nThe following warnings were raised by the bisect job:\n\n * This is a demo warning\n\n\n===== SUSPECTED CL(s) =====\nSubject : Some random CL\nAuthor : DummyAuthor, dummy@nowhere.com\nCommit description:\n A long description for a CL.\nContaining multiple lines\nCommit : deadbeef\nDate : 01/01/2015\n\n\n===== TESTED REVISIONS =====\nDepot Revision Mean Value Std. Dev. Num Values 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\": \"deadbeef\", \"bisect_bot\": \"linux_perf_tester\", \"bug_id\": \"\", \"buildbot_log_url\": \"http://build.chromium.org/p/chromium/builders//builds/\", \"change\": null, \"command\": \"src/tools/perf/run_benchmark -v --browser=release smoothness.tough_scrolling_cases\", \"culprit_data\": {\"author\": \"DummyAuthor\", \"cl\": \"deadbeef\", \"cl_date\": \"01/01/2015\", \"commit_info\": \"A long description for a CL.\\nContaining multiple lines\", \"email\": \"dummy@nowhere.com\", \"revisions_links\": [], \"subject\": \"Some random CL\"}, \"fail_reason\": null, \"good_revision\": \"c001c0de\", \"metric\": \"mean_input_event_latency/mean_input_event_latency\", \"revision_data\": [], \"score\": 0.0, \"status\": \"completed\", \"test_type\": \"perf\", \"try_job_id\": null, \"warnings\": [\"This is a demo warning\"]}, \"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