Support network service apptests in the upload_service script

mojo/services/upload_service.py

 #!/usr/bin/env python
 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import argparse
 import imp
 import os
 import subprocess
 import sys
 import tempfile
 import time
 import zipfile
 
-# A service's name is defined as the name of its subdirectory in the directory
-# containing this file.
-SERVICES = [ "network", "html_viewer" ]
+SERVICES = ["html_viewer", "network_service", "network_service_apptests"]
 
-SERVICE_BINARY_NAMES = {
-    "network" : "network_service.mojo",
-    "html_viewer" : "html_viewer.mojo"
+# A service does not need to expose interfaces. Those that do expose interfaces
+# have their mojoms located in the directories listed below, in paths relative
+# to the directory of this script.
+MOJOMS_IN_DIR = {
+    "network_service": os.path.join("network", "public", "interfaces")
 }
 
 # The network service is downloaded out-of-band rather than dynamically by the
 # shell and thus can be stored zipped in the cloud. Other services are intended
 # to be downloaded dynamically by the shell, which doesn't currently understand
 # zipped binaries.
-SERVICES_WITH_ZIPPED_BINARIES = [ "network" ]
+SERVICES_WITH_ZIPPED_BINARIES = ["network_service", "network_service_apptests"]
 
 if not sys.platform.startswith("linux"):
   print "Only support linux for now"
   sys.exit(1)
 
 root_path = os.path.realpath(
     os.path.join(
         os.path.dirname(
             os.path.realpath(__file__)),
         os.pardir,
         os.pardir))
 version = subprocess.check_output(["git", "rev-parse", "HEAD"], cwd=root_path)
 version = version.strip()
 
 find_depot_tools_path = os.path.join(root_path, "tools", "find_depot_tools.py")
 find_depot_tools = imp.load_source("find_depot_tools", find_depot_tools_path)
 
 depot_tools_path = find_depot_tools.add_depot_tools_to_path()
 gsutil_exe = os.path.join(depot_tools_path, "third_party", "gsutil", "gsutil")
 
 
 def gsutil_cp(source, dest, dry_run):
   if dry_run:
     print "gsutil cp %s %s" % (source, dest)
   else:
     subprocess.check_call([gsutil_exe, "cp", source, dest])
 
 
-def upload_mojoms(service, dry_run):
-  script_dir = os.path.dirname(os.path.realpath(__file__))
-  service_dir = os.path.join(script_dir, service)
-  absolute_mojom_directory_path = os.path.join(
-      service_dir,
-      "public",
-      "interfaces")
+def upload_mojoms(service, absolute_mojom_directory_path, dry_run):
+  dest = "gs://mojo/" + service + "/" + version + "/" + "mojoms.zip"
 
-  if not os.path.exists(absolute_mojom_directory_path):
-    # This service has no interfaces.
-    return
-
-  dest = "gs://mojo/" + service + "/" + version + "/" + "mojoms.zip"
   with tempfile.NamedTemporaryFile() as mojom_zip_file:
     with zipfile.ZipFile(mojom_zip_file, 'w') as z:
       for root, _, files in os.walk(absolute_mojom_directory_path):
         for filename in files:
           absolute_file_path = os.path.join(root, filename)
           relative_file_path = os.path.relpath(absolute_file_path, root)
           z.write(absolute_file_path, relative_file_path)
     gsutil_cp(mojom_zip_file.name, dest, dry_run)
 
 
 def upload_binary(service, binary_dir, platform, dry_run):
-  binary_name = SERVICE_BINARY_NAMES[service]
+  dest_dir = "gs://mojo/" + service + "/" + version + "/" + platform + "/"
+  should_zip = service in SERVICES_WITH_ZIPPED_BINARIES
+  binary_name = service + ".mojo"
   absolute_binary_path = os.path.join(root_path, binary_dir, binary_name)
-  binary_dest_prefix = "gs://mojo/" + service + "/" + version + "/" + platform
 
-  if service not in SERVICES_WITH_ZIPPED_BINARIES:
-    binary_dest = binary_dest_prefix + "/" + binary_name
-    gsutil_cp(absolute_binary_path, binary_dest, dry_run)
+  if not should_zip:
+    dest = dest_dir + binary_name
+    gsutil_cp(absolute_binary_path, dest, dry_run)
     return
 
   # Zip the binary before uploading it to the cloud.
-  binary_dest = binary_dest_prefix + ".zip"
+  dest = dest_dir + binary_name + ".zip"

[blundell, 2015/02/06 13:10:44]

nit: You can just specify dest with the "+ binary name" at line 71, and then add
the ".zip" extension here.

[ppi, 2015/02/06 13:16:34]

Unless you feel strongly, I'd prefer not to have "dest" mean full and actual
destination for not zipped binaries and only a prefix of the destination path
for zipped binaries. I like that dest_dir means the same in both cases.

   with tempfile.NamedTemporaryFile() as binary_zip_file:
     with zipfile.ZipFile(binary_zip_file, 'w') as z:
       with open(absolute_binary_path) as service_binary:
         zipinfo = zipfile.ZipInfo(binary_name)
         zipinfo.external_attr = 0o777 << 16
         zipinfo.compress_type = zipfile.ZIP_DEFLATED
         zipinfo.date_time = time.gmtime(os.path.getmtime(absolute_binary_path))
         z.writestr(zipinfo, service_binary.read())
-    gsutil_cp(binary_zip_file.name, binary_dest, dry_run)
+    gsutil_cp(binary_zip_file.name, dest, dry_run)
 
 
 def main():
   parser = argparse.ArgumentParser(
       description="Upload service mojoms and binaries to Google storage")
   parser.add_argument("-n", "--dry-run", action="store_true", help="Dry run")
   parser.add_argument(
       "--linux-x64-binary-dir",
       help="Path to the dir containing the linux-x64 service binary relative "
            "to the repo root, e.g. out/Release")
   parser.add_argument(
       "--android-arm-binary-dir",
       help="Path to the dir containing the android-arm service binary relative "
            "to the repo root, e.g. out/android_Release")
   parser.add_argument("service",
                       help="The service to be uploaded (one of %s)" % SERVICES)
 
   args = parser.parse_args()
 
   if args.service not in SERVICES:
     print args.service + " is not one of the recognized services:"
     print SERVICES
     return 1
 
-  upload_mojoms(args.service, args.dry_run)
+  if args.service in MOJOMS_IN_DIR:
+    script_dir = os.path.dirname(os.path.realpath(__file__))
+    absolute_mojom_directory_path = os.path.join(script_dir,
+                                                 MOJOMS_IN_DIR[args.service])
+    upload_mojoms(args.service, absolute_mojom_directory_path, args.dry_run)
+
   if args.linux_x64_binary_dir:
     upload_binary(args.service, args.linux_x64_binary_dir,
                   "linux-x64", args.dry_run)
+
   if args.android_arm_binary_dir:
     upload_binary(args.service, args.android_arm_binary_dir,
                   "android-arm", args.dry_run)
 
   if not args.dry_run:
     print "Uploaded artifacts for version %s" % (version, )
   else:
     print "No artifacts uploaded (dry run)"
   return 0
 
 if __name__ == '__main__':
   sys.exit(main())