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

Service Principal auth #4255

Merged

Conversation

KonstantinTyukalov
Copy link
Contributor

@KonstantinTyukalov KonstantinTyukalov commented Apr 27, 2023

This PR adds SP auth approach.
To do so, you'll need to set:

  • auth: SP
  • clientid: <your client(app) id>
  • tenantid: <your tenant id>
  • clientsecret: <your client secret value>

Work item: internal 2032949

@KonstantinTyukalov KonstantinTyukalov marked this pull request as ready for review April 28, 2023 11:42
@KonstantinTyukalov KonstantinTyukalov requested review from a team as code owners April 28, 2023 11:42
@akanieski
Copy link

It was mentioned elsewhere about managed identity support, what about client certificate support for the service principal? Many consider client secrets as less ideal (as is suggested by the AAD best practices docs here . I know the agent currently supports client certs, can we just piggy back on those existing configuration options?

@kirill-ivlev
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@avekrivoy
Copy link

When will this feature be available?

@KonstantinTyukalov
Copy link
Contributor Author

Blocked while telemetry changes are not merged: #4333

@ismayilov-ismayil ismayilov-ismayil force-pushed the users/KonstantinTyukalov/ServicePrincipal_auth branch from 4aea33f to a42dc04 Compare August 15, 2023 18:25
@avekrivoy
Copy link

Blocked while telemetry changes are not merged: #4333

I see it's already merged. Any ETA on this? Really waiting for this SP auth feature because I want to get rid of PATs in my agent's configuration

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.

10 participants