Skip to content

Commit 527cb04

Browse files
committed
Fix help messages for Makefile targets
1 parent 4eb0f5b commit 527cb04

File tree

1 file changed

+26
-26
lines changed

1 file changed

+26
-26
lines changed

Makefile

Lines changed: 26 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -249,8 +249,8 @@ image: build-all-ccm-images build-all-node-images ## Build all images.
249249
.PHONY: push
250250
push: push-multi-arch-controller-manager-image push-multi-arch-node-manager-image ## Push all images.
251251

252-
.PHONY: push-multi-arch-controller-manager-image ## Push multi-arch controller-manager image
253-
push-multi-arch-controller-manager-image: push-all-ccm-images ## Create and push a manifest list containing all the Linux ccm images.
252+
.PHONY: push-multi-arch-controller-manager-image
253+
push-multi-arch-controller-manager-image: push-all-ccm-images ## Push multi-arch controller-manager image
254254
## Linux amd64 ccm image name has no amd64
255255
docker tag $(CONTROLLER_MANAGER_FULL_IMAGE_NAME):$(IMAGE_TAG) $(CONTROLLER_MANAGER_FULL_IMAGE_NAME)-amd64:$(IMAGE_TAG)
256256
docker push $(CONTROLLER_MANAGER_FULL_IMAGE_NAME)-amd64:$(IMAGE_TAG)
@@ -261,8 +261,8 @@ push-multi-arch-controller-manager-image: push-all-ccm-images ## Create and push
261261
done
262262
docker manifest push --purge $(CONTROLLER_MANAGER_IMAGE)
263263

264-
.PHONY: push-multi-arch-node-manager-image ## Push multi-arch node-manager image
265-
push-multi-arch-node-manager-image: push-all-node-images ## Create and push a manifest list containing all the Windows and Linux images.
264+
.PHONY: push-multi-arch-node-manager-image
265+
push-multi-arch-node-manager-image: push-all-node-images ## Push multi-arch node-manager image
266266
docker manifest create --amend $(NODE_MANAGER_IMAGE) $(ALL_NODE_MANAGER_IMAGES)
267267
for arch in $(ALL_ARCH.linux); do \
268268
docker manifest annotate --os linux --arch $${arch} $(NODE_MANAGER_IMAGE) $(NODE_MANAGER_LINUX_FULL_IMAGE_PREFIX)-$${arch}; \
@@ -278,17 +278,17 @@ push-multi-arch-node-manager-image: push-all-node-images ## Create and push a ma
278278
done
279279
docker manifest push --purge $(NODE_MANAGER_IMAGE)
280280

281-
.PHONY: push-all-node-images ## Push node-manager image for os and archs.
282-
push-all-node-images: push-all-node-images-linux push-all-node-images-windows
281+
.PHONY: push-all-node-images
282+
push-all-node-images: push-all-node-images-linux push-all-node-images-windows ## Push node-manager image for os and archs.
283283

284-
.PHONY: push-all-node-images-linux ## Push node-manager image for Linux.
285-
push-all-node-images-linux: $(addprefix push-node-image-linux-,$(ALL_ARCH.linux))
284+
.PHONY: push-all-node-images-linux
285+
push-all-node-images-linux: $(addprefix push-node-image-linux-,$(ALL_ARCH.linux)) ## Push node-manager image for Linux.
286286

287-
.PHONY: push-all-node-images-windows ## Push node-manager image for Windows.
288-
push-all-node-images-windows: $(addprefix push-node-image-windows-,$(ALL_OS_ARCH.windows))
287+
.PHONY: push-all-node-images-windows
288+
push-all-node-images-windows: $(addprefix push-node-image-windows-,$(ALL_OS_ARCH.windows)) ## Push node-manager image for Windows.
289289

290-
.PHONY: push-all-node-images-windows-hpc ## Push node-manager image for Windows.
291-
push-all-node-images-windows-hpc: $(addprefix push-node-images-windows-hpc-,$(ALL_OS_ARCH.windows))
290+
.PHONY: push-all-node-images-windows-hpc
291+
push-all-node-images-windows-hpc: $(addprefix push-node-images-windows-hpc-,$(ALL_OS_ARCH.windows)) ## Push node-manager image for Windows.
292292

293293
# split words on hyphen, access by 1-index
294294
word-hyphen = $(word $2,$(subst -, ,$1))
@@ -302,14 +302,14 @@ push-node-image-windows-%:
302302
push-node-image-windows-hpc-%:
303303
$(MAKE) ARCH=$(call word-hyphen,$*,1) OUTPUT_TYPE=registry build-node-image-windows-hpc
304304

305-
.PHONY: build-all-node-images ## Build node-manager image for all OS and archs.
306-
build-all-node-images: build-all-node-images-linux build-all-node-images-windows
305+
.PHONY: build-all-node-images
306+
build-all-node-images: build-all-node-images-linux build-all-node-images-windows ## Build node-manager image for all OS and archs.
307307

308-
.PHONY: build-all-node-images-linux ## Build node-manager image for Linux.
309-
build-all-node-images-linux: $(addprefix build-node-image-linux-,$(ALL_ARCH.linux))
308+
.PHONY: build-all-node-images-linux
309+
build-all-node-images-linux: $(addprefix build-node-image-linux-,$(ALL_ARCH.linux)) ## Build node-manager image for Linux.
310310

