Skip to content

Conversation

@jiasli
Copy link
Member

@jiasli jiasli commented Oct 8, 2021

Description

#18312 announced the deprecation of name. It is time to drop name property completely.

This eliminates the inconsistency between input parameter --name and output name property.

> az ad sp create-for-rbac --name azure-cli-testsp
In a future release, this command will NOT create a 'Contributor' role assignment by default. If needed, use the --role argument to explicitly create a role assignment.
Creating 'Contributor' role assignment under scope '/subscriptions/0b1f6471-1bf0-4dda-aec3-cb9272f09590'
The output includes credentials that you must protect. Be sure that you do not include these credentials in your code or check the credentials into your source control. For more information, see https://aka.ms/azadsp-cli
'name' property in the output is deprecated and will be removed in the future. Use 'appId' instead.
{
  "appId": "d0afbd9e-4805-408f-a24a-91f5c2a08081",
  "displayName": "azure-cli-testsp",
  "name": "d0afbd9e-4805-408f-a24a-91f5c2a08081",   <<<<<
  "password": "...",
  "tenant": "54826b22-38d6-4fb2-bad9-b7b93a3e9c5a"
}

History Notes

[Role] BREAKING CHANGE: az ad sp create-for-rbac: Drop name property from the output. Use appId instead.

@jiasli jiasli requested a review from evelyn-ys as a code owner October 8, 2021 07:50
@yonzhan
Copy link
Collaborator

yonzhan commented Oct 8, 2021

Role

@yonzhan yonzhan added this to the Oct 2021 (2021-11-02) milestone Oct 8, 2021
@jiasli jiasli marked this pull request as draft November 9, 2021 05:36
@jiasli jiasli marked this pull request as ready for review November 23, 2021 06:32
@jiasli jiasli requested a review from calvinhzy as a code owner November 23, 2021 06:32
@jiasli jiasli merged commit 7abeeb8 into Azure:dev Dec 10, 2021
@jiasli jiasli deleted the rbac-drop-name branch December 10, 2021 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants