Skip to content

Commit 3d3b3e1

Browse files
saiarcot895mssonicbld
authored andcommitted
[build]: Fix docker load image tag not being the expected tag (#12959)
PR #12829 modified the docker tagging scheme such that optional docker containers would be tagged with the SONiC image version. However, the docker-image-load macro wasn't updated for these changes. Update it here. Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
1 parent eabd881 commit 3d3b3e1

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

slave.mk

+3-3
Original file line numberDiff line numberDiff line change
@@ -497,11 +497,11 @@ define docker-image-load
497497
@echo "Obtained docker image lock for $(1) load" $(LOG)
498498
@echo "Loading docker image $(TARGET_PATH)/$(1).gz" $(LOG)
499499
docker load -i $(TARGET_PATH)/$(1).gz $(LOG)
500-
@echo "Tagging docker image $(1):latest as $(1)-$(DOCKER_USERNAME):$(DOCKER_USERTAG)" $(LOG)
501-
docker tag $(1):latest $(1)-$(DOCKER_USERNAME):$(DOCKER_USERTAG) $(LOG)
500+
@echo "Tagging docker image $(1):$(call docker-get-tag,$(1)) as $(1)-$(DOCKER_USERNAME):$(DOCKER_USERTAG)" $(LOG)
501+
docker tag $(1):$(call docker-get-tag,$(1)) $(1)-$(DOCKER_USERNAME):$(DOCKER_USERTAG) $(LOG)
502502
if [ x$(SONIC_CONFIG_USE_NATIVE_DOCKERD_FOR_BUILD) == x"y" ]; then
503503
@echo "Removing docker image $(1):latest" $(LOG)
504-
docker rmi -f $(1):latest $(LOG)
504+
docker rmi -f $(1):$(call docker-get-tag,$(1)) $(LOG)
505505
fi
506506
$(call MOD_UNLOCK,$(1))
507507
@echo "Released docker image lock for $(1) load" $(LOG)

0 commit comments

Comments
 (0)