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

Unified Diff: scripts/slave/recipes/bisect_test/example_bisect_results.expected/basic_test.json

Issue 1782333002: Add bisect config validation. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Rebased Created 4 years, 9 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 0d11058b5727b6ad6fc1ad4204e47ed5b64e0594..bf0ca13454e0a08685b62d32bab45e811f2b0647 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
@@ -25,7 +25,6 @@
"~followup_annotations": [
"@@@STEP_LOG_LINE@Bisect job configuration@{@@@",
"@@@STEP_LOG_LINE@Bisect job configuration@ \"bad_revision\": \"314017\", @@@",
- "@@@STEP_LOG_LINE@Bisect job configuration@ \"bug_id\": \"\", @@@",
"@@@STEP_LOG_LINE@Bisect job configuration@ \"builder_host\": \"master4.golo.chromium.org\", @@@",
"@@@STEP_LOG_LINE@Bisect job configuration@ \"builder_port\": \"8341\", @@@",
"@@@STEP_LOG_LINE@Bisect job configuration@ \"bypass_stats_check\": \"True\", @@@",
@@ -38,8 +37,7 @@
"@@@STEP_LOG_LINE@Bisect job configuration@ \"metric\": \"mean_input_event_latency/mean_input_event_latency\", @@@",
"@@@STEP_LOG_LINE@Bisect job configuration@ \"repeat_count\": \"2\", @@@",
"@@@STEP_LOG_LINE@Bisect job configuration@ \"skip_gclient_ops\": \"True\", @@@",
- "@@@STEP_LOG_LINE@Bisect job configuration@ \"test_type\": \"perf\", @@@",
- "@@@STEP_LOG_LINE@Bisect job configuration@ \"truncate_percent\": \"0\"@@@",
+ "@@@STEP_LOG_LINE@Bisect job configuration@ \"test_type\": \"perf\"@@@",
"@@@STEP_LOG_LINE@Bisect job configuration@}@@@",
"@@@STEP_LOG_END@Bisect job configuration@@@"
]
@@ -151,7 +149,7 @@
],
"cwd": "[SLAVE_BUILD]",
"name": "Post bisect results",
- "stdin": "{\"data\": {\"aborted_reason\": null, \"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\"}, \"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\"}",
+ "stdin": "{\"data\": {\"aborted_reason\": null, \"bad_revision\": \"deadbeef\", \"bisect_bot\": \"linux_perf_tester\", \"bug_id\": null, \"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\"}, \"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@json.output@{@@@",

Powered by Google App Engine
This is Rietveld 408576698