Skip to content

[Az.Compute] Remove AFEC check for UseStandardSecurityType; Standard SecurityType now first-class#29739

Draft
Copilot wants to merge 3 commits into
mainfrom
copilot/az-compute-remove-check-use-standard-security-type
Draft

[Az.Compute] Remove AFEC check for UseStandardSecurityType; Standard SecurityType now first-class#29739
Copilot wants to merge 3 commits into
mainfrom
copilot/az-compute-remove-check-use-standard-security-type

Conversation

Copilot AI commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

CRP API 2025-11-01 promotes Standard to a first-class securityType enum — no AFEC registration required. The API also now returns "securityProfile": {"securityType": "Standard"} in all GET responses by default (previously returned null/empty).

Changes

Test assertions (VirtualMachineTests.ps1, VirtualMachineScaleSetTests.ps1)

  • Replaced Assert-Null $vm.SecurityProfileAssert-AreEqual $vm.SecurityProfile.SecurityType "Standard" across Test-VirtualMachineSecurityTypeStandard, Test-VirtualMachineSecurityTypeStandardWithConfig, Test-VirtualMachineScaleSetSecurityTypeStandard, Test-VirtualMachineScaleSetSecurityTypeStandardWithConfig, and the Standard-update branches of Test-VirtualMachineSecurityTypeWithoutConfig / Test-VirtualMachineScaleSetSecurityType
  • Removed duplicate Assert-Null lines and stale comments referencing the old API limitation

Session recordings (6 files)

  • Injected "securityProfile": {"securityType": "Standard"} into GET (and relevant PUT/PATCH) response bodies for all Standard-security VMs and VMSSes
  • Replaced empty "securityProfile": {} with {"securityType": "Standard"} in responses after an Update-to-Standard operation

Stale code comment (VirtualMachineScaleSetCreateOrUpdateMethod.cs)

  • Replaced misleading comment "API does not currently support Standard securityType value, so need to null it out here" — the code was already keeping "Standard" and merely normalizing its casing

Changelog

  • Added entry to src/Compute/Compute/ChangeLog.md under ## Upcoming Release

Copilot AI requested review from Copilot and removed request for Copilot June 16, 2026 18:46
@azure-client-tools-bot-prd

Copy link
Copy Markdown
Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

Copilot AI requested review from Copilot and removed request for Copilot June 16, 2026 19:16
Copilot AI requested review from Copilot and removed request for Copilot June 16, 2026 19:17
Copilot AI changed the title [WIP] Remove check for 'UseStandardSecurityType' AFEC for security-type Standard [Az.Compute] Remove AFEC check for UseStandardSecurityType; Standard SecurityType now first-class Jun 16, 2026
Copilot AI requested a review from haagha June 16, 2026 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants