Recipes: make cwd default to slave_build

Recipes: make cwd default to slave_build

This is a step towards running recipes on swarming.

BUG=437264
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=295056

[Paweł Hajdan Jr., 2015-04-28 15:54:13]

phajdan.jr@chromium.org changed reviewers:
+ iannucci@chromium.org

[Paweł Hajdan Jr., 2015-04-28 15:54:19]

Mostly expectation changes. The real change is in step recipe module, look for
.py

[iannucci, 2015-04-28 16:46:37]

lgtm!

[Paweł Hajdan Jr., 2015-04-29 09:26:58]

The CQ bit was checked by phajdan.jr@chromium.org

[commit-bot: I haz the power, 2015-04-29 09:27:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1115453002/1

[commit-bot: I haz the power, 2015-04-29 09:30:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-29 09:30:57]

Failed to apply patch for
build/scripts/slave/recipes/chromium.expected/full_chromium_fyi_Android_Remoting_Tests.json:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file
build/scripts/slave/recipes/chromium.expected/full_chromium_fyi_Android_Remoting_Tests.json
  Hunk #8 succeeded at 181 with fuzz 2 (offset 1 line).
  Hunk #9 succeeded at 194 (offset 1 line).
  Hunk #10 succeeded at 208 (offset 1 line).
  Hunk #11 succeeded at 220 (offset 1 line).
  Hunk #12 succeeded at 261 (offset 1 line).
  Hunk #13 succeeded at 276 (offset 1 line).
  Hunk #14 FAILED at 296.
  Hunk #15 succeeded at 346 (offset 35 lines).
  Hunk #16 succeeded at 357 (offset 35 lines).
  Hunk #17 succeeded at 370 (offset 35 lines).
  Hunk #18 succeeded at 383 (offset 35 lines).
  1 out of 18 hunks FAILED -- saving rejects to file
build/scripts/slave/recipes/chromium.expected/full_chromium_fyi_Android_Remoting_Tests.json.rej

Patch:      
build/scripts/slave/recipes/chromium.expected/full_chromium_fyi_Android_Remoting_Tests.json
Index:
scripts/slave/recipes/chromium.expected/full_chromium_fyi_Android_Remoting_Tests.json
diff --git
build/scripts/slave/recipes/chromium.expected/full_chromium_fyi_Android_Remoting_Tests.json
build/scripts/slave/recipes/chromium.expected/full_chromium_fyi_Android_Remoting_Tests.json
index
bfecf8b42bc802310dc21b3a2900dbb908816f70..4560c6ce40738356568043f412814c45e530b9f8
100644
---
a/build/scripts/slave/recipes/chromium.expected/full_chromium_fyi_Android_Remoting_Tests.json
+++
b/build/scripts/slave/recipes/chromium.expected/full_chromium_fyi_Android_Remoting_Tests.json
@@ -21,6 +21,7 @@
       "--revision",
       "src@HEAD"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "name": "bot_update",
     "~followup_annotations": [
       "@@@STEP_TEXT@Some step text@@@",
@@ -67,6 +68,7 @@
       "[DEPOT_TOOLS]/gclient.py",
       "runhooks"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "env": {
       "GYP_CROSSCOMPILE": "1",
       "GYP_DEFINES": "OS=android component=static_library fastbuild=1
gomadir='[BUILD]/goma' target_arch=arm use_goma=1",
@@ -82,6 +84,7 @@
       "[SLAVE_BUILD]/src/testing/buildbot/chromium.fyi.json",
       "/path/to/tmp/json"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "name": "read test spec",
     "~followup_annotations": [
       "@@@STEP_TEXT@path:
[SLAVE_BUILD]/src/testing/buildbot/chromium.fyi.json@@@",
@@ -104,6 +107,7 @@
       "--properties",
       "{\"buildername\": \"Android Remoting Tests\", \"buildnumber\": 571,
\"mastername\": \"chromium.fyi\", \"slavename\": \"TestSlavename\",
\"target_platform\": \"android\"}"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "name": "get compile targets for scripts",
     "~followup_annotations": [
       "@@@STEP_LOG_LINE@json.output@{}@@@",
@@ -116,6 +120,7 @@
       "-u",
       "[BUILD]/scripts/slave/cleanup_temp.py"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "name": "cleanup_temp"
   },
   {
@@ -128,6 +133,7 @@
       "[SLAVE_BUILD]/src/out/build_product.zip",
       "[SLAVE_BUILD]/src"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "name": "clean local files",
     "~followup_annotations": [
       "@@@STEP_LOG_LINE@python.inline@@@@",
@@ -152,6 +158,7 @@
       "[SLAVE_BUILD]/src",
       "src"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "name": "tree truth steps"
   },
   {
@@ -173,6 +180,7 @@
       "remoting_apk",
       "remoting_test_apk"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "env": {
       "PATH":
"[SLAVE_BUILD]/src/third_party/android_tools/sdk/platform-tools:[SLAVE_BUILD]/src/build/android:%(PATH)s"
     },
@@ -185,6 +193,7 @@
       "[SLAVE_BUILD]/src/build/android/adb_logcat_monitor.py",
       "[SLAVE_BUILD]/src/out/logcat"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "env": {
       "PATH":
"[SLAVE_BUILD]/src/third_party/android_tools/sdk/platform-tools:[SLAVE_BUILD]/src/build/android:%(PATH)s"
     },
@@ -198,6 +207,7 @@
       "--adb-path",
       "[SLAVE_BUILD]/src/third_party/android_tools/sdk/platform-tools/adb"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "env": {
       "PATH":
"[SLAVE_BUILD]/src/third_party/android_tools/sdk/platform-tools:[SLAVE_BUILD]/src/build/android:%(PATH)s"
     },
@@ -209,6 +219,7 @@
       "--json-output",
       "/path/to/tmp/json"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "env": {
       "PATH":
"[SLAVE_BUILD]/src/third_party/android_tools/sdk/platform-tools:[SLAVE_BUILD]/src/build/android:%(PATH)s"
     },
@@ -249,6 +260,7 @@
       "-t",
       "Release"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "env": {
       "PATH":
"[SLAVE_BUILD]/src/third_party/android_tools/sdk/platform-tools:[SLAVE_BUILD]/src/build/android:%(PATH)s"
     },
@@ -263,6 +275,7 @@
       "org.chromium.chromoting",
       "--release"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "env": {
       "PATH":
"[SLAVE_BUILD]/src/third_party/android_tools/sdk/platform-tools:[SLAVE_BUILD]/src/build/android:%(PATH)s"
     },
@@ -283,6 +296,7 @@
       "--json-results-file",
       "/path/to/tmp/json"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "name": "Instrumentation test ChromotingTest",
     "~followup_annotations": [
       "@@@STEP_TEXT@<br/>failures:<br/>TestB<br/>@@@"
@@ -298,6 +312,7 @@
       "[SLAVE_BUILD]/src/build/android/adb_logcat_printer.py",
       "[SLAVE_BUILD]/src/out/logcat"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "name": "logcat_dump"
   },
   {
@@ -308,6 +323,7 @@
       "--more-info",
       "[SLAVE_BUILD]/src/out/Release/full_log"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "env": {
       "PATH":
"[SLAVE_BUILD]/src/third_party/android_tools/sdk/platform-tools:[SLAVE_BUILD]/src/build/android:%(PATH)s"
     },
@@ -320,6 +336,7 @@
       "-s",
       "-w"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "env": {
       "PATH":
"[SLAVE_BUILD]/src/third_party/android_tools/sdk/platform-tools:[SLAVE_BUILD]/src/build/android:%(PATH)s"
     },
@@ -332,6 +349,7 @@
       "\nimport glob, os, sys\nfor report in glob.glob(sys.argv[1]):\n  with
open(report, 'r') as f:\n    for l in f.readlines():\n      print l\n 
os.remove(report)\n",
       "[SLAVE_BUILD]/src/out/Release/test_logs/*.log"
     ],
+    "cwd": "[SLAVE_BUILD]",
     "name": "test_report",
     "~followup_annotations": [
       "@@@STEP_LOG_LINE@python.inline@@@@",

[Paweł Hajdan Jr., 2015-04-29 10:06:02]

The CQ bit was checked by phajdan.jr@chromium.org

[Paweł Hajdan Jr., 2015-04-29 10:06:05]

The patchset sent to the CQ was uploaded after l-g-t-m from
iannucci@chromium.org
Link to the patchset: https://codereview.chromium.org/1115453002/#ps10001
(title: "rebase")

[commit-bot: I haz the power, 2015-04-29 10:06:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1115453002/10001

[commit-bot: I haz the power, 2015-04-29 10:19:43]

Committed patchset #2 (id:10001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=295056