Skip to content

Commit

Permalink
Merge pull request #1412 from consideRatio/pr/fix-mistake-in-arm-pr-3
Browse files Browse the repository at this point in the history
fix: docker's -t flag expects a full image name
  • Loading branch information
consideRatio authored Jul 17, 2021
2 parents 40b549a + 84cfc8b commit ecabcb8
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion tagging/tag_image.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,9 @@ def tag_image(short_image_name: str, owner: str) -> None:

if tags:
env_name = f'{short_image_name.replace("-", "_")}_EXTRA_TAG_ARGS'
docker_build_tag_args = "-t " + " -t ".join(tags)
docker_build_tag_args = " ".join(
[f"-t {owner}/{short_image_name}:{tag}" for tag in tags]
)
github_set_env(env_name, docker_build_tag_args)


Expand Down

0 comments on commit ecabcb8

Please sign in to comment.