Skip to content

Commit ec4c6e2

Browse files
Merge pull request #298 from caseydavenport/fix-release-issues
Fix release issues
2 parents e5c1ed7 + dd5d24e commit ec4c6e2

File tree

1 file changed

+4
-10
lines changed

1 file changed

+4
-10
lines changed

Makefile

+4-10
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@ comma := ,
5353
prefix_linux = $(addprefix linux/,$(strip $1))
5454
join_platforms = $(subst $(space),$(comma),$(call prefix_linux,$(strip $1)))
5555

56-
5756
# list of arches *not* to build when doing *-all
5857
# until s390x works correctly
5958
EXCLUDEARCH ?= s390x
@@ -240,7 +239,6 @@ else
240239
$(NOECHO) $(NOOP)
241240
endif
242241

243-
244242
## tag images of one arch
245243
tag-images: imagetag $(addprefix sub-single-tag-images-arch-,$(call escapefs,$(PUSH_IMAGES))) $(addprefix sub-single-tag-images-non-manifest-,$(call escapefs,$(PUSH_NONMANIFEST_IMAGES)))
246244
sub-single-tag-images-arch-%:
@@ -254,14 +252,11 @@ else
254252
$(NOECHO) $(NOOP)
255253
endif
256254

257-
258-
259255
## tag images of all archs
260256
tag-images-all: imagetag $(addprefix sub-tag-images-,$(VALIDARCHES))
261257
sub-tag-images-%:
262258
$(MAKE) tag-images ARCH=$* IMAGETAG=$(IMAGETAG)
263259

264-
265260
###############################################################################
266261
# Static checks
267262
###############################################################################
@@ -376,8 +371,8 @@ endif
376371
## Verifies the release artifacts produces by `make release-build` are correct.
377372
release-verify: release-prereqs
378373
# Check the reported version is correct for each release artifact.
379-
if ! docker run $(BUILD_IMAGE):$(VERSION) -v | grep '^$(VERSION)$$'; then echo "Reported version:" `docker run $(BUILD_IMAGE):$(VERSION) -v` "\nExpected version: $(VERSION)"; false; else echo "\nVersion check passed\n"; fi
380-
if ! docker run quay.io/$(BUILD_IMAGE):$(VERSION) -v | grep '^$(VERSION)$$'; then echo "Reported version:" `docker run quay.io/$(BUILD_IMAGE):$(VERSION) -v` "\nExpected version: $(VERSION)"; false; else echo "\nVersion check passed\n"; fi
374+
if ! docker run $(BUILD_IMAGE):$(VERSION)-$(ARCH) -v | grep '^$(VERSION)$$'; then echo "Reported version:" `docker run $(BUILD_IMAGE):$(VERSION)-$(ARCH) -v` "\nExpected version: $(VERSION)"; false; else echo "\nVersion check passed\n"; fi
375+
if ! docker run quay.io/$(BUILD_IMAGE):$(VERSION)-$(ARCH) -v | grep '^$(VERSION)$$'; then echo "Reported version:" `docker run quay.io/$(BUILD_IMAGE):$(VERSION)-$(ARCH) -v` "\nExpected version: $(VERSION)"; false; else echo "\nVersion check passed\n"; fi
381376

382377
## Generates release notes based on commits in this version.
383378
release-notes: release-prereqs
@@ -391,7 +386,7 @@ release-publish: release-prereqs
391386
git push origin $(VERSION)
392387

393388
# Push images.
394-
$(MAKE) push-all IMAGETAG=$(VERSION)
389+
$(MAKE) push-all push-manifests push-non-manifests IMAGETAG=$(VERSION)
395390

396391
@echo "Finalize the GitHub release based on the pushed tag."
397392
@echo ""
@@ -410,15 +405,14 @@ release-publish-latest: release-prereqs
410405
if ! docker run $(BUILD_IMAGE):latest -v | grep '^$(VERSION)$$'; then echo "Reported version:" `docker run $(BUILD_IMAGE):latest -v` "\nExpected version: $(VERSION)"; false; else echo "\nVersion check passed\n"; fi
411406
if ! docker run quay.io/$(BUILD_IMAGE):latest -v | grep '^$(VERSION)$$'; then echo "Reported version:" `docker run quay.io/$(BUILD_IMAGE):latest -v` "\nExpected version: $(VERSION)"; false; else echo "\nVersion check passed\n"; fi
412407

413-
$(MAKE) push-all IMAGETAG=latest
408+
$(MAKE) push-all push-manifests push-non-manifests IMAGETAG=latest
414409

415410
# release-prereqs checks that the environment is configured properly to create a release.
416411
release-prereqs:
417412
ifndef VERSION
418413
$(error VERSION is undefined - run using make release VERSION=vX.Y.Z)
419414
endif
420415

421-
422416
###############################################################################
423417
# Developer helper scripts (not used by build or test)
424418
###############################################################################

0 commit comments

Comments
 (0)