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

Side by Side Diff: scripts/slave/recipes/chromium.expected/full_chromium_chromiumos_ChromiumOS_amd64_generic_Compile.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]/ChromiumOS_amd64_generic_Compile", 7 "[BUILDER_CACHE]/ChromiumOS_amd64_generic_Compile",
8 "511" 8 "511"
9 ], 9 ],
10 "name": "makedirs checkout path", 10 "name": "makedirs checkout path",
(...skipping 171 matching lines...) Expand 10 before | Expand all | Expand 10 after
182 { 182 {
183 "cmd": [ 183 "cmd": [
184 "RECIPE_PACKAGE_REPO[depot_tools]/cros", 184 "RECIPE_PACKAGE_REPO[depot_tools]/cros",
185 "chrome-sdk", 185 "chrome-sdk",
186 "--board=amd64-generic", 186 "--board=amd64-generic",
187 "--nocolor", 187 "--nocolor",
188 "--use-external-config", 188 "--use-external-config",
189 "--clear-sdk-cache", 189 "--clear-sdk-cache",
190 "--gomadir", 190 "--gomadir",
191 "[CACHE]/cipd/goma", 191 "[CACHE]/cipd/goma",
192 "--nostart-goma",
192 "--", 193 "--",
193 "python", 194 "python",
194 "-u", 195 "-u",
195 "RECIPE_PACKAGE_REPO[depot_tools]/gclient.py", 196 "RECIPE_PACKAGE_REPO[depot_tools]/gclient.py",
196 "runhooks" 197 "runhooks"
197 ], 198 ],
198 "cwd": "[BUILDER_CACHE]/ChromiumOS_amd64_generic_Compile/src", 199 "cwd": "[BUILDER_CACHE]/ChromiumOS_amd64_generic_Compile/src",
199 "env": { 200 "env": {
200 "GOMA_SERVICE_ACCOUNT_JSON_FILE": "/creds/service_accounts/service-account -goma-client.json", 201 "GOMA_SERVICE_ACCOUNT_JSON_FILE": "/creds/service_accounts/service-account -goma-client.json",
201 "GYP_CHROMIUM_NO_ACTION": "1", 202 "GYP_CHROMIUM_NO_ACTION": "1",
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
254 { 255 {
255 "cmd": [ 256 "cmd": [
256 "RECIPE_PACKAGE_REPO[depot_tools]/cros", 257 "RECIPE_PACKAGE_REPO[depot_tools]/cros",
257 "chrome-sdk", 258 "chrome-sdk",
258 "--board=amd64-generic", 259 "--board=amd64-generic",
259 "--nocolor", 260 "--nocolor",
260 "--use-external-config", 261 "--use-external-config",
261 "--clear-sdk-cache", 262 "--clear-sdk-cache",
262 "--gomadir", 263 "--gomadir",
263 "[CACHE]/cipd/goma", 264 "[CACHE]/cipd/goma",
265 "--nostart-goma",
264 "--", 266 "--",
265 "python", 267 "python",
266 "-u", 268 "-u",
267 "[BUILDER_CACHE]/ChromiumOS_amd64_generic_Compile/src/tools/mb/mb.py", 269 "[BUILDER_CACHE]/ChromiumOS_amd64_generic_Compile/src/tools/mb/mb.py",
268 "gen", 270 "gen",
269 "-m", 271 "-m",
270 "chromium.chromiumos", 272 "chromium.chromiumos",
271 "-b", 273 "-b",
272 "ChromiumOS amd64-generic Compile", 274 "ChromiumOS amd64-generic Compile",
273 "--config-file", 275 "--config-file",
(...skipping 10 matching lines...) Expand all
284 }, 286 },
285 { 287 {
286 "cmd": [ 288 "cmd": [
287 "RECIPE_PACKAGE_REPO[depot_tools]/cros", 289 "RECIPE_PACKAGE_REPO[depot_tools]/cros",
288 "chrome-sdk", 290 "chrome-sdk",
289 "--board=amd64-generic", 291 "--board=amd64-generic",
290 "--nocolor", 292 "--nocolor",
291 "--use-external-config", 293 "--use-external-config",
292 "--gomadir", 294 "--gomadir",
293 "[CACHE]/cipd/goma", 295 "[CACHE]/cipd/goma",
296 "--nostart-goma",
294 "--", 297 "--",
295 "python", 298 "python",
296 "-u", 299 "-u",
297 "RECIPE_PACKAGE_REPO[build]/scripts/tools/runit.py", 300 "RECIPE_PACKAGE_REPO[build]/scripts/tools/runit.py",
298 "--show-path", 301 "--show-path",
299 "python", 302 "python",
300 "RECIPE_PACKAGE_REPO[build]/scripts/slave/compile.py", 303 "RECIPE_PACKAGE_REPO[build]/scripts/slave/compile.py",
301 "--gsutil-py-path", 304 "--gsutil-py-path",
302 "RECIPE_PACKAGE_REPO[depot_tools]/gsutil.py", 305 "RECIPE_PACKAGE_REPO[depot_tools]/gsutil.py",
303 "--ninja-path", 306 "--ninja-path",
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
371 "{\"blamelist\": [\"cool_dev1337@chromium.org\", \"hax@chromium.org\"], \" build_data_dir\": \"[ROOT]/build_data_dir\", \"buildername\": \"ChromiumOS amd64 -generic Compile\", \"buildnumber\": 571, \"mastername\": \"chromium.chromiumos\ ", \"parent_buildername\": null, \"path_config\": \"kitchen\", \"recipe\": \"chr omium\", \"slavename\": \"TestSlavename\", \"workdir\": \"/path/to/workdir/TestS lavename\"}" 374 "{\"blamelist\": [\"cool_dev1337@chromium.org\", \"hax@chromium.org\"], \" build_data_dir\": \"[ROOT]/build_data_dir\", \"buildername\": \"ChromiumOS amd64 -generic Compile\", \"buildnumber\": 571, \"mastername\": \"chromium.chromiumos\ ", \"parent_buildername\": null, \"path_config\": \"kitchen\", \"recipe\": \"chr omium\", \"slavename\": \"TestSlavename\", \"workdir\": \"/path/to/workdir/TestS lavename\"}"
372 ], 375 ],
373 "name": "package build" 376 "name": "package build"
374 }, 377 },
375 { 378 {
376 "name": "$result", 379 "name": "$result",
377 "recipe_result": null, 380 "recipe_result": null,
378 "status_code": 0 381 "status_code": 0
379 } 382 }
380 ] 383 ]
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698