[DdoS]Add DDoS custom policy cmdlets, LB integration, tests, and help#29300
[DdoS]Add DDoS custom policy cmdlets, LB integration, tests, and help#29300venkatasaimaram7 wants to merge 5 commits intoAzure:mainfrom
Conversation
| Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. |
|
Thank you for your contribution @venkatasaimaram7! We will review the pull request and get back to you soon. |
|
/azp run |
|
Commenter does not have sufficient privileges for PR 29300 in repo Azure/azure-powershell |
@microsoft-github-policy-service agree company="Microsoft" |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
0696b2c to
3c8353f
Compare
|
/azp run |
|
Commenter does not have sufficient privileges for PR 29300 in repo Azure/azure-powershell |
|
| /// Gets the list of frontend IP configurations associated with the | ||
| /// custom policy. This list is read-only. | ||
| /// </summary> | ||
| [Newtonsoft.Json.JsonProperty(PropertyName = "properties.frontEndIpConfiguration")] |
There was a problem hiding this comment.
Why did the casing change here? Is the auto-generated?
There was a problem hiding this comment.
The GET response returns a collection, so the wire mapping needs to be plural: frontendIpConfigurations.
|
/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. |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Commenter does not have sufficient privileges for PR 29300 in repo Azure/azure-powershell |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
058fad2 to
97e9faf
Compare
bb71c12 to
3288a79
Compare
3288a79 to
2db0eeb
Compare
|
/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. |
## Description
This PR added DDoS Custom Policy support in Az.Network, including new cmdlets, load balancer frontend integration, model/mapping updates, help docs, and scenario test coverage.
Changes included:
Cmdlet design review:
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:src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.## Upcoming Releaseheader in the past tense.## Validation
- Validation artifact:
-532462766(tool/runtime issue).