X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/308a6da1a9fd716f3957b116110a932c08aefafe..f4c3100bad26dff3c99ff4bb9fa19b0d9f7995c7:/sdk/python/arvados/commands/run.py diff --git a/sdk/python/arvados/commands/run.py b/sdk/python/arvados/commands/run.py index f2bf0f353b..8815565460 100644 --- a/sdk/python/arvados/commands/run.py +++ b/sdk/python/arvados/commands/run.py @@ -14,11 +14,12 @@ import logging import arvados.commands._util as arv_cmd logger = logging.getLogger('arvados.arv-run') +logger.setLevel(logging.INFO) arvrun_parser = argparse.ArgumentParser(parents=[arv_cmd.retry_opt]) arvrun_parser.add_argument('--dry-run', action="store_true", help="Print out the pipeline that would be submitted and exit") arvrun_parser.add_argument('--local', action="store_true", help="Run locally using arv-run-pipeline-instance") -arvrun_parser.add_argument('--docker-image', type=str, default="arvados/jobs", help="Docker image to use, default arvados/jobs") +arvrun_parser.add_argument('--docker-image', type=str, help="Docker image to use, otherwise use instance default.") arvrun_parser.add_argument('--ignore-rcode', action="store_true", help="Commands that return non-zero return codes should not be considered failed.") arvrun_parser.add_argument('--no-reuse', action="store_true", help="Do not reuse past jobs.") arvrun_parser.add_argument('--no-wait', action="store_true", help="Do not wait and display logs after submitting command, just exit.") @@ -51,7 +52,7 @@ def is_in_collection(root, branch): else: sp = os.path.split(root) return is_in_collection(sp[0], os.path.join(sp[1], branch)) - except IOError, OSError: + except (IOError, OSError): return (None, None) # Determine the project to place the output of this command by searching upward @@ -72,7 +73,7 @@ def determine_project(root, current_user): else: sp = os.path.split(root) return determine_project(sp[0], current_user) - except IOError, OSError: + except (IOError, OSError): return current_user # Determine if string corresponds to a file, and if that file is part of a @@ -100,6 +101,63 @@ def statfile(prefix, fn): return prefix+fn +def uploadfiles(files, api, dry_run=False, num_retries=0, project=None, fnPattern="$(file %s/%s)"): + # Find the smallest path prefix that includes all the files that need to be uploaded. + # This starts at the root and iteratively removes common parent directory prefixes + # until all file pathes no longer have a common parent. + n = True + pathprefix = "/" + while n: + pathstep = None + for c in files: + if pathstep is None: + sp = c.fn.split('/') + if len(sp) < 2: + # no parent directories left + n = False + break + # path step takes next directory + pathstep = sp[0] + "/" + else: + # check if pathstep is common prefix for all files + if not c.fn.startswith(pathstep): + n = False + break + if n: + # pathstep is common parent directory for all files, so remove the prefix + # from each path + pathprefix += pathstep + for c in files: + c.fn = c.fn[len(pathstep):] + + orgdir = os.getcwd() + os.chdir(pathprefix) + + logger.info("Upload local files: \"%s\"", '" "'.join([c.fn for c in files])) + + if dry_run: + logger.info("$(input) is %s", pathprefix.rstrip('/')) + pdh = "$(input)" + else: + files = sorted(files, key=lambda x: x.fn) + collection = arvados.CollectionWriter(api, num_retries=num_retries) + stream = None + for f in files: + sp = os.path.split(f.fn) + if sp[0] != stream: + stream = sp[0] + collection.start_new_stream(stream) + collection.write_file(f.fn, sp[1]) + item = api.collections().create(body={"owner_uuid": project, "manifest_text": collection.manifest_text()}).execute() + pdh = item["portable_data_hash"] + logger.info("Uploaded to %s", item["uuid"]) + + for c in files: + c.fn = fnPattern % (pdh, c.fn) + + os.chdir(orgdir) + + def main(arguments=None): args = arvrun_parser.parse_args(arguments) @@ -178,62 +236,9 @@ def main(arguments=None): command[i] = statfile(m.group(1), m.group(2)) break - n = True - pathprefix = "/" files = [c for command in slots[1:] for c in command if isinstance(c, UploadFile)] - if len(files) > 0: - # Find the smallest path prefix that includes all the files that need to be uploaded. - # This starts at the root and iteratively removes common parent directory prefixes - # until all file pathes no longer have a common parent. - while n: - pathstep = None - for c in files: - if pathstep is None: - sp = c.fn.split('/') - if len(sp) < 2: - # no parent directories left - n = False - break - # path step takes next directory - pathstep = sp[0] + "/" - else: - # check if pathstep is common prefix for all files - if not c.fn.startswith(pathstep): - n = False - break - if n: - # pathstep is common parent directory for all files, so remove the prefix - # from each path - pathprefix += pathstep - for c in files: - c.fn = c.fn[len(pathstep):] - - orgdir = os.getcwd() - os.chdir(pathprefix) - - print("Upload local files: \"%s\"" % '" "'.join([c.fn for c in files])) - - if args.dry_run: - print("$(input) is %s" % pathprefix.rstrip('/')) - pdh = "$(input)" - else: - files = sorted(files, key=lambda x: x.fn) - collection = arvados.CollectionWriter(api, num_retries=args.retries) - stream = None - for f in files: - sp = os.path.split(f.fn) - if sp[0] != stream: - stream = sp[0] - collection.start_new_stream(stream) - collection.write_file(f.fn, sp[1]) - item = api.collections().create(body={"owner_uuid": project, "manifest_text": collection.manifest_text()}).execute() - pdh = item["portable_data_hash"] - print "Uploaded to %s" % item["uuid"] - - for c in files: - c.fn = "$(file %s/%s)" % (pdh, c.fn) - - os.chdir(orgdir) + if files: + uploadfiles(files, api, dry_run=args.dry_run, num_retries=args.num_retries, project=project) for i in xrange(1, len(slots)): slots[i] = [("%s%s" % (c.prefix, c.fn)) if isinstance(c, ArvFile) else c for c in slots[i]] @@ -244,11 +249,12 @@ def main(arguments=None): "repository": args.repository, "script_parameters": { }, - "runtime_constraints": { - "docker_image": args.docker_image - } + "runtime_constraints": {} } + if args.docker_image: + component["runtime_constraints"]["docker_image"] = args.docker_image + task_foreach = [] group_parser = argparse.ArgumentParser() group_parser.add_argument('-b', '--batch-size', type=int) @@ -298,7 +304,7 @@ def main(arguments=None): else: pipeline["owner_uuid"] = project pi = api.pipeline_instances().create(body=pipeline, ensure_unique_name=True).execute() - print "Running pipeline %s" % pi["uuid"] + logger.info("Running pipeline %s", pi["uuid"]) if args.local: subprocess.call(["arv-run-pipeline-instance", "--instance", pi["uuid"], "--run-jobs-here"] + (["--no-reuse"] if args.no_reuse else [])) @@ -306,11 +312,11 @@ def main(arguments=None): ws.main(["--pipeline", pi["uuid"]]) pi = api.pipeline_instances().get(uuid=pi["uuid"]).execute() - print "Pipeline is %s" % pi["state"] + logger.info("Pipeline is %s", pi["state"]) if "output_uuid" in pi["components"]["command"]: - print "Output is %s" % pi["components"]["command"]["output_uuid"] + logger.info("Output is %s", pi["components"]["command"]["output_uuid"]) else: - print "No output" + logger.info("No output") if __name__ == '__main__': main()