Archiving of AppRTC APK in client.webrtc waterfall

scripts/slave/recipes/webrtc/standalone.expected/client_webrtc_android32_builder.json

 [
   {
     "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",
       "[BUILDER_CACHE]/Android32_Builder",
       "511"
     ],
     "name": "makedirs checkout path",
@@ skipping 262 matching lines @@
       "--build_revision",
       "f27fede2220bcd326aee3e86ddfd4ebd0fe58cb9",
       "--factory-properties",
       "{\"BUILD_CONFIG\": \"Release\", \"buildername\": \"Android32 Builder\", \"buildnumber\": 1337, \"mastername\": \"client.webrtc\", \"path_config\": \"kitchen\", \"recipe\": \"webrtc/standalone\", \"revision\": \"12345\", \"slavename\": \"slavename\"}",
       "--build-properties",
       "{\"BUILD_CONFIG\": \"Release\", \"buildername\": \"Android32 Builder\", \"buildnumber\": 1337, \"mastername\": \"client.webrtc\", \"path_config\": \"kitchen\", \"recipe\": \"webrtc/standalone\", \"revision\": \"12345\", \"slavename\": \"slavename\"}"
     ],
     "name": "package build"
   },
   {
+    "cmd": [
+      "python",
+      "-u",
+      "RECIPE_MODULE[build::zip]/resources/zip.py"
+    ],
+    "name": "AppRTCDemo zip archive",
+    "stdin": "{\"entries\": [{\"archive_name\": null, \"path\": \"[BUILDER_CACHE]/Android32_Builder/src/out/Release/apks/AppRTCDemo.apk\", \"type\": \"file\"}], \"output\": \"[SLAVE_BUILD]/AppRTCDemo_apk.zip\", \"root\": \"[BUILDER_CACHE]/Android32_Builder/src/out/Release/apks\"}"
+  },
+  {
+    "cmd": [
+      "python",
+      "-u",
+      "RECIPE_MODULE[build::gsutil]/resources/gsutil_wrapper.py",
+      "--",
+      "RECIPE_PACKAGE_REPO[depot_tools]/gsutil.py",
+      "----",
+      "cp",
+      "-a",
+      "public-read",
+      "[SLAVE_BUILD]/AppRTCDemo_apk.zip",
+      "gs://chromium-webrtc/Android32_Builder/AppRTCDemo_apk_170242.zip"

[kjellander_chromium, 2016/09/18 17:33:23]

With the suggested approach above, this will be
gs://chromium-webrtc/client.webrtc/Android32 Builder
(dbg)/AppRTCDemo_apk_170242.zip

Which makes more sense, since the master-name directories already exists in the
bucket (and we want to ensure we don't have collisions if we accidentally use
the same buildernames for builders in multiple waterfalls).

[ehmaldonado_chromium, 2016/09/19 06:50:37]

Done.

+    ],
+    "name": "gsutil upload",
+    "~followup_annotations": [
+      "@@@STEP_LINK@gsutil.upload@https://storage.googleapis.com/chromium-webrtc/Android32_Builder/AppRTCDemo_apk_170242.zip@@@"
+    ]
+  },
+  {
     "cmd": [],
     "name": "trigger",
     "trigger_specs": [
       {
         "builder_name": "Android32 Tests (L Nexus5)",
         "properties": {
           "parent_got_revision": "f27fede2220bcd326aee3e86ddfd4ebd0fe58cb9",
           "parent_got_revision_cp": "refs/heads/master@{#170242}",
           "revision": "f27fede2220bcd326aee3e86ddfd4ebd0fe58cb9"
         }
@@ skipping 10 matching lines @@
     "~followup_annotations": [
       "@@@STEP_TEXT@Android32 Tests (L Nexus5)<br />Android32 Tests (L Nexus7.2)@@@"
     ]
   },
   {
     "name": "$result",
     "recipe_result": null,
     "status_code": 0
   }
 ]