[Skia] Convert SKPs to a CIPD package

scripts/slave/skia/upload_skps.py

Index: scripts/slave/skia/upload_skps.py
diff --git a/scripts/slave/skia/recreate_skps.py b/scripts/slave/skia/upload_skps.py
similarity index 67%
rename from scripts/slave/skia/recreate_skps.py
rename to scripts/slave/skia/upload_skps.py
index 25f636f34c8ddb75539495a32e15b32651050283..0900a71cf040b32da21ed949de8159a9153d90e2 100644
--- a/scripts/slave/skia/recreate_skps.py
+++ b/scripts/slave/skia/upload_skps.py
@@ -29,10 +29,7 @@ SKIA_COMMITTER_NAME = 'skia.buildbots'
 SKIA_REPO = 'https://skia.googlesource.com/skia.git'
 
 
-def main(chrome_src_path, browser_executable, dry_run=False):
-  if dry_run:
-    print 'Not committing results since --dry-run was provided'
-
+def main(target_dir):
   subprocess.check_call(['git', 'config', '--local', 'user.name',
                          SKIA_COMMITTER_NAME])
   subprocess.check_call(['git', 'config', '--local', 'user.email',
@@ -41,18 +38,24 @@ def main(chrome_src_path, browser_executable, dry_run=False):
     subprocess.check_call(['git', 'remote', 'set-url', 'origin', SKIA_REPO,
                            CHROMIUM_SKIA])
 
+  # Download CIPD.
+  cipd_sha1 = os.path.join(os.getcwd(), 'infra', 'bots', 'tools', 'luci-go',
+                           'linux64', 'cipd.sha1')
+  subprocess.check_call(['download_from_google_storage', '-s', cipd_sha1,
+                         '--bucket', 'chromium-luci'])
+
   with git_utils.GitBranch(branch_name='update_skp_version',
                            commit_msg=COMMIT_MSG,
-                           commit_queue=not dry_run):
-    subprocess.check_call(['python', os.path.join('tools', 'skp',
-                                                  'recreate_skps.py'),
-                           chrome_src_path, browser_executable, str(dry_run)])
+                           commit_queue=True):
+    upload_script = os.path.join(
+        os.getcwd(), 'infra', 'bots', 'assets', 'skp', 'upload.py')
+    subprocess.check_call(['python', upload_script, '-t', target_dir])
+
+  # TODO(borenet): Upload to partner GS bucket?

[rmistry, 2016/07/22 14:26:15]

Is this not being done? I thought it would be done by webpages_playback.py ?

[borenet, 2016/07/22 14:49:53]

I removed the flag which caused that to occur in
https://codereview.chromium.org/2167763002/diff/60001/infra/bots/assets/skp/c...
.  Maybe it should be added back there?

 
 
 if '__main__' == __name__:
   parser = argparse.ArgumentParser()
-  parser.add_argument("chrome_src_path")
-  parser.add_argument("browser_executable")
-  parser.add_argument("--dry-run", action="store_true")
+  parser.add_argument("--target_dir")
   args = parser.parse_args()
-  main(args.chrome_src_path, args.browser_executable, args.dry_run)
+  main(args.target_dir)