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.

Sign up for GitHub

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

Jump to bottom

[External/Tracking] update needed? Option '--sdk-auth' has been deprecated and will be removed in a future release. #178

Closed
annasoracco opened this issue Mar 18, 2023 · 2 comments · Fixed by #203
Closed

[External/Tracking] update needed? Option '--sdk-auth' has been deprecated and will be removed in a future release. #178

annasoracco opened this issue Mar 18, 2023 · 2 comments · Fixed by #203
Labels
external
Milestone
Sprint 7

Comments

@annasoracco
Copy link
Member

Create an Azure Service Principal and add it the Owner role in your Azure subscription:

az ad sp create-for-rbac -n contosotraders-sp --role Owner --scopes /subscriptions/ --sdk-auth. Replace with your Azure subscription ID.
Please make a note of the JSON output from above step (especially the clientId, clientSecret, subscriptionId and tenantId properties). These will be required later.

see screenshot
I am going through the deployment instructions, on this step ^ I get "Option '--sdk-auth' has been deprecated and will be removed in a future release

2023-03-18_sdk-auth_issue

Do we need an update?
@annasoracco annasoracco added the investigation needed This doesn't seem right label Mar 18, 2023
@mithunshanbhag mithunshanbhag added this to the Sprint 7 milestone Mar 19, 2023
@mithunshanbhag
Copy link
Collaborator

That deprecation warning has been there for years. The product team (Azure AD) hasn't provided a workaround yet.
Azure/azure-cli#20743

@mithunshanbhag mithunshanbhag changed the title update needed? Option '--sdk-auth' has been deprecated and will be removed in a future release. [External/Tracking] update needed? Option '--sdk-auth' has been deprecated and will be removed in a future release. Mar 20, 2023
@mithunshanbhag mithunshanbhag added external and removed investigation needed This doesn't seem right labels Mar 22, 2023
@mithunshanbhag
Copy link
Collaborator

Added necessary documentation updates in private branch. Will merge into main after code-review: 7d46877

@mithunshanbhag mithunshanbhag mentioned this issue Apr 12, 2023
Merged
@mithunshanbhag mithunshanbhag linked a pull request Apr 12, 2023 that will close this issue
Documentation updates. Fixes #198, #178, #199, #187 #203
Merged
mithunshanbhag added a commit that referenced this issue Apr 12, 2023
@mithunshanbhag
Documentation updates. Fixes #198, #178, #199, #187 (#203)
ee2d72f 
Doc updates. Fixes:
- #198 
- #178 
- #199
- #187
bovrhovn pushed a commit to vrhovnik/azure-sample-cloudtesting that referenced this issue Apr 15, 2024
@mithunshanbhag
Documentation updates. Fixes microsoft#198, microsoft#178, microsoft#199
b98d17f 
, microsoft#187  (microsoft#203)

Doc updates. Fixes:
- microsoft#198 
- microsoft#178 
- microsoft#199
- microsoft#187
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
external
Projects
None yet
Milestone
Sprint 7
Development

Successfully merging a pull request may close this issue.

Documentation updates. Fixes #198, #178, #199, #187 microsoft/contosotraders-cloudtesting
2 participants
@mithunshanbhag @annasoracco