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

{Profile} az login: Refine warning messages #12726

Merged
merged 2 commits into from
Mar 25, 2020
Merged

Conversation

jiasli
Copy link
Member

@jiasli jiasli commented Mar 24, 2020

Refine and unify the warning messages for

MFA tenants (#12516)

The following tenants require Multi-Factor Authentication (MFA). Use 'az login --tenant TENANT_ID' to explicitly login to a tenant.
da086fa3-023f-4500-bf56-0f19d3065440 'Inbarck Labs'
...

Empty tenants (#2929)

The following tenants doesn't contain accessible subscriptions. Use 'az login --allow-no-subscriptions' to have tenant level access.
3f85f91a-b450-4dbb-b0f5-703606d9161f 'Contoso'
...

Multi-tenant subscriptions (#11886)

Subscription 95350823-1b99-4afd-92f8-b1d6db6f7ab2 'Visual Studio Enterprise' can be accessed from tenants 72f988bf-86f1-41af-91ab-2d7cd011db47(default) and 2b8e6bbc-631a-4bf6-b0c6-d4947b3c79dd. To select a specific tenant when accessing this subscription, use 'az login --tenant TENANT_ID'.

⚠ azure-mgmt-resource SDK version is temporarily locked due to missing properties added in Azure/azure-rest-api-specs#8384

@yonzhan yonzhan added this to the S167 milestone Mar 24, 2020
@yonzhan
Copy link
Collaborator

yonzhan commented Mar 24, 2020

add to S167

Copy link
Member

@qianwens qianwens left a comment

Choose a reason for hiding this comment

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

:shipit:

@jiasli jiasli changed the title [Profile] az login: Refine warning messages {Profile} az login: Refine warning messages Mar 25, 2020
@jiasli jiasli merged commit c97d70d into Azure:dev Mar 25, 2020
@jiasli jiasli deleted the login-warning branch March 25, 2020 04:10
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