Skip to content

Commit 86743a4

Browse files
Fix reuse artifacts logic for Windows CNM images
1 parent 0a1cd79 commit 86743a4

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

scripts/ci-build-azure-ccm.sh

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ setup() {
6666

6767
main() {
6868
if [[ "$(can_reuse_artifacts)" =~ "false" ]]; then
69-
echo "Build Linux Azure amd64 cloud controller manager"
69+
echo "Building Linux Azure amd64 cloud controller manager"
7070
make -C "${AZURE_CLOUD_PROVIDER_ROOT}" build-ccm-image-amd64 push-ccm-image-amd64
7171
echo "Building Linux amd64 and Windows ${WINDOWS_IMAGE_VERSION} amd64 cloud node managers"
7272
make -C "${AZURE_CLOUD_PROVIDER_ROOT}" build-node-image-linux-amd64 push-node-image-linux-amd64 push-node-image-windows-"${WINDOWS_IMAGE_VERSION}"-amd64 manifest-node-manager-image-windows-"${WINDOWS_IMAGE_VERSION}"-amd64
@@ -82,8 +82,7 @@ can_reuse_artifacts() {
8282
fi
8383
done
8484

85-
FULL_VERSION=$(docker manifest inspect mcr.microsoft.com/windows/nanoserver:${WINDOWS_IMAGE_VERSION} | jq -r '.manifests[0].platform["os.version"]')
86-
if ! docker manifest inspect "${REGISTRY}/${CNM_IMAGE_NAME}:${IMAGE_TAG_CNM}" | grep -q "\"os.version\": \"${FULL_VERSION}\""; then
85+
if ! docker manifest inspect "${REGISTRY}/${CNM_IMAGE_NAME}:${IMAGE_TAG_CNM}" | grep -q "\"os.version\": \"${WINDOWS_IMAGE_VERSION}\""; then
8786
echo "false" && return
8887
fi
8988

0 commit comments

Comments
 (0)