Initial commit for the Websites commandlets for ARM

[NaveenGoli]

No description provided.

[azuresdkci]

Can one of the admins verify this patch?

[azurecla]

Hi @NaveenGoli, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (ngoli). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

[amitapl]

why did you remove this project?

[amitapl]

Is the file src/ResourceManager/Websites/Commands.Websites/MSSharedLibKey.snk required in the repo?

[galiniliev]

extra new lines

[ogail]

@azuresdkci add to whitelist

[NaveenGoli]

I have incorporated all the feedback changes. please review..

[amitapl]

you have some extra \n throughout

[NaveenGoli]

removed

[amitapl]

looks good

[NaveenGoli]

Will this be merged? AFAI can see, Microsoft.Azure.Commands.ScenarioTest.SqlTests.SecurityTests has no relation to websites code changes.. please let me know what the next steps are..

[matt-gibbs]

Not until the tests are passing. The reason New-AzureResourceGroup is not being found will need to be investigated.

[matt-gibbs]

re-test this please.

[NaveenGoli]

its passing on my local machine..

Assemblies Run

C:\GitHUb\azure-powershell\src\ResourceManager\Sql\Commands.Sql.Test\bin\Debug\Microsoft.Azure.Commands.Sql.Test.dll

Summary

Tests run: 3 Skipped: 2, Run time: 57.330s

Skipped tests

Skipped
❢
Microsoft.Azure.Commands.ScenarioTest.SqlTests.SecurityTests.TestDatabaseUpdatePolicyWithStorage

Skipped
❢
Microsoft.Azure.Commands.ScenarioTest.SqlTests.SecurityTests.TestFailedDatabaseUpdatePolicyWithNoStorage

All tests

Click test class name to expand/collapse test details

57.070s
✔
Microsoft.Azure.Commands.ScenarioTest.SqlTests.SecurityTests (3 tests)

[NaveenGoli]

when will this pull be taken in? I was under the impression it will happen today?

[yadavbdev]

There is one issue with this merge, the project does not build now as it is using the deprecated property of AzurePSCmdlet 'CurrentContext', where it should have been Profile.Context?

[NaveenGoli]

+Mark

Sent from my Windows Phone

---

From: Yadav Bhattarai (Microsoft)mailto:[email protected]
Sent: ‎2/‎25/‎2015 3:42 PM
To: Azure/azure-powershellmailto:[email protected]
Cc: Naveen Golimailto:[email protected]
Subject: Re: [azure-powershell] Initial commit for the Websites commandlets for ARM (#154)

There is one issue with this merge, the project does not build now as it is using the deprecated property of AzurePSCmdlet 'CurrentContext', where it should have been Profile.Context?

---

Reply to this email directly or view it on GitHub:
#154 (comment)