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

Unified Diff: scripts/slave/recipes/bisection/android_bisect_staging.expected/local_basic_recipe_no_device.json

Issue 2489473002: Moving code that clears the bisect_results directory to the right place. (Closed)
Patch Set: Created 4 years, 1 month 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
« no previous file with comments | « scripts/slave/recipes/bisection/android_bisect_staging.expected/local_basic_recipe_failed_device.json ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipes/bisection/android_bisect_staging.expected/local_basic_recipe_no_device.json
diff --git a/scripts/slave/recipes/bisection/android_bisect_staging.expected/local_basic_recipe_no_device.json b/scripts/slave/recipes/bisection/android_bisect_staging.expected/local_basic_recipe_no_device.json
index a64b0ecf6f3a65c61b5d103fa9f56000917e01fe..9848720d312d05ea51e1992e6c81a416795807e7 100644
--- a/scripts/slave/recipes/bisection/android_bisect_staging.expected/local_basic_recipe_no_device.json
+++ b/scripts/slave/recipes/bisection/android_bisect_staging.expected/local_basic_recipe_no_device.json
@@ -404,6 +404,56 @@
"name": "***BISECT***"
},
{
+ "cmd": [],
+ "cwd": "[BUILDER_CACHE]/android_one_perf_bisect",
+ "name": "Clearing results directory"
+ },
+ {
+ "cmd": [
+ "python",
+ "-u",
+ "\nimport os, sys\nfrom common import chromium_utils # Error? See https://crbug.com/584783.\n\n\nif os.path.exists(sys.argv[1]):\n chromium_utils.RemoveDirectory(sys.argv[1])\n",
+ "[SLAVE_BUILD]/bisect_results"
+ ],
+ "cwd": "[BUILDER_CACHE]/android_one_perf_bisect",
+ "name": "Clearing results directory.rmtree old results directory",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@1@@@",
+ "@@@STEP_LOG_LINE@python.inline@@@@",
+ "@@@STEP_LOG_LINE@python.inline@import os, sys@@@",
+ "@@@STEP_LOG_LINE@python.inline@from common import chromium_utils # Error? See https://crbug.com/584783.@@@",
+ "@@@STEP_LOG_LINE@python.inline@@@@",
+ "@@@STEP_LOG_LINE@python.inline@@@@",
+ "@@@STEP_LOG_LINE@python.inline@if os.path.exists(sys.argv[1]):@@@",
+ "@@@STEP_LOG_LINE@python.inline@ chromium_utils.RemoveDirectory(sys.argv[1])@@@",
+ "@@@STEP_LOG_END@python.inline@@@"
+ ]
+ },
+ {
+ "cmd": [
+ "python",
+ "-u",
+ "\nimport sys, os\npath = sys.argv[1]\nmode = int(sys.argv[2])\nif not os.path.isdir(path):\n if os.path.exists(path):\n print \"%s exists but is not a dir\" % path\n sys.exit(1)\n os.makedirs(path, mode)\n",
+ "[SLAVE_BUILD]/bisect_results",
+ "511"
+ ],
+ "cwd": "[BUILDER_CACHE]/android_one_perf_bisect",
+ "name": "Clearing results directory.makedirs new results directory",
+ "~followup_annotations": [
+ "@@@STEP_NEST_LEVEL@1@@@",
+ "@@@STEP_LOG_LINE@python.inline@@@@",
+ "@@@STEP_LOG_LINE@python.inline@import sys, os@@@",
+ "@@@STEP_LOG_LINE@python.inline@path = sys.argv[1]@@@",
+ "@@@STEP_LOG_LINE@python.inline@mode = int(sys.argv[2])@@@",
+ "@@@STEP_LOG_LINE@python.inline@if not os.path.isdir(path):@@@",
+ "@@@STEP_LOG_LINE@python.inline@ if os.path.exists(path):@@@",
+ "@@@STEP_LOG_LINE@python.inline@ print \"%s exists but is not a dir\" % path@@@",
+ "@@@STEP_LOG_LINE@python.inline@ sys.exit(1)@@@",
+ "@@@STEP_LOG_LINE@python.inline@ os.makedirs(path, mode)@@@",
+ "@@@STEP_LOG_END@python.inline@@@"
+ ]
+ },
+ {
"cmd": [
"[BUILDER_CACHE]/android_one_perf_bisect/src/third_party/catapult/devil/devil/android/tools/device_status.py",
"--json-output",
« no previous file with comments | « scripts/slave/recipes/bisection/android_bisect_staging.expected/local_basic_recipe_failed_device.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698