Skip to content

Add a test script which is to calculate storage container billable size. #7877

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

Merged
merged 2 commits into from
Nov 26, 2018

Conversation

EmmaZhu
Copy link
Member

@EmmaZhu EmmaZhu commented Nov 21, 2018

Description

Checklist

Copy link
Member

@cormacpayne cormacpayne left a comment

Choose a reason for hiding this comment

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

@EmmaZhu a couple of small comments. Out of curiosity, what is the intended usage of this script?


if((Get-Module -ListAvailable Azure) -eq $null)
{
throw "Azure Powershell not found! Please install from http://www.windowsazure.com/en-us/downloads/#cmd-line-tools"
Copy link
Member

Choose a reason for hiding this comment

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

@EmmaZhu do you have any objection to using the following link instead?

https://docs.microsoft.com/en-us/powershell/azure/install-azurerm-ps

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks for your comments. Changed and tested module installing and checking as the suggestions.

Copy link
Member Author

Choose a reason for hiding this comment

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

To answer the above question"Out of curiosity, what is the intended usage of this script?", seems I cannot reply to it directly:
The script will be a sample for Azure Storage customers to reference to, to get billable size of an Azure Storage container. We will have an article to introduce it.

@cormacpayne cormacpayne merged commit 707f5f9 into Azure:preview Nov 26, 2018
@EmmaZhu EmmaZhu deleted the preview branch November 19, 2024 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants