Skip to content

Enhance role-definitions.md with control plane examples - #128683

Open
Rafael Fernández (rfernandezdo) wants to merge 1 commit into
MicrosoftDocs:mainfrom
rfernandezdo:patch-6
Open

Enhance role-definitions.md with control plane examples#128683
Rafael Fernández (rfernandezdo) wants to merge 1 commit into
MicrosoftDocs:mainfrom
rfernandezdo:patch-6

Conversation

@rfernandezdo

Copy link
Copy Markdown
Contributor

Added examples and explanations for control plane permissions, highlighting the interaction between Actions and NotActions.

Added examples and explanations for control plane permissions, highlighting the interaction between Actions and NotActions.
Copilot AI lite review requested due to automatic review settings August 4, 2026 15:25
@prmerger-automator

Copy link
Copy Markdown
Contributor

Rafael Fernández (@rfernandezdo) : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. Robert Lyon (@rolyon)

1 similar comment
@prmerger-automator

Copy link
Copy Markdown
Contributor

Rafael Fernández (@rfernandezdo) : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. Robert Lyon (@rolyon)

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Enhances the RBAC role definitions documentation by adding more control-plane examples to clarify how Actions and NotActions combine, especially when they overlap.

Changes:

  • Added a new table with overlapping Actions/NotActions examples and their effective permissions.
  • Added explanatory bullets describing precedence/overlap behavior.
  • Extended the NOTE with guidance about validating Actions/NotActions entries.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +343 to +344
> | `*`</br>`Microsoft.Network/virtualNetworks/subnets/join/action` | `Microsoft.Network/*` | All control plane permissions except `Microsoft.Network/*`</br>`Microsoft.Network/virtualNetworks/subnets/join/action` is excluded |
> | `Microsoft.Authorization/*/read` | `Microsoft.Authorization/*/read` | *none* |
> [!div class="mx-tableFixed"]
> | Actions | NotActions | Effective control plane permissions |
> | --- | --- | --- |
> | `*`</br>`Microsoft.Network/virtualNetworks/subnets/join/action` | `Microsoft.Network/*` | All control plane permissions except `Microsoft.Network/*`</br>`Microsoft.Network/virtualNetworks/subnets/join/action` is excluded |
> [!NOTE]
> If a user is assigned a role that excludes an action in `NotActions`, and is assigned a second role that grants access to the same action, the user is allowed to perform that action. `NotActions` is not a deny rule – it is simply a convenient way to create a set of allowed actions when specific actions need to be excluded.
>
> Also note that `NotActions` only excludes actions that match valid Azure resource provider operations. If an entry in `NotActions` does not correspond to a valid operation pattern, it does not exclude anything in practice. For this reason, both `Actions` and `NotActions` should be validated against the current Azure resource provider operations catalog.
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit ecff47a:

✅ Validation status: passed

File Status Preview URL Details
articles/role-based-access-control/role-definitions.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards

Copy link
Copy Markdown
Contributor

Robert Lyon (@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

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged tracking label for the PR review team label Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants