Skip to content

Commit

Permalink
fix: hpc image build on AKS
Browse files Browse the repository at this point in the history
  • Loading branch information
andyzhangx committed May 25, 2023
1 parent 6813bd0 commit da641a0
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 6 deletions.
3 changes: 3 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -157,9 +157,12 @@ container-windows:
-t $(CSI_IMAGE_TAG)-windows-$(OSVERSION)-$(ARCH) --build-arg OSVERSION=$(OSVERSION) \
--provenance=false --sbom=false \
--build-arg ARCH=${ARCH} -f ./pkg/azurefileplugin/Windows.Dockerfile .
# workaround: only build hostprocess image once
ifdef WINDOWS_USE_HOST_PROCESS_CONTAINERS
ifeq ($(OSVERSION),ltsc2022)
$(MAKE) container-windows-hostprocess
endif
endif

# Set --provenance=false to not generate the provenance (which is what causes the multi-platform index to be generated, even for a single platform).
.PHONY: container-windows-hostprocess
Expand Down
12 changes: 6 additions & 6 deletions hack/release-image.sh
Original file line number Diff line number Diff line change
Expand Up @@ -21,12 +21,12 @@ if [[ "$#" -lt 1 ]]; then
exit 1
fi

REGISTRY_NAME="$1"
REGISTRY=$REGISTRY_NAME.azurecr.io
IMAGE_NAME=public/k8s/csi/azurefile-csi
CI=1
PUBLISH=1
WINDOWS_USE_HOST_PROCESS_CONTAINERS=true
export REGISTRY_NAME="$1"
export REGISTRY=$REGISTRY_NAME.azurecr.io
export IMAGE_NAME=public/k8s/csi/azurefile-csi
export CI=1
export PUBLISH=1
export WINDOWS_USE_HOST_PROCESS_CONTAINERS=true

az acr login --name $REGISTRY_NAME
make container-all container-windows-hostprocess-latest push-manifest push-latest
Expand Down

0 comments on commit da641a0

Please sign in to comment.