-
Notifications
You must be signed in to change notification settings - Fork 106
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
Update container-instances-quickstart-powershell.md #37
base: main
Are you sure you want to change the base?
Conversation
Correctly reference New-AzContainerGroup. The snippet incorrectly references New-AzContainerInstanceObject, the above copy on line 55 currently states to use New-AzContainerGroup. Copy also incorrectly states a status will be returned, so I updated it so that it only instructs the user to check the deployment state with Get-AzContainerGroup. container needs to be a container object, not a string
…t-Powershell-Doc-Change Update container-instances-quickstart-powershell.md
@blakedeckard : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit 515be85: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
@tomvcassidy Important: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
This pull request has been inactive for at least 14 days. If you are finished with your changes, don't forget to sign off. See the contributor guide for instructions. |
Correctly reference New-AzContainerGroup. The snippet incorrectly references New-AzContainerInstanceObject, the above copy on line 55 currently states to use New-AzContainerGroup.
Copy also incorrectly states a status will be returned, so I updated it so that it only instructs the user to check the deployment state with Get-AzContainerGroup.
The container needs to be a container object, not a string as well.
See: MicrosoftDocs/azure-docs#124082