Skip to content

Update descriptions for identity source parameters#128293

Open
et1975 wants to merge 1 commit intoMicrosoftDocs:mainfrom
et1975:patch-1
Open

Update descriptions for identity source parameters#128293
et1975 wants to merge 1 commit intoMicrosoftDocs:mainfrom
et1975:patch-1

Conversation

@et1975
Copy link

@et1975 et1975 commented Mar 9, 2026

Clarify the GroupName role/optionality when adding an external source.

Clarify the GroupName role/optionality when adding an external source.
@prmerger-automator
Copy link
Contributor

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

Copy link
Contributor

@boumenot boumenot left a comment

Choose a reason for hiding this comment

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

LGTM

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit fc26b00:

✅ Validation status: passed

File Status Preview URL Details
articles/azure-vmware/configure-identity-source-vcenter.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards v-dirichards requested a review from Copilot March 9, 2026 20:25
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR clarifies the documentation for identity source parameters in vCenter configuration, specifically updating the GroupName field descriptions to indicate it is optional and can be configured separately after the source is added.

Changes:

  • Updated GroupName description in the LDAP with SSL section to indicate it's optional and reference the Add-GroupToCloudAdmins cmdlet
  • Updated GroupName description in the LDAP (non-SSL) section to indicate it's optional with a note that it can be done separately

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

| **BaseDNGroups** | The location to search for groups. For example, **CN=group1, DC=avslab,DC=local**. Base DN is required for LDAP authentication. |
| **Credential** | The domain username and password for authentication with the Windows Server Active Directory source (not CloudAdmin). The user must be in the `<username@avslab.local>` format. |
| **GroupName** | The group in your external identity source that grants CloudAdmin access. For example, **avs-admins**. |
| **GroupName** | Optional. The group in your external identity that will be granted CloudAdmin membership upon addition of the source. For example, **avs-admins**. Can be done separately later. |
Copy link

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

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

The phrase "Can be done separately later." is vague. For consistency with line 117, consider referencing the Add-GroupToCloudAdmins cmdlet explicitly, e.g., "This can be done separately later via Add-GroupToCloudAdmins."

Suggested change
| **GroupName** | Optional. The group in your external identity that will be granted CloudAdmin membership upon addition of the source. For example, **avs-admins**. Can be done separately later. |
| **GroupName** | Optional. The group in your external identity that will be granted CloudAdmin membership upon addition of the source. For example, **avs-admins**. This can be done separately later via `Add-GroupToCloudAdmins`. |

Copilot uses AI. Check for mistakes.
| **BaseDNGroups** | The location to search for groups. For example, **CN=group1, DC=avslab,DC=local**. Base DN is required for LDAP authentication. |
| **Credential** | The domain username and password for authentication with the Windows Server Active Directory source (not CloudAdmin). The user must be in the `<username@avslab.local>` format. |
| **GroupName** | The group in your external identity source that grants CloudAdmin access. For example, **avs-admins**. |
| **GroupName** | Optional. The group in your external identity that will be granted CloudAdmin membership upon addition of the source. For example, **avs-admins**. Can be done separately later. |
Copy link

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

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

The phrase 'external identity' appears to be missing the word 'source' — should read 'external identity source' to match the terminology used throughout the document and on line 117.

Suggested change
| **GroupName** | Optional. The group in your external identity that will be granted CloudAdmin membership upon addition of the source. For example, **avs-admins**. Can be done separately later. |
| **GroupName** | Optional. The group in your external identity source that will be granted CloudAdmin membership upon addition of the source. For example, **avs-admins**. Can be done separately later. |

Copilot uses AI. Check for mistakes.
@v-dirichards
Copy link
Contributor

@jjaygbay1

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 Mar 9, 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.

5 participants