Skip to content
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.

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

VPN: P2SVpnGateway.P2SConnectionConfigurationProperties.configurationPolicyGroupAssociations property should not be readonly. #30698 #33083

Open
wants to merge 3 commits into
base: kamboj-prjwl/release-network-Microsoft.Network-2024-07-01
Choose a base branch
from

Conversation

Nilambari
Copy link
Member

@Nilambari Nilambari commented Mar 7, 2025

VPN: P2SVpnGateway.P2SConnectionConfigurationProperties.configurationPolicyGroupAssociations property should not be readonly.

Choose a PR Template

Switch to "Preview" on this description then select one of the choices below.

Click here to open a PR for a Data Plane API.

Click here to open a PR for a Control Plane (ARM) API.

Click here to open a PR for only SDK configuration.

…PolicyGroupAssociations property should not be readonly.
@Nilambari Nilambari requested a review from kamboj-prjwl as a code owner March 7, 2025 19:40
Copy link

openapi-pipeline-app bot commented Mar 7, 2025

Next Steps to Merge

✅ All automated merging requirements have been met! To get your PR merged, see aka.ms/azsdk/specreview/merge.

Copy link

openapi-pipeline-app bot commented Mar 7, 2025

Generated ApiView

Language Package Name ApiView Link
Go sdk/resourcemanager/network/armnetwork Create ApiView timeout. Package is too large and we cannot create ApiView for it.
JavaScript @azure/arm-network https://apiview.dev/Assemblies/Review/c13fd66ecbcc4c2a9995c8a1f4187240?revisionId=23ea5d7681244922b1c9af0e4b878046

@github-actions github-actions bot added the brownfield Brownfield services will soon be required to convert to TypeSpec. See https://aka.ms/azsdk/typespec. label Mar 7, 2025
Copy link
Member

@rkmanda rkmanda left a comment

Choose a reason for hiding this comment

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

:shipit:

@rkmanda rkmanda added the ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review label Mar 9, 2025
@openapi-pipeline-app openapi-pipeline-app bot removed the WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required label Mar 9, 2025
@Nilambari
Copy link
Member Author

:shipit:

Thanks a lot.

…01' into nilamd_kamboj-prjwl/release-network-Microsoft.Network-2024-07-01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ARMReview ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review brownfield Brownfield services will soon be required to convert to TypeSpec. See https://aka.ms/azsdk/typespec. Network resource-manager
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants