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

Cmdlet reference doc - New-AzDataCollectionEndpoint - Required parameter conflict #24758

Open
Get-Ryan opened this issue Apr 29, 2024 · 3 comments
Assignees
Labels
customer-reported Monitor Service Attention This issue is responsible by Azure service team.

Comments

@Get-Ryan
Copy link

Type of issue

Other (describe below)

Feedback

Under the Parameters heading, the parameter NetworkAclsPublicNetworkAccess is listed as not being required. The Syntax area also doesn't indicate it is required. However, both examples shown include a value for that parameter. And when trying to use the cmdlet without that parameter resulted in an error message "Invalid resource payload: 'properties' are missing". Running the same set of parameters with the addition of "Enabled" for the NetworkAclsPublicNetworkAccess parameter resulted in success.

This would indicate the NetworkAclsPublicNetworkAccess parameter is required. The Required field in it's parameter table should be updated to True.

Page URL

https://learn.microsoft.com/en-us/powershell/module/az.monitor/new-azdatacollectionendpoint?view=azps-11.5.0

Content source URL

https://github.com/Azure/azure-powershell/blob/main/src/Monitor/Monitor/help/New-AzDataCollectionEndpoint.md

Author

@mikefrobbins

Document Id

02f95826-d824-38cc-7bff-9b2592ce7fed

@Get-Ryan Get-Ryan added the needs-triage This is a new issue that needs to be triaged to the appropriate team. label Apr 29, 2024
@isra-fel
Copy link
Member

isra-fel commented May 7, 2024

Hi Joyer, please look into this issue. Thanks.

@isra-fel isra-fel added Monitor Azure PS Team Investigate 🔍 and removed needs-triage This is a new issue that needs to be triaged to the appropriate team. labels May 7, 2024
@JoyerJin
Copy link
Contributor

JoyerJin commented May 7, 2024

Hi Yeming, I confirmed with the swagger file, this property in body is not required, only the location is required. required property for body schema swagger file

@isra-fel
Copy link
Member

isra-fel commented May 7, 2024

It looks most likely a bug in swagger where the property is actuall required but not marked as required. Or there's possibility that under some corner cases this parameter is not required.
I'll loop in Monitor team to share the insights.

@isra-fel isra-fel added Service Attention This issue is responsible by Azure service team. and removed Azure PS Team Investigate 🔍 labels May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customer-reported Monitor Service Attention This issue is responsible by Azure service team.
Projects
None yet
Development

No branches or pull requests

3 participants