MicrosoftDocs / azure-docs

Open source documentation of Microsoft Azure
https://docs.microsoft.com/azure
Creative Commons Attribution 4.0 International
10.3k stars 21.48k forks source link

Update containers.md #124862

Open 4c74356b41 opened 1 week ago

4c74356b41 commented 1 week ago

there is an obvious typo, however ot sure that is right, as such permissions dont exist at all accodring to:

Get-AzProviderOperation -OperationSearchString 'Microsoft.ContainerService/managedClusters/*' | ? { $_.Operation -match 'snapshot' } | ft
prmerger-automator[bot] commented 1 week ago

@4c74356b41 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

4c74356b41 commented 1 week ago

oh wait, it actually exists. but why is this operation name wrong than? shouldn't it be managedClusterSnapshots? why is managedcluster single and not plural? confusing stuff.

learn-build-service-prod[bot] commented 1 week ago

Learn Build status updates of commit 66ff8ad:

:white_check_mark: Validation status: passed

File Status Preview URL Details
articles/role-based-access-control/permissions/containers.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

Jak-MS commented 1 week ago

@rolyon Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

label:"aq-pr-triaged"

@MicrosoftDocs/public-repo-pr-review-team

4c74356b41 commented 2 days ago

@rolyon