Skip to content

Update repo Readme.mds to use 1.2.9 Version & Remove unsupported parameters - #113

Merged
bganapa merged 4 commits into
Azure:masterfrom
viananth:viananth-patch
Mar 20, 2017
Merged

Update repo Readme.mds to use 1.2.9 Version & Remove unsupported parameters#113
bganapa merged 4 commits into
Azure:masterfrom
viananth:viananth-patch

Conversation

@viananth

Copy link
Copy Markdown
Member
  1. Update Readme.mds to use AzureStack version 1.2.9
  2. Update Readme.mds to use API version profiles to install AzureRm modules
  3. Update AzureStack Admin cmdlets to remove unsupported parameters like 'SubscriptionId'

@msftclas

Copy link
Copy Markdown

@viananth,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@viananth

Copy link
Copy Markdown
Member Author

@bganapa, @knithinc to review

$defaultSubscription = Get-AzureRmSubscription -SubscriptionName "Default Provider Subscription" -ErrorAction Stop
$asCanaryQuotas = NewAzureStackDefaultQuotas -ResourceLocation $ResourceLocation -SubscriptionId $defaultSubscription.SubscriptionId -AADTenantID $AADTenantID -EnvironmentDomainFQDN $EnvironmentDomainFQDN -Credentials $ServiceAdminCredentials -ArmEndPoint $AdminArmEndPoint
New-AzureRMPlan -Name $tenantPlanName -DisplayName $tenantPlanName -ArmLocation $ResourceLocation -ResourceGroup $subscriptionRGName -SubscriptionId $defaultSubscription.SubscriptionId -AdminUri $AdminArmEndPoint -Token $asToken -QuotaIds $asCanaryQuotas -ErrorAction Stop
if((Get-Module AzureStack).Version -ge [System.Version] "1.2.9")

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

If Part is good enough

@bganapa
bganapa merged commit 7cce7cc into Azure:master Mar 20, 2017
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