Skip to content

Update "Create a service connection for app registration authentication in Azure DevOps" - #14393

Open
Ronald Bosma (ronaldbosma) wants to merge 3 commits into
MicrosoftDocs:mainfrom
ronaldbosma:update-serivce-connection-steps-in-configure-workload-identity
Open

Update "Create a service connection for app registration authentication in Azure DevOps"#14393
Ronald Bosma (ronaldbosma) wants to merge 3 commits into
MicrosoftDocs:mainfrom
ronaldbosma:update-serivce-connection-steps-in-configure-workload-identity

Conversation

@ronaldbosma

Copy link
Copy Markdown

The screen to create a new service connection with type "App registration or Managed identity (manual)" has changed. The Environment and Directory (tenant) ID fields are now part of the first step instead of the second step. I've updated the text for both the managed identity and app registration versions to reflect this.

@prmerger-automator

Copy link
Copy Markdown
Contributor

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

@prmerger-automator

Copy link
Copy Markdown
Contributor

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

@v-regandowner

Copy link
Copy Markdown
Contributor

Rami Bououni (@ramiMSFT)

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

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

Updates the workload identity federation service connection setup instructions to match the latest Azure DevOps UI, where Environment and Directory (tenant) ID are collected in Step 1: Basics instead of later in the flow.

Changes:

  • Reworks the managed identity service connection instructions to document Step 1: Basics parameters (including Environment and Directory (tenant) ID).
  • Reworks the app registration service connection instructions similarly to align with the updated wizard layout.
  • Simplifies the Authentication section wording by removing now-duplicated tenant/environment instructions from later steps.
Suppressed comments (2)

docs/pipelines/release/configure-workload-identity.md:278

  • This sentence repeats "following parameters" twice, which reads awkwardly. Consider simplifying it to a single instruction line.
    **Step 1: Basics** contains the following parameters. Enter or select the following parameters:

docs/pipelines/release/configure-workload-identity.md:282

  • The UI field name is typically written in sentence case as "Service connection name" across the docs (for example, docs/pipelines/library/add-devops-entra-service-connection.md:74). Using title case here is inconsistent.
   | **Service Connection Name** | Required. You'll use this value in your federated credential subject identifier. |

Comment thread docs/pipelines/release/configure-workload-identity.md Outdated
Comment thread docs/pipelines/release/configure-workload-identity.md
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.

4 participants