Skip to content

Commit

Permalink
Add pipeline var to skip extension check (#5802)
Browse files Browse the repository at this point in the history
  • Loading branch information
timmy-wright authored Feb 12, 2025
1 parent 0bd48c9 commit a7dd571
Show file tree
Hide file tree
Showing 7 changed files with 37 additions and 2 deletions.
5 changes: 5 additions & 0 deletions .pipelines/.vsts-vhd-builder-release-windows.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,10 @@ parameters:
displayName: VHD Debug
type: boolean
default: False
- name: skipExtensionCheck
displayName: Skip Extension Check
type: boolean
default: False

# Use variable group "ab-windows-ame-tenant" and link it to the pipeline "AKS Windows VHD Build"
# Use variable group "ab-windows-ame-tenant" and link it to the pipeline "AKS Windows VHD Build - PR check-in gate"
Expand All @@ -54,3 +58,4 @@ stages:
build2022containerdgen2: ${{ parameters.build2022containerdgen2 }}
build23H2: ${{ parameters.build23H2 }}
build23H2gen2: ${{ parameters.build23H2gen2 }}
skipExtensionCheck: ${{ parameters.skipExtensionCheck }}
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,9 @@ parameters:
- name: hyperVGeneration
type: string
displayName: V1 or V2.
- name: skipExtensionCheck
displayName: Skip Extension Check
type: boolean
default: False
- name: windowsBaseImageUrl
displayName: Windows Base Image URL Override
Expand Down Expand Up @@ -58,6 +61,7 @@ stages:
echo '##vso[task.setvariable variable=HYPERV_GENERATION]${{ parameters.hyperVGeneration }}'
echo '##vso[task.setvariable variable=AZURE_VM_SIZE]${{ parameters.buildVmSize }}'
echo '##vso[task.setvariable variable=WINDOWS_SKU]${{ parameters.windowsSku }}'
echo '##vso[task.setvariable variable=SKIP_EXTENSION_CHECK]${{ parameters.skipExtensionCheck }}'
echo '##vso[task.setvariable variable=WINDOWS_BASE_IMAGE_URL]${{ parameters.windowsBaseImageUrl }}'
echo '##vso[task.setvariable variable=WINDOWS_NANO_IMAGE_URL]${{ parameters.windowsNanoImageUrl }}'
echo '##vso[task.setvariable variable=WINDOWS_CORE_IMAGE_URL]${{ parameters.windowsCoreImageUrl }}'
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,10 @@ parameters:
displayName: VHD Debug
type: boolean
default: False
- name: skipExtensionCheck
displayName: Skip Extension Check
type: boolean
default: False

stages:
- template: ./.build-and-test-windows-vhd-template.yaml
Expand All @@ -34,6 +38,7 @@ stages:
build: ${{ parameters.build2019containerd }}
vhddebug: ${{ parameters.vhddebug }}
dryrun: ${{ parameters.dryrun }}
skipExtensionCheck: ${{ parameters.skipExtensionCheck }}
windowsBaseImageUrl: $(WINDOWS_2019_BASE_IMAGE_URL)
windowsNanoImageUrl: $(WINDOWS_2019_NANO_IMAGE_URL)
windowsCoreImageUrl: $(WINDOWS_2019_CORE_IMAGE_URL)
Expand All @@ -48,6 +53,7 @@ stages:
build: ${{ parameters.build2022containerd }}
vhddebug: ${{ parameters.vhddebug }}
dryrun: ${{ parameters.dryrun }}
skipExtensionCheck: ${{ parameters.skipExtensionCheck }}
windowsBaseImageUrl: $(WINDOWS_2022_BASE_IMAGE_URL)
windowsNanoImageUrl: $(WINDOWS_2022_NANO_IMAGE_URL)
windowsCoreImageUrl: $(WINDOWS_2022_CORE_IMAGE_URL)
Expand All @@ -62,6 +68,7 @@ stages:
build: ${{ parameters.build2022containerdgen2 }}
vhddebug: ${{ parameters.vhddebug }}
dryrun: ${{ parameters.dryrun }}
skipExtensionCheck: ${{ parameters.skipExtensionCheck }}
windowsBaseImageUrl: $(WINDOWS_23H2_GEN2_BASE_IMAGE_URL)
windowsNanoImageUrl: $(WINDOWS_2022_NANO_IMAGE_URL)
windowsCoreImageUrl: $(WINDOWS_2022_CORE_IMAGE_URL)
Expand All @@ -76,6 +83,7 @@ stages:
build: ${{ parameters.build23H2 }}
vhddebug: ${{ parameters.vhddebug }}
dryrun: ${{ parameters.dryrun }}
skipExtensionCheck: ${{ parameters.skipExtensionCheck }}
windowsBaseImageUrl: $(WINDOWS_23H2_BASE_IMAGE_URL)
windowsNanoImageUrl: $(WINDOWS_2022_NANO_IMAGE_URL)
windowsCoreImageUrl: $(WINDOWS_2022_CORE_IMAGE_URL)
Expand All @@ -90,6 +98,7 @@ stages:
build: ${{ parameters.build23H2gen2 }}
vhddebug: ${{ parameters.vhddebug }}
dryrun: ${{ parameters.dryrun }}
skipExtensionCheck: ${{ parameters.skipExtensionCheck }}
windowsBaseImageUrl: $(WINDOWS_23H2_GEN2_BASE_IMAGE_URL)
windowsNanoImageUrl: $(WINDOWS_2022_NANO_IMAGE_URL)
windowsCoreImageUrl: $(WINDOWS_2022_CORE_IMAGE_URL)
Expand Down
5 changes: 5 additions & 0 deletions .pipelines/templates/.builder-release-template-windows.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,7 @@ steps:
-e BUILD_NUMBER=$(Build.BuildNumber) \
-e WINDOWS_SKU=$(WINDOWS_SKU) \
-e OS_TYPE="Windows" \
-e SKIP_EXTENSION_CHECK=$(SKIP_EXTENSION_CHECK) \
-e MODE=$(MODE) \
-e SIG_GALLERY_NAME=${SIG_GALLERY_NAME} \
-e SIG_IMAGE_NAME=${SIG_IMAGE_NAME} \
Expand Down Expand Up @@ -121,6 +122,7 @@ steps:
-e MANAGED_SIG_ID=${MANAGED_SIG_ID} \
-e AZURE_LOCATION=${AZURE_BUILD_LOCATION} \
-e WINDOWS_SKU=$(WINDOWS_SKU) \
-e SKIP_EXTENSION_CHECK=$(SKIP_EXTENSION_CHECK) \
-e SKIPVALIDATEREOFFERUPDATE=${SKIPVALIDATEREOFFERUPDATE} \
-e OS_TYPE="Windows" \
-e MODE=$(MODE) \
Expand Down Expand Up @@ -177,6 +179,7 @@ steps:
-e LOCATION="${AZURE_BUILD_LOCATION}" \
-e OS_TYPE="Windows" \
-e CLASSIC_BLOB="$(STORAGE_ACCT_BLOB_URL)" \
-e SKIP_EXTENSION_CHECK=$(SKIP_EXTENSION_CHECK) \
-e SIG_IMAGE_NAME=${SIG_IMAGE_NAME} \
-e SIG_GALLERY_NAME=${SIG_GALLERY_NAME} \
-e SIG_IMAGE_VERSION=${SIG_IMAGE_VERSION} \
Expand Down Expand Up @@ -214,6 +217,7 @@ steps:
-e IMAGE_NAME=${IMAGE_NAME} \
-e VNET_RESOURCE_GROUP_NAME=${VNET_RESOURCE_GROUP_NAME} \
-e TEST_VM_RESOURCE_GROUP_NAME=${TEST_VM_RESOURCE_GROUP_NAME} \
-e SKIP_EXTENSION_CHECK=$(SKIP_EXTENSION_CHECK) \
-e WINDOWS_SKU=$(WINDOWS_SKU) \
-e IMPORTED_IMAGE_NAME=${IMPORTED_IMAGE_NAME} \
-e SIG_GALLERY_NAME=${SIG_GALLERY_NAME} \
Expand Down Expand Up @@ -244,6 +248,7 @@ steps:
-e STORAGE_ACCT_BLOB_URL=${STORAGE_ACCT_BLOB_URL} \
-e VHD_NAME=${VHD_NAME} \
-e OS_NAME="Windows" \
-e SKIP_EXTENSION_CHECK=${SKIP_EXTENSION_CHECK} \
-e SKU_NAME=${SKU_NAME} \
-e OFFER_NAME="Windows" \
-e MODE=$(MODE) \
Expand Down
1 change: 1 addition & 0 deletions vhdbuilder/packer/init-variables.sh
Original file line number Diff line number Diff line change
Expand Up @@ -500,6 +500,7 @@ cat <<EOF > vhdbuilder/packer/settings.json
"vm_size": "${AZURE_VM_SIZE}",
"create_time": "${CREATE_TIME}",
"img_version": "${IMG_VERSION}",
"SKIP_EXTENSION_CHECK": "${SKIP_EXTENSION_CHECK}",
"vhd_build_timestamp": "${VHD_BUILD_TIMESTAMP}",
"windows_image_publisher": "${WINDOWS_IMAGE_PUBLISHER}",
"windows_image_offer": "${WINDOWS_IMAGE_OFFER}",
Expand Down
12 changes: 11 additions & 1 deletion vhdbuilder/packer/windows/configure-windows-vhd.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -798,13 +798,23 @@ function Log-ReofferUpdate

function Test-AzureExtensions
{
if ($env:SKIP_EXTENSION_CHECK -eq "true")
{
Write-Log "Skipping extension check because SKIP_EXTENSION_CHECK is set to true"
return
}

# Expect the Windows VHD without any other extensions
if (Test-Path "C:\Packages\Plugins")
{
$actualExtensions = (Get-ChildItem "C:\Packages\Plugins").Name
if ($actualExtensions.Length -gt 0)
{
Write-Log "Azure extensions are not expected. Details: $( $actualExtensions | Out-String )"
Write-Log "Azure extensions are not expected and skip extension checks was $env:SKIP_EXTENSION_CHECK. Details:"
foreach ($extension in $actualExtensions)
{
Write-Log "* $extension"
}
exit 1
}
}
Expand Down
3 changes: 2 additions & 1 deletion vhdbuilder/packer/windows/windows-vhd-builder-sig.json
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,8 @@
"elevated_password": "{{.WinRMPassword}}",
"environment_vars": [
"ProvisioningPhase=3",
"WindowsSKU={{user `windows_sku`}}"
"WindowsSKU={{user `windows_sku`}}",
"SKIP_EXTENSION_CHECK={{ user `SKIP_EXTENSION_CHECK` }}"
],
"type": "powershell",
"scripts": [
Expand Down

0 comments on commit a7dd571

Please sign in to comment.