311-
.PHONY: build-all-node-images-windows ## Build node-manager image for Windows.
312-
build-all-node-images-windows: $(addprefix build-node-image-windows-,$(ALL_OS_ARCH.windows))
311+
.PHONY: build-all-node-images-windows
312+
build-all-node-images-windows: $(addprefix build-node-image-windows-,$(ALL_OS_ARCH.windows)) ## Build node-manager image for Windows.
313313

314314
build-node-image-linux-%:
315315
$(MAKE) ARCH=$* build-node-image-linux
@@ -318,13 +318,13 @@ build-node-image-windows-%:
318318
$(MAKE) WINDOWS_OSVERSION=$(call word-hyphen,$*,1) ARCH=$(call word-hyphen,$*,2) build-node-image-windows
319319

320320
.PHONY: build-all-ccm-images
321-
build-all-ccm-images: $(addprefix build-ccm-image-,$(ALL_ARCH.linux))
321+
build-all-ccm-images: $(addprefix build-ccm-image-,$(ALL_ARCH.linux)) ## Build all CCM images.
322322

323323
build-ccm-image-%:
324324
$(MAKE) ARCH=$* build-ccm-image
325325

326326
.PHONY: push-all-ccm-images
327-
push-all-ccm-images: $(addprefix push-ccm-image-,$(ALL_ARCH.linux))
327+
push-all-ccm-images: $(addprefix push-ccm-image-,$(ALL_ARCH.linux)) ## Push all CCM images.
328328

329329
push-ccm-image-%:
330330
$(MAKE) ARCH=$* push-ccm-image
@@ -336,7 +336,7 @@ manifest-node-manager-image-windows-hpc-%:
336336
$(MAKE) ARCH=$(call word-hyphen,$*,1) manifest-node-manager-image-windows-hpc
337337

338338
.PHONY: manifest-node-manager-image-windows
339-
manifest-node-manager-image-windows:
339+
manifest-node-manager-image-windows: ## Create and push Windows node-manager manifest.
340340
set -x
341341
docker manifest create --amend $(NODE_MANAGER_IMAGE) $(NODE_MANAGER_LINUX_FULL_IMAGE_PREFIX)-$(ARCH) $(NODE_MANAGER_WINDOWS_FULL_IMAGE_PREFIX)-$(WINDOWS_OSVERSION)-$(ARCH)
342342
docker manifest annotate --os linux --arch $(ARCH) $(NODE_MANAGER_IMAGE) $(NODE_MANAGER_LINUX_FULL_IMAGE_PREFIX)-$(ARCH)
@@ -345,7 +345,7 @@ manifest-node-manager-image-windows:
345345
docker manifest push --purge $(NODE_MANAGER_IMAGE)
346346

347347
.PHONY: manifest-node-manager-images-windows-hpc
348-
manifest-node-manager-image-windows-hpc:
348+
manifest-node-manager-image-windows-hpc: ## Create and push Windows HPC node-manager manifest.
349349
set -x
350350
docker manifest create --amend $(NODE_MANAGER_IMAGE) $(NODE_MANAGER_LINUX_FULL_IMAGE_PREFIX)-$(ARCH) $(NODE_MANAGER_WINDOWS_FULL_IMAGE_PREFIX)-hpc-$(ARCH)
351351
docker manifest annotate --os linux --arch $(ARCH) $(NODE_MANAGER_IMAGE) $(NODE_MANAGER_LINUX_FULL_IMAGE_PREFIX)-$(ARCH)
@@ -412,7 +412,7 @@ test-e2e: ## Run k8s e2e tests.
412412
test-e2e-capz: ## Run k8s e2e tests with capz
413413
hack/test_k8s_e2e_capz.sh $(TEST_E2E_ARGS)
414414

415-
ensure-azcli:
415+
ensure-azcli: ## Ensure az CLI is installed.
416416
hack/ensure-azcli.sh
417417

418418
test-ccm-e2e: ensure-azcli ## Run cloud provider e2e tests.
@@ -438,7 +438,7 @@ deploy: image push ## Build, push and deploy an aks-engine cluster.
438438
CCM_IMAGE=$(CONTROLLER_MANAGER_IMAGE) CNM_IMAGE=$(NODE_MANAGER_IMAGE) HYPERKUBE_IMAGE=$(HYPERKUBE_IMAGE) hack/deploy-cluster.sh
439439

440440
.PHONY: cloud-build-prerequisites
441-
cloud-build-prerequisites:
441+
cloud-build-prerequisites: ## Install prerequisites for cloud build.
442442
apk add --no-cache jq
443443

444444
.PHONY: release-staging
@@ -454,11 +454,11 @@ endif
454454
## --------------------------------------
455455

456456
.PHONY: deploy-workload-cluster
457-
deploy-workload-cluster:
457+
deploy-workload-cluster: ## Deploy a workload cluster with CAPZ.
458458
hack/deploy-workload-cluster.sh
459459

460460
.PHONY: delete-workload-cluster
461-
delete-workload-cluster:
461+
delete-workload-cluster: ## Delete a CAPZ workload cluster.
462462
hack/delete-workload-cluster.sh
463463

464464
##@ Tools

0 commit comments

Comments
 (0)