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

Side by Side Diff: scripts/slave/recipes/chromium_trybot.expected/chromeos_analyze.json

Issue 2315653002: Do not start goma in ChromeOS gclient runhook if gomadir is given. (Closed)
Patch Set: added a comment. Created 4 years, 3 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
OLDNEW
1 [ 1 [
2 { 2 {
3 "cmd": [ 3 "cmd": [
4 "python", 4 "python",
5 "-u", 5 "-u",
6 "\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", 6 "\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",
7 "[BUILDER_CACHE]/chromeos_amd64_generic_chromium_compile_only_ng", 7 "[BUILDER_CACHE]/chromeos_amd64_generic_chromium_compile_only_ng",
8 "511" 8 "511"
9 ], 9 ],
10 "name": "makedirs checkout path", 10 "name": "makedirs checkout path",
(...skipping 177 matching lines...) Expand 10 before | Expand all | Expand 10 after
188 { 188 {
189 "cmd": [ 189 "cmd": [
190 "RECIPE_PACKAGE_REPO[depot_tools]/cros", 190 "RECIPE_PACKAGE_REPO[depot_tools]/cros",
191 "chrome-sdk", 191 "chrome-sdk",
192 "--board=amd64-generic", 192 "--board=amd64-generic",
193 "--nocolor", 193 "--nocolor",
194 "--use-external-config", 194 "--use-external-config",
195 "--clear-sdk-cache", 195 "--clear-sdk-cache",
196 "--gomadir", 196 "--gomadir",
197 "[CACHE]/cipd/goma", 197 "[CACHE]/cipd/goma",
198 "--nostart-goma",
198 "--fastbuild", 199 "--fastbuild",
199 "--", 200 "--",
200 "python", 201 "python",
201 "-u", 202 "-u",
202 "RECIPE_PACKAGE_REPO[depot_tools]/gclient.py", 203 "RECIPE_PACKAGE_REPO[depot_tools]/gclient.py",
203 "runhooks" 204 "runhooks"
204 ], 205 ],
205 "cwd": "[BUILDER_CACHE]/chromeos_amd64_generic_chromium_compile_only_ng/src" , 206 "cwd": "[BUILDER_CACHE]/chromeos_amd64_generic_chromium_compile_only_ng/src" ,
206 "env": { 207 "env": {
207 "GOMA_SERVICE_ACCOUNT_JSON_FILE": "/creds/service_accounts/service-account -goma-client.json", 208 "GOMA_SERVICE_ACCOUNT_JSON_FILE": "/creds/service_accounts/service-account -goma-client.json",
(...skipping 82 matching lines...) Expand 10 before | Expand all | Expand 10 after
290 }, 291 },
291 { 292 {
292 "cmd": [ 293 "cmd": [
293 "RECIPE_PACKAGE_REPO[depot_tools]/cros", 294 "RECIPE_PACKAGE_REPO[depot_tools]/cros",
294 "chrome-sdk", 295 "chrome-sdk",
295 "--board=amd64-generic", 296 "--board=amd64-generic",
296 "--nocolor", 297 "--nocolor",
297 "--use-external-config", 298 "--use-external-config",
298 "--gomadir", 299 "--gomadir",
299 "[CACHE]/cipd/goma", 300 "[CACHE]/cipd/goma",
301 "--nostart-goma",
300 "--fastbuild", 302 "--fastbuild",
301 "--", 303 "--",
302 "python", 304 "python",
303 "-u", 305 "-u",
304 "[BUILDER_CACHE]/chromeos_amd64_generic_chromium_compile_only_ng/src/tools /mb/mb.py", 306 "[BUILDER_CACHE]/chromeos_amd64_generic_chromium_compile_only_ng/src/tools /mb/mb.py",
305 "analyze", 307 "analyze",
306 "-m", 308 "-m",
307 "tryserver.chromium.linux", 309 "tryserver.chromium.linux",
308 "-b", 310 "-b",
309 "chromeos_amd64-generic_chromium_compile_only_ng", 311 "chromeos_amd64-generic_chromium_compile_only_ng",
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
344 "import sys; sys.exit(0)" 346 "import sys; sys.exit(0)"
345 ], 347 ],
346 "name": "mark: before_tests" 348 "name": "mark: before_tests"
347 }, 349 },
348 { 350 {
349 "name": "$result", 351 "name": "$result",
350 "recipe_result": null, 352 "recipe_result": null,
351 "status_code": 0 353 "status_code": 0
352 } 354 }
353 ] 355 ]
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698