Skip to content

Conversation

@yugangw-msft
Copy link
Contributor

Notes:

  1. We put 2 commands under account group for the reason that they are for login, not dealing with general service principal. In future, the placement might change.
  2. role assignment create command contains the minimum, and it is not the final version. We will get to it when we are done with the network set commands
  3. The new commands don't follow the default output format, as it means to output the credentials information as clear as possible for people to write down.
  4. The access token acquired for management endpoint (our default case) will not work for the graph endpoint, so the profile management code gets updated to handle the new situation.

@yugangw-msft
Copy link
Contributor Author

/CC:@JasonRShaver, submit this for demo on this Friday

@yugangw-msft
Copy link
Contributor Author

/CC:@BurtBiel @tjprescott

(env) D:\sdk\azure-cli>az account -h

Group
    az account

Commands:
    clear
    create-sp
    list
    list-location
    reset-sp-credentials
    set

(env) D:\sdk\azure-cli>az account create-sp -h

Command
    az account create-sp

Arguments
    --name -n  : An unique uri. If missing, the command will generate one.
    --secret   : The secret used to login. If missing, command will generate one.
    --years    : Years the secret will be valid.  Default: 1.

(env) D:\sdk\azure-cli>az account reset-sp-credentials -h

Command
    az account reset-sp-credentials

Arguments
    --name -n [Required]: The uri representing the name of the service principal.
    --secret            : The secret used to login. If missing, command will generate one.
    --years             : Years the secret will be valid.  Default: 1.

ACTIVE_DIRECTORY_AUTHORITY = 'active_directory_authority'
ACTIVE_DIRECTORY_GRAPH_RESOURCE_ID = 'active_directory_graph_resource_id'

_environments = {
Copy link
Contributor

Choose a reason for hiding this comment

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

black forest?

Copy link
Contributor Author

@yugangw-msft yugangw-msft Jul 7, 2016

Choose a reason for hiding this comment

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

You are right on this, we are missing usgovenment as well. Since we have not exposed the --env flag in login, this can wait till we add that flag.

@yugangw-msft
Copy link
Contributor Author

@BurtBiel, @tjprescott, since this is needed by demo, i am doing optimistic merge, please still feel free to leave comments later.

@yugangw-msft yugangw-msft merged commit 97ecf82 into Azure:master Jul 7, 2016
@yugangw-msft yugangw-msft deleted the rbac branch July 13, 2016 20:01
@jiasli
Copy link
Member

jiasli commented Oct 20, 2023

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.

3 participants