Skip to content

Commit

Permalink
[cloudbuild] Build all images in a single step. (kubernetes-sigs#1891)
Browse files Browse the repository at this point in the history
* [cloudbuild] Build all images in a single step.

* [makefile] Use `STAGING_IMAGE_REGISTRY` only to default `IMAGE_REGISTRY`

* [cloudbuild] Set `IMAGE_REGISTRY` instead of `STAGING_IMAGE_REGISTRY`
  • Loading branch information
trasc authored and vsoch committed Apr 18, 2024
1 parent 1be756b commit dcd61c8
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 22 deletions.
10 changes: 5 additions & 5 deletions Makefile
Expand Up @@ -35,7 +35,7 @@ DOCKER_BUILDX_CMD ?= docker buildx
IMAGE_BUILD_CMD ?= $(DOCKER_BUILDX_CMD) build
IMAGE_BUILD_EXTRA_OPTS ?=
# TODO(#52): Add kueue to k8s gcr registry
STAGING_IMAGE_REGISTRY ?= gcr.io/k8s-staging-kueue
STAGING_IMAGE_REGISTRY := gcr.io/k8s-staging-kueue
IMAGE_REGISTRY ?= $(STAGING_IMAGE_REGISTRY)
IMAGE_NAME := kueue
IMAGE_REPO ?= $(IMAGE_REGISTRY)/$(IMAGE_NAME)
Expand Down Expand Up @@ -303,7 +303,7 @@ artifacts: kustomize yq helm
$(HELM) package --version $(GIT_TAG) --app-version $(GIT_TAG) charts/kueue -d artifacts/
mv artifacts/kueue-$(GIT_TAG).tgz artifacts/kueue-chart-$(GIT_TAG).tgz
# Revert the image changes
$(YQ) e '.controllerManager.manager.image.repository = "$(STAGING_IMAGE_REGISTRY)/$(IMAGE_NAME)" | .controllerManager.manager.image.tag = "main" | .controllerManager.manager.image.pullPolicy = "Always"' -i charts/kueue/values.yaml
$(YQ) e '.controllerManager.manager.image.repository = "$(IMAGE_REGISTRY)/$(IMAGE_NAME)" | .controllerManager.manager.image.tag = "main" | .controllerManager.manager.image.pullPolicy = "Always"' -i charts/kueue/values.yaml

.PHONY: prepare-release-branch
prepare-release-branch: yq kustomize
Expand All @@ -317,7 +317,7 @@ prepare-release-branch: yq kustomize
# Developers don't need to build this image, as it will be available as gcr.io/k8s-staging-kueue/debug
.PHONY: debug-image-push
debug-image-push:
$(IMAGE_BUILD_CMD) -t $(STAGING_IMAGE_REGISTRY)/debug:$(GIT_TAG) \
$(IMAGE_BUILD_CMD) -t $(IMAGE_REGISTRY)/debug:$(GIT_TAG) \
--platform=$(PLATFORMS) \
--push ./hack/debugpod

Expand All @@ -329,8 +329,8 @@ importer-build:
.PHONY: importer-image-build
importer-image-build:
$(IMAGE_BUILD_CMD) \
-t $(STAGING_IMAGE_REGISTRY)/importer:$(GIT_TAG) \
-t $(STAGING_IMAGE_REGISTRY)/importer:$(RELEASE_BRANCH)-latest \
-t $(IMAGE_REGISTRY)/importer:$(GIT_TAG) \
-t $(IMAGE_REGISTRY)/importer:$(RELEASE_BRANCH)-latest \
--platform=$(PLATFORMS) \
--build-arg BASE_IMAGE=$(BASE_IMAGE) \
--build-arg BUILDER_IMAGE=$(BUILDER_IMAGE) \
Expand Down
18 changes: 1 addition & 17 deletions cloudbuild.yaml
Expand Up @@ -8,26 +8,10 @@ steps:
entrypoint: make
args:
- image-push
env:
- IMAGE_REGISTRY=gcr.io/$PROJECT_ID
- GIT_TAG=$_GIT_TAG
- EXTRA_TAG=$_PULL_BASE_REF
- DOCKER_BUILDX_CMD=/buildx-entrypoint
- name: 'gcr.io/k8s-staging-test-infra/gcb-docker-gcloud:v20221214-1b4dd4d69a'
entrypoint: make
args:
- debug-image-push
env:
- IMAGE_REGISTRY=gcr.io/$PROJECT_ID
- GIT_TAG=$_GIT_TAG
- EXTRA_TAG=$_PULL_BASE_REF
- DOCKER_BUILDX_CMD=/buildx-entrypoint
- name: 'gcr.io/k8s-staging-test-infra/gcb-docker-gcloud:v20221214-1b4dd4d69a'
entrypoint: make
args:
- importer-image-push
env:
- STAGING_IMAGE_REGISTRY=gcr.io/$PROJECT_ID
- IMAGE_REGISTRY=gcr.io/$PROJECT_ID
- GIT_TAG=$_GIT_TAG
- EXTRA_TAG=$_PULL_BASE_REF
- DOCKER_BUILDX_CMD=/buildx-entrypoint
Expand Down

0 comments on commit dcd61c8

Please sign in to comment.