From e3574867077979f7e50cc54175786baa7bde2c47 Mon Sep 17 00:00:00 2001 From: Edward Chen <18449977+edgchen1@users.noreply.github.com> Date: Tue, 8 Dec 2020 15:16:50 -0800 Subject: [PATCH] Fix build definition template typo, add logging (#6065) Fix a typo in tools/ci_build/github/azure-pipelines/templates/get-docker-image-steps.yml. Add logging to tools/ci_build/get_docker_image.py for easier debugging. --- tools/ci_build/get_docker_image.py | 3 +++ .../azure-pipelines/templates/get-docker-image-steps.yml | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/tools/ci_build/get_docker_image.py b/tools/ci_build/get_docker_image.py index 9ef141c30e..68ac4a7a9c 100755 --- a/tools/ci_build/get_docker_image.py +++ b/tools/ci_build/get_docker_image.py @@ -131,6 +131,9 @@ def container_registry_has_image(full_image_name, docker_path): def main(): args = parse_args() + log.debug("Dockerfile: {}, context: {}, docker build args: '{}'".format( + args.dockerfile, args.context, args.docker_build_args)) + tag = generate_tag(args.dockerfile, args.context, args.docker_build_args) full_image_name = "{}.azurecr.io/{}:{}".format( diff --git a/tools/ci_build/github/azure-pipelines/templates/get-docker-image-steps.yml b/tools/ci_build/github/azure-pipelines/templates/get-docker-image-steps.yml index 4420a24a86..dbc561ef17 100644 --- a/tools/ci_build/github/azure-pipelines/templates/get-docker-image-steps.yml +++ b/tools/ci_build/github/azure-pipelines/templates/get-docker-image-steps.yml @@ -19,7 +19,7 @@ steps: tools/ci_build/get_docker_image.py \ --dockerfile "${{ parameters.Dockerfile }}" \ --context "${{ parameters.Context }}" \ - --docker-build-args "${{ variables.DockerBuildArgs }}" \ + --docker-build-args "${{ parameters.DockerBuildArgs }}" \ --container-registry onnxruntimebuildcache \ --repository "${{ parameters.Repository }}" displayName: "Get ${{ parameters.Repository }} image"