Skip to content

Commit b69477e

Browse files
Ensure image version consistency (#426)
1 parent 10cbfea commit b69477e

File tree

2 files changed

+15
-24
lines changed

2 files changed

+15
-24
lines changed

Makefile

Lines changed: 4 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -20,18 +20,6 @@
2020
all: clean build
2121

2222
# Tag parameters
23-
ifndef MAJOR
24-
MAJOR=2
25-
endif
26-
ifndef MINOR
27-
MINOR=13
28-
endif
29-
ifndef PATCH
30-
PATCH=0
31-
endif
32-
ifndef NOTES
33-
NOTES=
34-
endif
3523
ifndef TAGMSG
3624
TAGMSG="CSI Spec 1.6"
3725
endif
@@ -51,18 +39,18 @@ install:
5139

5240
# Tags the release with the Tag parameters set above
5341
tag:
54-
-git tag -d v$(MAJOR).$(MINOR).$(PATCH)$(NOTES)
55-
git tag -a -m $(TAGMSG) v$(MAJOR).$(MINOR).$(PATCH)$(NOTES)
42+
go run core/semver/semver.go -f mk >semver.mk
43+
make -f docker.mk tag TAGMSG='$(TAGMSG)'
5644

5745
# Generates the docker container (but does not push)
5846
docker:
5947
go run core/semver/semver.go -f mk >semver.mk
60-
make -f docker.mk DOCKER_FILE=Dockerfile docker
48+
make -f docker.mk docker
6149

6250
# Same as `docker` but without cached layers and will pull latest version of base image
6351
docker-no-cache:
6452
go run core/semver/semver.go -f mk >semver.mk
65-
make -f docker.mk DOCKER_FILE=Dockerfile docker-no-cache
53+
make -f docker.mk docker-no-cache
6654

6755
# Pushes container to the repository
6856
push: docker

docker.mk

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,10 @@ else
2525
RELNOTE=
2626
endif
2727

28+
ifeq ($(IMAGETAG),)
29+
IMAGETAG=v$(MAJOR).$(MINOR).$(PATCH)$(RELNOTE)
30+
endif
31+
2832
ifndef DOCKER_REGISTRY
2933
DOCKER_REGISTRY=dellemc
3034
endif
@@ -45,19 +49,18 @@ endif
4549

4650
docker: download-csm-common
4751
$(eval include csm-common.mk)
48-
@echo "MAJOR $(MAJOR) MINOR $(MINOR) PATCH $(PATCH) RELNOTE $(RELNOTE) SEMVER $(SEMVER)"
49-
@echo "$(DOCKER_FILE)"
50-
$(BUILDER) build --pull -f $(DOCKER_FILE) -t "$(DOCKER_REGISTRY)/$(DOCKER_IMAGE_NAME):v$(MAJOR).$(MINOR).$(PATCH)$(RELNOTE)" --build-arg GOIMAGE=$(DEFAULT_GOIMAGE) --build-arg BASEIMAGE=$(CSM_BASEIMAGE) .
52+
$(BUILDER) build --pull -t "$(DOCKER_REGISTRY)/$(DOCKER_IMAGE_NAME):$(IMAGETAG)" --build-arg GOIMAGE=$(DEFAULT_GOIMAGE) --build-arg BASEIMAGE=$(CSM_BASEIMAGE) .
5153

5254
docker-no-cache: download-csm-common
5355
$(eval include csm-common.mk)
54-
@echo "MAJOR $(MAJOR) MINOR $(MINOR) PATCH $(PATCH) RELNOTE $(RELNOTE) SEMVER $(SEMVER)"
55-
@echo "$(DOCKER_FILE) --no-cache"
56-
$(BUILDER) build --pull --no-cache -f $(DOCKER_FILE) -t "$(DOCKER_REGISTRY)/$(DOCKER_IMAGE_NAME):v$(MAJOR).$(MINOR).$(PATCH)$(RELNOTE)" --build-arg GOIMAGE=$(DEFAULT_GOIMAGE) --build-arg BASEIMAGE=$(CSM_BASEIMAGE) .
56+
$(BUILDER) build --pull --no-cache -t "$(DOCKER_REGISTRY)/$(DOCKER_IMAGE_NAME):$(IMAGETAG)" --build-arg GOIMAGE=$(DEFAULT_GOIMAGE) --build-arg BASEIMAGE=$(CSM_BASEIMAGE) .
5757

5858
push:
59-
echo "MAJOR $(MAJOR) MINOR $(MINOR) PATCH $(PATCH) RELNOTE $(RELNOTE) SEMVER $(SEMVER)"
60-
$(BUILDER) push "$(DOCKER_REGISTRY)/$(DOCKER_IMAGE_NAME):v$(MAJOR).$(MINOR).$(PATCH)$(RELNOTE)"
59+
$(BUILDER) push "$(DOCKER_REGISTRY)/$(DOCKER_IMAGE_NAME):$(IMAGETAG)"
6160

6261
download-csm-common:
6362
curl -O -L https://raw.githubusercontent.com/dell/csm/main/config/csm-common.mk
63+
64+
tag:
65+
-git tag -d $(IMAGETAG)
66+
git tag -a -m $(TAGMSG) $(IMAGETAG)

0 commit comments

Comments
 (0)