Skip to content

Upgrade Cdn 2025-09-01-preview, move Edge action out of cdn #29354

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Ptnan7 wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Upgrade Cdn 2025-09-01-preview, move Edge action out of cdn #29354

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
4fbe88d
generate with 2025-09-01-p
Mar 12, 2026
70dbd04
clean tests
Mar 30, 2026
2995adb
test
Mar 31, 2026
569746b
run test
Apr 1, 2026
8d7b921
complete test
Apr 1, 2026
b28e364
re gen
Apr 15, 2026
69f7c76
Merge branch 'main' into edgeaction-opti-test
Apr 15, 2026
bfc01cf
regen
Apr 15, 2026
f9ca3d9
remove unused
Apr 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
5 changes: 3 additions & 2 deletions src/Cdn/Cdn.Autorest/Properties/AssemblyInfo.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,8 @@
[assembly: System.Reflection.AssemblyCopyrightAttribute("Copyright © Microsoft")]
[assembly: System.Reflection.AssemblyProductAttribute("Microsoft Azure PowerShell")]
[assembly: System.Reflection.AssemblyTitleAttribute("Microsoft Azure PowerShell - Cdn")]
[assembly: System.Reflection.AssemblyFileVersionAttribute("5.0.1")]
[assembly: System.Reflection.AssemblyVersionAttribute("5.0.1")]
[assembly: System.Reflection.AssemblyFileVersionAttribute("6.0.1")]
[assembly: System.Reflection.AssemblyVersionAttribute("6.0.1")]
Comment on lines +23 to +24
Copy link

Copilot AI Apr 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copilot uses AI. Check for mistakes.
[assembly: System.Runtime.InteropServices.ComVisibleAttribute(false)]
[assembly: System.CLSCompliantAttribute(false)]

31 changes: 26 additions & 5 deletions src/Cdn/Cdn.Autorest/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,9 +32,7 @@ require:
- $(this-folder)/../../readme.azure.noprofile.md
input-file:
# You need to specify your swagger files here.
- $(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
Copy link

Copilot AI Apr 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Suggested change
- $(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

Copilot uses AI. Check for mistakes.

# If the swagger has not been put in the repo, you may uncomment the following line and refer to it locally
# - (this-folder)/relative-path-to-your-swagger
Expand All @@ -44,7 +42,7 @@ module-version: 0.1.0
# Normally, title is the service name
title: Cdn
subject-prefix: $(service-name)
commit: 9b87e611b5016ed5c8d0eea2ee4578be782e7feb
commit: c738a642bb7e41ed458ff2b1bb7271a865dcadef
Copy link

Copilot AI Apr 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copilot uses AI. Check for mistakes.

# If there are post APIs for some kinds of actions in the RP, you may need to
# uncomment following line to support viaIdentity for these post APIs
Expand Down Expand Up @@ -197,6 +195,9 @@ directive:
# cmdlet-name: New-AzFrontDoorCdnRuleUrlRedirectActionObject
# - model-name: UrlRewriteAction
# cmdlet-name: New-AzFrontDoorCdnRuleUrlRewriteActionObject
# Generate the new Front Door CDN rule edge action helper object from the 2025-09-01-preview model.
- model-name: EdgeAction
cmdlet-name: New-AzFrontDoorCdnRuleEdgeActionObject
# - model-name: UrlAction
# cmdlet-name: New-AzFrontDoorCdnRuleUrlActionObject
# - model-name: UrlSigningAction
Expand Down Expand Up @@ -272,6 +273,26 @@ directive:
variant: ^Create$|^CreateViaIdentity$|^CreateViaIdentityExpanded$|^Update$|^UpdateViaIdentity$|^Patch$|^PatchViaIdentity$|^Check$|^Check1$|^CheckViaIdentity$|^Validate$|^ValidateViaIdentity$
subject: ^(?!RuleSet).+$
remove: true

# 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:
Comment on lines +277 to 296
Copy link

Copilot AI Apr 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Suggested change
# 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:

Copilot uses AI. Check for mistakes.
variant: ^CreateViaIdentity$|^CreateViaIdentityExpanded$|^Update$|^UpdateViaIdentity$
subject: ^RuleSet$
Expand Down Expand Up @@ -469,7 +490,7 @@ directive:
"default": {
"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"
Copy link

Copilot AI Apr 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Suggested change
"$ref": "../../../../../../common-types/resource-management/v6/types.json#/definitions/ErrorResponse"
"$ref": "../../../../../common-types/resource-management/v6/types.json#/definitions/ErrorResponse"

Copilot uses AI. Check for mistakes.
}
}
}
Expand Down
95 changes: 0 additions & 95 deletions src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/edgeActions-executionFilters.json
View file
Open in desktop

This file was deleted.

138 changes: 0 additions & 138 deletions src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/edgeActions-versions.json
View file
Open in desktop

This file was deleted.

85 changes: 0 additions & 85 deletions src/Cdn/Cdn.Autorest/UX/Microsoft.Cdn/edgeActions.json
View file
Open in desktop

This file was deleted.

Loading
Loading