Skip to content

Commit 67dd549

Browse files
committed
Fix args order when running python modules
1 parent 1c32064 commit 67dd549

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

.github/workflows/docker-tag-push.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ jobs:
6969
name: ${{ inputs.image }}-${{ inputs.platform }}-${{ inputs.variant }}-tags
7070
path: /tmp/jupyter/tags/
7171
- name: Apply tags to the loaded image 🏷
72-
run: python3 -m tagging.apply_tags --short-image-name ${{ inputs.image }} --tags-dir /tmp/jupyter/tags/ --platform ${{ inputs.platform }} --variant ${{ inputs.variant }} --registry ${{ env.REGISTRY }} --owner ${{ env.OWNER }}
72+
run: python3 -m tagging.apply_tags --short-image-name ${{ inputs.image }} --tags-dir /tmp/jupyter/tags/ --platform ${{ inputs.platform }} --registry ${{ env.REGISTRY }} --owner ${{ env.OWNER }} --variant ${{ inputs.variant }}
7373
# This step is needed to prevent pushing non-multi-arch "latest" tag
7474
- name: Remove the "latest" tag from the image 🗑️
7575
run: docker image rmi ${{ env.REGISTRY }}/${{ env.OWNER }}/${{ inputs.image }}:latest

tagging/merge_tags.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -66,17 +66,17 @@ def merge_tags(
6666
required=True,
6767
help="Short image name",
6868
)
69-
arg_parser.add_argument(
70-
"--variant",
71-
required=True,
72-
help="Variant tag prefix",
73-
)
7469
arg_parser.add_argument(
7570
"--tags-dir",
7671
required=True,
7772
type=Path,
7873
help="Directory with saved tags file",
7974
)
75+
arg_parser.add_argument(
76+
"--variant",
77+
required=True,
78+
help="Variant tag prefix",
79+
)
8080
args = arg_parser.parse_args()
8181

8282
merge_tags(args.short_image_name, args.variant, args.tags_dir)

0 commit comments

Comments
 (0)