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

Add a note to create a context in different subscription #11065

Merged
merged 2 commits into from
Jul 1, 2020

Conversation

sujaypillai
Copy link
Contributor

Proposed changes

If you have already created a ACI context in a subscription then to create a new context in different subscription you need to login again to Azure to the respective account. To do this you need to execute the command docker login azure each time for change in subscription for a new context change.

Added a NOTE section highlighting the same.

Related issues (optional)

#11064

@netlify
Copy link

netlify bot commented Jun 30, 2020

Deploy preview for docsdocker ready!

Built with commit bce0d50

https://deploy-preview-11065--docsdocker.netlify.app

chris-crone
chris-crone previously approved these changes Jul 1, 2020
Copy link
Member

@chris-crone chris-crone left a comment

Choose a reason for hiding this comment

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

LGTM. Just need to check with @usha-mandya if the note format is what we normally do

Thanks @sujaypillai!

@@ -50,6 +50,7 @@ docker login azure

This opens your web browser and prompts you to enter your Azure login credentials.


Copy link
Member

Choose a reason for hiding this comment

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

nit: extra \n

Made a minor tweak to the style and wrapped the new note to 80 chars
@usha-mandya
Copy link
Member

Thank you @sujaypillai and @chris-crone. The note format looks good.

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