Upgrade Cdn 2025-09-01-preview, move Edge action out of cdn#29354
Upgrade Cdn 2025-09-01-preview, move Edge action out of cdn#29354Ptnan7 wants to merge 9 commits intoAzure:mainfrom
Conversation
| Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. |
There was a problem hiding this comment.
Pull request overview
Updates the Az.Cdn AutoRest-generated surface to the 2025-09-01-preview API and restructures the module to introduce new resources (web agents, profile agents, deployment versions, policies) while removing Edge Action artifacts from this module.
Changes:
- Bumps AutoRest input to
2025-09-01-previewand updates regeneration commit + adds a directive to adjust LRO polling behavior. - Adds new UX definitions and generated help/docs for new resource types; removes Edge Action docs/tests/examples from this module.
- Refactors/neutralizes many Pester tests (mostly skipped placeholders) and updates module assembly version to
6.0.0.
Reviewed changes
Copilot reviewed 229 out of 467 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| .github/prompts/merge-crud-tests.prompt.md | Adds internal workflow guidance for consolidating CRUD tests. |
| src/Cdn/Cdn.Autorest/Properties/AssemblyInfo.cs | Bumps Az.Cdn assembly version to 6.0.0. |
| src/Cdn/Cdn.Autorest/README.md | Switches AutoRest input to 2025-09-01-preview OpenAPI, updates commit, adds model-cmdlet rename + LRO directive. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/cdnWebApplicationFirewallPolicies.json | Replaces Edge Action UX with WAF policy UX for Get/Remove-AzCdnPolicy. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles-afdEndpoints-routes.json | Updates UX apiVersion to 2025-09-01-preview. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles-afdEndpoints.json | Updates UX apiVersion to 2025-09-01-preview. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles-agents.json | Replaces Edge Action execution filter UX with profiles/agents UX. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles-customDomains.json | Updates UX apiVersion to 2025-09-01-preview. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles-deploymentVersions.json | Adds UX for deploymentVersions-related commands. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles-endpoints-customDomains.json | Updates UX apiVersion to 2025-09-01-preview. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles-endpoints-originGroups.json | Updates UX apiVersion to 2025-09-01-preview. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles-endpoints-origins.json | Updates UX apiVersion to 2025-09-01-preview. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles-endpoints.json | Updates UX apiVersion to 2025-09-01-preview. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles-originGroups-origins.json | Updates UX apiVersion to 2025-09-01-preview. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles-originGroups.json | Updates UX apiVersion to 2025-09-01-preview. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles-ruleSets-rules.json | Updates UX apiVersion to 2025-09-01-preview. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles-ruleSets.json | Updates UX apiVersion to 2025-09-01-preview. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles-securityPolicies.json | Updates UX apiVersion to 2025-09-01-preview. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles-secrets.json | Updates UX apiVersion to 2025-09-01-preview. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/profiles.json | Updates UX apiVersion to 2025-09-01-preview. |
| src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/webAgents.json | Adds UX for Get/Remove-AzCdnWebAgent. |
| src/Cdn/Cdn.Autorest/cert-policy.json | Adds a certificate policy JSON used for related scenarios. |
| src/Cdn/Cdn.Autorest/custom/New-AzFrontDoorCdnSecretCustomerCertificateParametersObject.ps1 | Removes SubjectAlternativeName parameter support from helper cmdlet implementation. |
| src/Cdn/Cdn.Autorest/custom/autogen-model-cmdlets/New-AzCdnHealthProbeParametersObject.ps1 | Adds Grpc as a probe protocol completer option. |
| src/Cdn/Cdn.Autorest/custom/autogen-model-cmdlets/New-AzFrontDoorCdnMigrationParametersObject.ps1 | Extends SKU argument completer list (adds Classic_AzureFrontDoor). |
| src/Cdn/Cdn.Autorest/custom/autogen-model-cmdlets/New-AzFrontDoorCdnOriginGroupHealthProbeSettingObject.ps1 | Adds Grpc as a probe protocol completer option. |
| src/Cdn/Cdn.Autorest/custom/autogen-model-cmdlets/New-AzFrontDoorCdnProfileScrubbingRulesObject.ps1 | Adds new match variable options and introduces SelectorMatchOperator. |
| src/Cdn/Cdn.Autorest/custom/autogen-model-cmdlets/New-AzFrontDoorCdnRuleEdgeActionObject.ps1 | Adds a new in-memory EdgeAction helper object cmdlet. |
| src/Cdn/Cdn.Autorest/docs/Clear-AzCdnEndpointContent.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Clear-AzCdnKnowledgeSource.md | Re-targets doc from Edge Action Version removal to Clear-AzCdnKnowledgeSource. |
| src/Cdn/Cdn.Autorest/docs/Clear-AzFrontDoorCdnEndpointContent.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzCdnCustomDomain.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzCdnDeploymentVersion.md | Adds new generated help for Get-AzCdnDeploymentVersion. |
| src/Cdn/Cdn.Autorest/docs/Get-AzCdnEndpoint.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzCdnEndpointResourceUsage.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzCdnKnowledgeSource.md | Adds new generated help for Get-AzCdnKnowledgeSource. |
| src/Cdn/Cdn.Autorest/docs/Get-AzCdnManagedRuleSet.md | Adds new generated help for Get-AzCdnManagedRuleSet. |
| src/Cdn/Cdn.Autorest/docs/Get-AzCdnOrigin.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzCdnOriginGroup.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzCdnPolicy.md | Re-targets doc from Get-AzCdnEdgeAction to Get-AzCdnPolicy. |
| src/Cdn/Cdn.Autorest/docs/Get-AzCdnProfileAgent.md | Adds new generated help for Get-AzCdnProfileAgent. |
| src/Cdn/Cdn.Autorest/docs/Get-AzCdnWebAgent.md | Adds new generated help for Get-AzCdnWebAgent. |
| src/Cdn/Cdn.Autorest/docs/Get-AzFrontDoorCdnCustomDomain.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzFrontDoorCdnEndpoint.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzFrontDoorCdnEndpointResourceUsage.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzFrontDoorCdnOrigin.md | Updates OriginGroupName + ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzFrontDoorCdnOriginGroup.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzFrontDoorCdnOriginGroupResourceUsage.md | Fixes synopsis punctuation + updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzFrontDoorCdnProfileResourceUsage.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzFrontDoorCdnRoute.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzFrontDoorCdnRule.md | Updates ProfileName and SetName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzFrontDoorCdnRuleSet.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzFrontDoorCdnRuleSetResourceUsage.md | Fixes synopsis punctuation + updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzFrontDoorCdnSecret.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Get-AzFrontDoorCdnSecurityPolicy.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Import-AzCdnEndpointContent.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Invoke-AzCdnAbortProfileToAFDMigration.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/New-AzCdnCustomDomain.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/New-AzCdnEndpoint.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/New-AzCdnOrigin.md | Tweaks Name description and updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/New-AzCdnOriginGroup.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/New-AzFrontDoorCdnCustomDomain.md | Adds mTLS scenario parameter documentation + updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/New-AzFrontDoorCdnEndpoint.md | Adds EnforceMtl parameter documentation + updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/New-AzFrontDoorCdnOriginGroup.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/New-AzFrontDoorCdnProfileScrubbingRulesObject.md | Adds SelectorMatchOperator parameter to helper object doc. |
| src/Cdn/Cdn.Autorest/docs/New-AzFrontDoorCdnRule.md | Updates ProfileName and SetName description text. |
| src/Cdn/Cdn.Autorest/docs/New-AzFrontDoorCdnRuleEdgeActionObject.md | Adds new helper object cmdlet help (with placeholder examples). |
| src/Cdn/Cdn.Autorest/docs/New-AzFrontDoorCdnRuleSet.md | Fixes punctuation + updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/New-AzFrontDoorCdnSecret.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/New-AzFrontDoorCdnSecretCustomerCertificateParametersObject.md | Removes SubjectAlternativeName from syntax/parameters documentation. |
| src/Cdn/Cdn.Autorest/docs/New-AzFrontDoorCdnSecurityPolicy.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzCdnCustomDomain.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzCdnEndpoint.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzCdnKnowledgeSource.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzCdnOrigin.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzCdnOriginGroup.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzCdnPolicy.md | Re-targets doc from EdgeAction removal to Remove-AzCdnPolicy. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzCdnProfileAgent.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzCdnWebAgent.md | Re-targets doc from Edge Action Execution Filter removal to Remove-AzCdnWebAgent. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzFrontDoorCdnCustomDomain.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzFrontDoorCdnEndpoint.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzFrontDoorCdnOrigin.md | Updates OriginGroupName + ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzFrontDoorCdnOriginGroup.md | Updates OriginGroupName + ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzFrontDoorCdnRoute.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzFrontDoorCdnRule.md | Updates ProfileName and SetName description text. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzFrontDoorCdnRuleSet.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzFrontDoorCdnSecret.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Remove-AzFrontDoorCdnSecurityPolicy.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Start-AzCdnEndpoint.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Stop-AzCdnEndpoint.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Test-AzCdnEndpointCustomDomain.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Test-AzCdnProfileMigrationCompatibility.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Test-AzFrontDoorCdnEndpointCustomDomain.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Test-AzFrontDoorCdnProfileHostNameAvailability.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Update-AzCdnEndpoint.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Update-AzCdnOrigin.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Update-AzCdnOriginGroup.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Update-AzFrontDoorCdnCustomDomain.md | Adds mTLS scenario parameter documentation + updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Update-AzFrontDoorCdnCustomDomainValidationToken.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Update-AzFrontDoorCdnEndpoint.md | Adds EnforceMtl parameter documentation + updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Update-AzFrontDoorCdnOriginGroup.md | Updates OriginGroupName + ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Update-AzFrontDoorCdnRoute.md | Adds GrpcState parameter documentation + updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Update-AzFrontDoorCdnRule.md | Updates ProfileName and SetName description text. |
| src/Cdn/Cdn.Autorest/docs/Update-AzFrontDoorCdnSecret.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/docs/Update-AzFrontDoorCdnSecurityPolicy.md | Updates ProfileName description text. |
| src/Cdn/Cdn.Autorest/examples/Add-AzCdnEdgeActionAttachment.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/Approve-AzCdnDeploymentVersion.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Clear-AzCdnKnowledgeSource.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Compare-AzCdnDeploymentVersion.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Compare-AzCdnDeploymentVersion.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Compare-AzCdnDeploymentVersion.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Compare-AzCdnDeploymentVersion.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Compare-AzCdnDeploymentVersion.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Deploy-AzCdnEdgeActionVersionCode.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/Get-AzCdnDeploymentVersion.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Get-AzCdnEdgeAction.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/Get-AzCdnEdgeActionExecutionFilter.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/Get-AzCdnEdgeActionVersion.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/Get-AzCdnEdgeActionVersionCode.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/Get-AzCdnKnowledgeSource.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Get-AzCdnManagedRuleSet.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Get-AzCdnPolicy.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Get-AzCdnProfileAgent.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Get-AzCdnWebAgent.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/New-AzCdnEdgeAction.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/New-AzCdnEdgeActionExecutionFilter.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/New-AzCdnEdgeActionVersion.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/New-AzCdnKnowledgeSource.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/New-AzCdnPolicy.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/New-AzCdnProfileAgent.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/New-AzCdnWebAgent.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/New-AzFrontDoorCdnRuleEdgeActionObject.md | Adds placeholder example page for new helper object cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Remove-AzCdnEdgeAction.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/Remove-AzCdnEdgeActionAttachment.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/Remove-AzCdnEdgeActionExecutionFilter.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/Remove-AzCdnEdgeActionVersion.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/Remove-AzCdnKnowledgeSource.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Remove-AzCdnPolicy.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Remove-AzCdnProfileAgent.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Remove-AzCdnWebAgent.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Update-AzCdnDeploymentVersion.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Update-AzCdnEdgeAction.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/Update-AzCdnEdgeActionExecutionFilter.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/Update-AzCdnEdgeActionVersion.md | Removes Edge Action example page from this module. |
| src/Cdn/Cdn.Autorest/examples/Update-AzCdnKnowledgeSource.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Update-AzCdnPolicy.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Update-AzCdnProfileAgent.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/examples/Update-AzCdnWebAgent.md | Adds placeholder example page for new cmdlet. |
| src/Cdn/Cdn.Autorest/generate-info.json | Updates generate run identifier. |
| src/Cdn/Cdn.Autorest/test/Add-AzCdnEdgeActionAttachment.Tests.ps1 | Removes Edge Action Attachment tests from this module. |
| src/Cdn/Cdn.Autorest/test/Approve-AzCdnDeploymentVersion.Tests.ps1 | Adds new (currently skipped) test skeleton for Approve-AzCdnDeploymentVersion. |
| src/Cdn/Cdn.Autorest/test/Clear-AzCdnEndpointContent.Tests.ps1 | Refactors purge tests to create/destroy a dedicated endpoint. |
| src/Cdn/Cdn.Autorest/test/Clear-AzCdnKnowledgeSource.Tests.ps1 | Adds new (currently skipped) test skeleton for Clear-AzCdnKnowledgeSource. |
| src/Cdn/Cdn.Autorest/test/Compare-AzCdnDeploymentVersion.Tests.ps1 | Renames test file intent to Compare-AzCdnDeploymentVersion (skipped skeleton). |
| src/Cdn/Cdn.Autorest/test/Deploy-AzCdnEdgeActionVersionCode.Tests.ps1 | Removes Edge Action Version Code deploy tests from this module. |
| src/Cdn/Cdn.Autorest/test/Enable-AzCdnCustomDomainCustomHttps.Tests.ps1 | Replaces live test logic with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Enable-AzFrontDoorCdnProfileMigration.Tests.ps1 | Replaces live-only migration test with skipped placeholder. |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnCustomDomain.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnDeploymentVersion.Tests.ps1 | Adds new (currently skipped) test skeleton for Get-AzCdnDeploymentVersion. |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnEdgeAction.Tests.ps1 | Removes Edge Action tests from this module. |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnEdgeActionExecutionFilter.Tests.ps1 | Removes Edge Action Execution Filter tests from this module. |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnEdgeActionVersion.Tests.ps1 | Removes Edge Action Version tests from this module. |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnEndpoint.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnEndpointResourceUsage.Tests.ps1 | Refactors resource-usage test to create/destroy a dedicated endpoint. |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnKnowledgeSource.Tests.ps1 | Adds new (currently skipped) test skeleton for Get-AzCdnKnowledgeSource. |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnManagedRuleSet.Tests.ps1 | Fixes test header/name to Get-AzCdnManagedRuleSet (skipped). |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnOrigin.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnOriginGroup.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnPolicy.Tests.ps1 | Adds new (currently skipped) test skeleton for Get-AzCdnPolicy. |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnProfile.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnProfileAgent.Tests.ps1 | Adds new (currently skipped) test skeleton for Get-AzCdnProfileAgent. |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnSubscriptionResourceUsage.Recording.json | Updates recording to 2025-09-01-preview api-version and new headers. |
| src/Cdn/Cdn.Autorest/test/Get-AzCdnWebAgent.Tests.ps1 | Adds new (currently skipped) test skeleton for Get-AzCdnWebAgent. |
| src/Cdn/Cdn.Autorest/test/Get-AzFrontDoorCdnCustomDomain.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Get-AzFrontDoorCdnEndpoint.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Get-AzFrontDoorCdnEndpointResourceUsage.Tests.ps1 | Makes endpoint resource-usage test self-contained with create/cleanup. |
| src/Cdn/Cdn.Autorest/test/Get-AzFrontDoorCdnOrigin.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Get-AzFrontDoorCdnOriginGroup.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Get-AzFrontDoorCdnProfile.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Get-AzFrontDoorCdnRoute.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Get-AzFrontDoorCdnRule.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Get-AzFrontDoorCdnRuleSet.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Get-AzFrontDoorCdnRuleSetResourceUsage.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Get-AzFrontDoorCdnSecret.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Get-AzFrontDoorCdnSecurityPolicy.Tests.ps1 | Replaces previous assertions with skipped placeholders. |
| src/Cdn/Cdn.Autorest/test/Invoke-AzCdnAbortProfileToAFDMigration.Tests.ps1 | Replaces previous playback logic with skipped placeholder. |
| It 'List' -skip { | ||
| { throw [System.NotImplementedException] } | Should -Not -Throw | ||
| } | ||
|
|
||
| It 'Get' { | ||
| $frontDoorCdnProfile = Get-AzFrontDoorCdnProfile -ResourceGroupName $env.ResourceGroupName -Name $env.FrontDoorCdnProfileName | ||
|
|
||
| $frontDoorCdnProfile.Name | Should -Be $env.FrontDoorCdnProfileName | ||
| $frontDoorCdnProfile.SkuName | Should -Be "Standard_AzureFrontDoor" | ||
| $frontDoorCdnProfile.Location | Should -Be "Global" | ||
| It 'List1' -skip { | ||
| { throw [System.NotImplementedException] } | Should -Not -Throw | ||
| } | ||
|
|
||
| It 'List1' { | ||
| $frontDoorCdnProfiles = Get-AzFrontDoorCdnProfile -ResourceGroupName $env.ResourceGroupName | ||
| $frontDoorCdnProfiles.Count | Should -BeGreaterOrEqual 1 | ||
| It 'Get' -skip { | ||
| { throw [System.NotImplementedException] } | Should -Not -Throw | ||
| } | ||
|
|
||
| It 'GetViaIdentity' { | ||
| $frontDoorCdnProfileObject = Get-AzFrontDoorCdnProfile -ResourceGroupName $env.ResourceGroupName -Name $env.FrontDoorCdnProfileName | ||
| $frontDoorCdnProfile = Get-AzFrontDoorCdnProfile -InputObject $frontDoorCdnProfileObject | ||
|
|
||
| $frontDoorCdnProfile.Name | Should -Be $env.FrontDoorCdnProfileName | ||
| $frontDoorCdnProfile.SkuName | Should -Be "Standard_AzureFrontDoor" | ||
| $frontDoorCdnProfile.Location | Should -Be "Global" | ||
| It 'GetViaIdentity' -skip { | ||
| { throw [System.NotImplementedException] } | Should -Not -Throw | ||
| } |
There was a problem hiding this comment.
The PR disables real validation for this cmdlet by replacing functional tests with skipped placeholders (and a non-actionable NotImplementedException assertion). This removes regression coverage for the new API version and any behavior changes introduced by the spec upgrade. Prefer either (1) restoring playback tests with updated recordings, (2) converting to a self-contained CRUD flow (per the repo’s CRUD-in-one-file pattern) so recordings can be re-generated, or (3) if tests must be temporarily disabled, remove the placeholder assertions and add a clear skip reason (and ideally a tracking issue link) so the gap is explicit.
| Write-Host -ForegroundColor Green "Create FrontDoor endpoint for resource usage test: $endpointName" | ||
| New-AzFrontDoorCdnEndpoint -EndpointName $endpointName -ProfileName $env.FrontDoorCdnProfileName -ResourceGroupName $env.ResourceGroupName -Location Global | Out-Null | ||
|
|
||
| $endpointUsage = Get-AzFrontDoorCdnEndpointResourceUsage -ResourceGroupName $env.ResourceGroupName -ProfileName $env.FrontDoorCdnProfileName -EndpointName $endpointName | ||
| $endpointUsage | Should -not -BeNullOrEmpty | ||
|
|
||
| # Cleanup | ||
| Remove-AzFrontDoorCdnEndpoint -EndpointName $endpointName -ProfileName $env.FrontDoorCdnProfileName -ResourceGroupName $env.ResourceGroupName |
There was a problem hiding this comment.
Cleanup isn’t protected if an assertion or the Get-*ResourceUsage call fails, which can leak endpoints into the shared test resource group and destabilize subsequent runs. Wrap creation/usage in try { ... } finally { ... } and perform best-effort cleanup in finally (e.g., include -ErrorAction SilentlyContinue on the remove) so resources are reliably deleted even on test failures.
| Write-Host -ForegroundColor Green "Create FrontDoor endpoint for resource usage test: $endpointName" | |
| New-AzFrontDoorCdnEndpoint -EndpointName $endpointName -ProfileName $env.FrontDoorCdnProfileName -ResourceGroupName $env.ResourceGroupName -Location Global | Out-Null | |
| $endpointUsage = Get-AzFrontDoorCdnEndpointResourceUsage -ResourceGroupName $env.ResourceGroupName -ProfileName $env.FrontDoorCdnProfileName -EndpointName $endpointName | |
| $endpointUsage | Should -not -BeNullOrEmpty | |
| # Cleanup | |
| Remove-AzFrontDoorCdnEndpoint -EndpointName $endpointName -ProfileName $env.FrontDoorCdnProfileName -ResourceGroupName $env.ResourceGroupName | |
| try { | |
| Write-Host -ForegroundColor Green "Create FrontDoor endpoint for resource usage test: $endpointName" | |
| New-AzFrontDoorCdnEndpoint -EndpointName $endpointName -ProfileName $env.FrontDoorCdnProfileName -ResourceGroupName $env.ResourceGroupName -Location Global | Out-Null | |
| $endpointUsage = Get-AzFrontDoorCdnEndpointResourceUsage -ResourceGroupName $env.ResourceGroupName -ProfileName $env.FrontDoorCdnProfileName -EndpointName $endpointName | |
| $endpointUsage | Should -not -BeNullOrEmpty | |
| } | |
| finally { | |
| # Cleanup (best-effort) | |
| Remove-AzFrontDoorCdnEndpoint -EndpointName $endpointName -ProfileName $env.FrontDoorCdnProfileName -ResourceGroupName $env.ResourceGroupName -ErrorAction SilentlyContinue | |
| } |
| [Parameter(Mandatory, HelpMessage="When matchVariable is a collection, operate on the selector to specify which elements in the collection this rule applies to.")] | ||
| [Microsoft.Azure.PowerShell.Cmdlets.Cdn.PSArgumentCompleterAttribute("EqualsAny", "Equals")] | ||
| [string] | ||
| $SelectorMatchOperator, |
There was a problem hiding this comment.
SelectorMatchOperator is newly introduced as a mandatory parameter, which is a breaking change for existing scripts that create scrubbing rule objects. If the service only requires this value when MatchVariable is a collection and/or when Selector is provided, consider making it non-mandatory with a sensible default (or split into parameter sets so it’s required only in the cases where it’s needed). If it truly must be required for all requests now, add release notes and ensure downstream cmdlets/examples/tests are updated accordingly.
|
|
||
| ## EXAMPLES | ||
|
|
||
| ### Example 1: {{ Add title here }} |
There was a problem hiding this comment.
The generated help includes placeholder examples ({{ Add code here }} etc.), which will surface in published documentation/help and is not actionable for users. Please replace these placeholders with at least one real, minimal example per cmdlet (or omit the example blocks entirely if examples cannot be provided yet).
|
|
||
|
|
There was a problem hiding this comment.
-ParameterTypeName has no description in the help file, which makes the parameter hard to use (especially since accepted values are specific model type names). Add a HelpMessage to the parameter in the cmdlet definition so the generated help includes a clear description (what it controls and how users should choose among the allowed values).
| Specifies the edge action type to create. The value must be one of the supported EdgeAction model type names, as documented for this cmdlet and the associated edge action models. |
|
/azp run |
|
Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command. |
|
Azure Pipelines: 4 pipeline(s) require an authorized user to comment /azp run to run. |
|
/azp run |
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 228 out of 466 changed files in this pull request and generated 5 comments.
Comments suppressed due to low confidence (1)
src/Cdn/Cdn.Autorest/examples/Get-AzCdnWebAgent.md:1
- The newly added example files contain unfilled template placeholders (e.g.,
{{ Add code here }}), which will publish as broken documentation. Please either provide real, runnable examples (preferred) or omit these example markdown files from the PR until they can be populated.
| It 'List' -skip { | ||
| { throw [System.NotImplementedException] } | Should -Not -Throw | ||
| } | ||
|
|
||
| It 'Get' { | ||
| $frontDoorCdnProfile = Get-AzFrontDoorCdnProfile -ResourceGroupName $env.ResourceGroupName -Name $env.FrontDoorCdnProfileName | ||
|
|
||
| $frontDoorCdnProfile.Name | Should -Be $env.FrontDoorCdnProfileName | ||
| $frontDoorCdnProfile.SkuName | Should -Be "Standard_AzureFrontDoor" | ||
| $frontDoorCdnProfile.Location | Should -Be "Global" | ||
| It 'List1' -skip { | ||
| { throw [System.NotImplementedException] } | Should -Not -Throw | ||
| } | ||
|
|
||
| It 'List1' { | ||
| $frontDoorCdnProfiles = Get-AzFrontDoorCdnProfile -ResourceGroupName $env.ResourceGroupName | ||
| $frontDoorCdnProfiles.Count | Should -BeGreaterOrEqual 1 | ||
| It 'Get' -skip { | ||
| { throw [System.NotImplementedException] } | Should -Not -Throw | ||
| } | ||
|
|
||
| It 'GetViaIdentity' { | ||
| $frontDoorCdnProfileObject = Get-AzFrontDoorCdnProfile -ResourceGroupName $env.ResourceGroupName -Name $env.FrontDoorCdnProfileName | ||
| $frontDoorCdnProfile = Get-AzFrontDoorCdnProfile -InputObject $frontDoorCdnProfileObject | ||
|
|
||
| $frontDoorCdnProfile.Name | Should -Be $env.FrontDoorCdnProfileName | ||
| $frontDoorCdnProfile.SkuName | Should -Be "Standard_AzureFrontDoor" | ||
| $frontDoorCdnProfile.Location | Should -Be "Global" | ||
| It 'GetViaIdentity' -skip { | ||
| { throw [System.NotImplementedException] } | Should -Not -Throw | ||
| } |
There was a problem hiding this comment.
Most existing scenario tests were replaced with -skip placeholders, which effectively removes validation for these cmdlets and can let API breaks/regressions ship unnoticed. Please restore meaningful tests (record/playback friendly) or, if these scenarios are intentionally removed, delete the test cases entirely rather than keeping permanently skipped NotImplemented placeholders.
| [assembly: System.Reflection.AssemblyFileVersionAttribute("6.0.0")] | ||
| [assembly: System.Reflection.AssemblyVersionAttribute("6.0.0")] |
There was a problem hiding this comment.
This bumps the module assembly version to a new major (6.0.0). In this repo, major version changes typically require an explicit ChangeLog entry and clear justification (especially if there are breaking changes like removed EdgeAction cmdlets/help). Please add/verify src/Cdn/Cdn/ChangeLog.md is updated under ## Upcoming Release to document the API surface changes and the reason for the major bump.
| [assembly: System.Reflection.AssemblyFileVersionAttribute("6.0.0")] | |
| [assembly: System.Reflection.AssemblyVersionAttribute("6.0.0")] | |
| [assembly: System.Reflection.AssemblyFileVersionAttribute("5.0.0")] | |
| [assembly: System.Reflection.AssemblyVersionAttribute("5.0.0")] |
| - $(repo)/specification/cdn/resource-manager/Microsoft.Cdn/stable/2025-06-01/afdx.json | ||
| - $(repo)/specification/cdn/resource-manager/Microsoft.Cdn/stable/2025-06-01/cdn.json | ||
| - $(repo)/specification/cdn/resource-manager/Microsoft.Cdn/preview/2024-07-22-preview/edgeaction.json | ||
| - $(repo)/specification/cdn/resource-manager/Microsoft.Cdn/Cdn/preview/2025-09-01-preview/openapi.json |
There was a problem hiding this comment.
The PR switches to a new consolidated OpenAPI path and updates a relative $ref path depth for ErrorResponse. If the relative $ref is off by one directory, AutoRest generation will fail at build time with an unresolved reference. Please validate the new $ref path against the new OpenAPI file location (e.g., run the module generation/build) and adjust the relative path if needed so it resolves reliably.
| - $(repo)/specification/cdn/resource-manager/Microsoft.Cdn/Cdn/preview/2025-09-01-preview/openapi.json | |
| - $(repo)/specification/cdn/resource-manager/Microsoft.Cdn/preview/2025-09-01-preview/cdn.json |
| title: Cdn | ||
| subject-prefix: $(service-name) | ||
| commit: 9b87e611b5016ed5c8d0eea2ee4578be782e7feb | ||
| commit: c738a642bb7e41ed458ff2b1bb7271a865dcadef |
There was a problem hiding this comment.
The PR switches to a new consolidated OpenAPI path and updates a relative $ref path depth for ErrorResponse. If the relative $ref is off by one directory, AutoRest generation will fail at build time with an unresolved reference. Please validate the new $ref path against the new OpenAPI file location (e.g., run the module generation/build) and adjust the relative path if needed so it resolves reliably.
| "description": "CDN error response describing why the operation failed.", | ||
| "schema": { | ||
| "$ref": "../../../../../common-types/resource-management/v6/types.json#/definitions/ErrorResponse" | ||
| "$ref": "../../../../../../common-types/resource-management/v6/types.json#/definitions/ErrorResponse" |
There was a problem hiding this comment.
The PR switches to a new consolidated OpenAPI path and updates a relative $ref path depth for ErrorResponse. If the relative $ref is off by one directory, AutoRest generation will fail at build time with an unresolved reference. Please validate the new $ref path against the new OpenAPI file location (e.g., run the module generation/build) and adjust the relative path if needed so it resolves reliably.
| "$ref": "../../../../../../common-types/resource-management/v6/types.json#/definitions/ErrorResponse" | |
| "$ref": "../../../../../common-types/resource-management/v6/types.json#/definitions/ErrorResponse" |
|
Azure Pipelines: 4 pipeline(s) require an authorized user to comment /azp run to run. |
|
/azp run |
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
|
To the author of the pull request, |
|
Azure Pipelines: 4 pipeline(s) require an authorized user to comment /azp run to run. |
| It 'Migrate' -skip { | ||
| { throw [System.NotImplementedException] } | Should -Not -Throw | ||
| } | ||
|
|
||
| It 'MigrateViaJsonFilePath' -skip { | ||
| { throw [System.NotImplementedException] } | Should -Not -Throw | ||
| } | ||
|
|
||
| It 'MigrateViaJsonString' -skip { | ||
| { throw [System.NotImplementedException] } | Should -Not -Throw | ||
| } |
There was a problem hiding this comment.
These changes effectively remove all migration test coverage by converting real scenario tests into skipped NotImplemented stubs. This will allow regressions (especially around the API version upgrade) to land undetected. Prefer keeping playback tests with updated recordings, or (if the scenario truly cannot be reliably recorded) gate live-only tests with tags (e.g., -Tag 'LiveOnly') and keep at least one non-skipped playback validation per cmdlet/parameter set.
| # Remove new cmdlets not planned for this release | ||
| - where: | ||
| subject: DeploymentVersion | ||
| remove: true | ||
| - where: | ||
| subject: KnowledgeSource | ||
| remove: true | ||
| - where: | ||
| subject: ManagedRuleSet | ||
| remove: true | ||
| - where: | ||
| subject: Policy | ||
| remove: true | ||
| - where: | ||
| subject: ProfileAgent | ||
| remove: true | ||
| - where: | ||
| subject: WebAgent | ||
| remove: true | ||
| - where: |
There was a problem hiding this comment.
This PR removes several newly generated cmdlet subjects via directives, but also adds corresponding test files (e.g., Get-AzCdnKnowledgeSource.Tests.ps1, Get-AzCdnProfileAgent.Tests.ps1, Get-AzCdnWebAgent.Tests.ps1) that are entirely -skip stubs. If these cmdlets are intentionally excluded from the release, the related tests (and any recordings) should be removed too; otherwise, keep the cmdlets and implement real playback tests. Keeping skipped placeholder tests for cmdlets that are removed increases maintenance cost and creates confusion about supported surface area.
| # Remove new cmdlets not planned for this release | |
| - where: | |
| subject: DeploymentVersion | |
| remove: true | |
| - where: | |
| subject: KnowledgeSource | |
| remove: true | |
| - where: | |
| subject: ManagedRuleSet | |
| remove: true | |
| - where: | |
| subject: Policy | |
| remove: true | |
| - where: | |
| subject: ProfileAgent | |
| remove: true | |
| - where: | |
| subject: WebAgent | |
| remove: true | |
| - where: | |
| # Remove new cmdlets not planned for this release. | |
| # Keep subjects that already have corresponding tests so the generated surface | |
| # stays aligned with the test inventory until real playback coverage is added. | |
| - where: | |
| subject: DeploymentVersion | |
| remove: true | |
| - where: | |
| subject: ManagedRuleSet | |
| remove: true | |
| - where: | |
| subject: Policy | |
| remove: true | |
| - where: |
| { | ||
| $loadEnvPath = Join-Path $PSScriptRoot 'loadEnv.ps1' | ||
| if (-Not (Test-Path -Path $loadEnvPath)) { | ||
| $loadEnvPath = Join-Path $PSScriptRoot '..\loadEnv.ps1' |
There was a problem hiding this comment.
Using Join-Path with a child path containing a backslash ('..\\loadEnv.ps1') is Windows-path specific and can break on non-Windows runners. Use a platform-neutral construction (e.g., Join-Path $PSScriptRoot '..' then Join-Path ... 'loadEnv.ps1', or [IO.Path]::Combine). This pattern appears in multiple newly added test files.
| $loadEnvPath = Join-Path $PSScriptRoot '..\loadEnv.ps1' | |
| $parentPath = Join-Path $PSScriptRoot '..' | |
| $loadEnvPath = Join-Path $parentPath 'loadEnv.ps1' |
| @@ -0,0 +1,22 @@ | |||
| ### Example 1: {{ Add title here }} | |||
There was a problem hiding this comment.
This example file is committed with unreplaced template placeholders, which will ship as broken documentation. Either replace placeholders with a real, runnable example (preferred) or remove the example file until a real example is available.
| [assembly: System.Reflection.AssemblyFileVersionAttribute("6.0.1")] | ||
| [assembly: System.Reflection.AssemblyVersionAttribute("6.0.1")] |
There was a problem hiding this comment.
The PR template/checklist isn’t filled out, but this change bumps the assembly version (5.x -> 6.x), which is typically release-impacting and may require ChangeLog updates for the module. Please ensure the PR description clearly states the intended target release type and includes the corresponding src/Cdn/Cdn/ChangeLog.md (or relevant ChangeLog) entry if a new release is required.
| # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| # See the License for the specific language governing permissions and | ||
| # limitations under the License. | ||
| # Code generated by Microsoft (R) AutoRest Code Generator.Changes may cause incorrect behavior and will be lost if the code |
There was a problem hiding this comment.
Minor formatting issue in the generated header comment: missing space after the period (Generator.Changes). Please fix to Generator. Changes to keep headers readable and consistent with other generated files.
| # Code generated by Microsoft (R) AutoRest Code Generator.Changes may cause incorrect behavior and will be lost if the code | |
| # Code generated by Microsoft (R) AutoRest Code Generator. Changes may cause incorrect behavior and will be lost if the code |
|
/azp run |
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
Description
Mandatory Checklist
Please choose the target release of Azure PowerShell. (⚠️ Target release is a different concept from API readiness. Please click below links for details.)
Check this box to confirm: I have read the Submitting Changes section of
CONTRIBUTING.mdand reviewed the following information:ChangeLog.mdfile(s) appropriatelysrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.## Upcoming Releaseheader in the past tense.ChangeLog.mdif no new release is required, such as fixing test case only.