Merge pull request #1 from Azure/master

Sync

Author: bashahee

.travis.yml

@@ -24,7 +24,7 @@ before_install:
 
  # https://github.com/travis-ci/travis-ci/issues/1066#issuecomment-383489298
 script:
- - sudo dotnet msbuild build.proj /t:Full /p:Configuration=$CONFIG || travis_terminate 1
+ - sudo dotnet msbuild build.proj /t:Full /p:ExcludeAuthenticators=true /p:Configuration=$CONFIG || travis_terminate 1
 
 after_success:
  - if [ "$TRAVIS_PULL_REQUEST" == "false" ]; then

ChangeLog.md

@@ -1,6 +1,142 @@
+## 1.2.0 - January 2019
+#### Az.Accounts
+* Add interactive and username/password authentication for Windows PowerShell 5.1 only
+* Update incorrect online help URLs
+* Add warning message in PS Core for Uninstall-AzureRm
+
+#### Az.Aks
+* Update incorrect online help URLs
+
+#### Az.Automation
+* Added support for Python 2 runbooks
+* Update incorrect online help URLs
+
+#### Az.Cdn
+* Update incorrect online help URLs
+
+#### Az.Compute
+* Add Invoke-AzVMReimage cmdlet
+* Add TempDisk parameter to Set-AzVmss
+* Fix the warning message of New-AzVM
+
+#### Az.ContainerRegistry
+* Update incorrect online help URLs
+
+#### Az.DataFactory
+* Updated ADF .Net SDK version to 3.0.0
+
+#### Az.DataLakeStore
+* Fix issue with ADLS endpoint when using MSI
+    - More information here: https://github.com/Azure/azure-powershell/issues/7462
+* Update incorrect online help URLs
+
+#### Az.IotHub
+* Add Encoding format to Add-IotHubRoutingEndpoint cmdlet.
+
+#### Az.KeyVault
+* Update incorrect online help URLs
+
+#### Az.Network
+* Update incorrect online help URLs
+
+#### Az.Resources
+* Fix incorrect examples in 'New-AzADAppCredential' and 'New-AzADSpCredential' reference documentation
+* Fix issue where path for '-TemplateFile' parameter was not being resolved before executing resource group deployment cmdlets
+* Az.Resources: Correct documentation for New-AzureRmPolicyDefinition -Mode default value
+* Az.Resources: Fix for issue https://github.com/Azure/azure-powershell/issues/7522
+* Az.Resources: Fix for issue https://github.com/Azure/azure-powershell/issues/5747
+* Fix formatting issue with 'PSResourceGroupDeployment' object
+    - More information here: https://github.com/Azure/azure-powershell/issues/2123
+
+#### Az.ServiceFabric
+* Rollback when a certificate is added to VMSS model but an exception is thrown this is to fix bug: https://github.com/Azure/service-fabric-issues/issues/932
+* Fix some error messages.
+* Fix create cluster with default ARM template for New-AzServiceFabriCluster which was not working with migration to Az.
+* Fix add cluster/application certificate to only add to VM Scale Sets that correspond to the cluster by checking cluster id in the extension.
+
+#### Az.SignalR
+* Update incorrect online help URLs
+
+#### Az.Sql
+* Update incorrect online help URLs
+* Updated parameter description for LicenseType parameter with possible values
+* Fix for updating managed instance identity not working when it is the only updated property
+* Support for custom collation on managed instance
+
+#### Az.Storage
+* Update incorrect online help URLs
+* Give detail error message when get/set classic Logging/Metric on Premium Storage Account, since Premium Storage Account not supoort classic Logging/Metric.
+    - Get/Set-AzStorageServiceLoggingProperty
+    - Get/Set-AzStorageServiceMetricsProperty
+
+#### Az.TrafficManager
+* Update incorrect online help URLs
+
+#### Az.Websites
+* Update incorrect online help URLs
+* Fixes 'New-AzWebAppSSLBinding' to upload the certificate to the correct resourcegroup+location if the app is hosted on an ASE.
+* Fixes 'New-AzWebAppSSLBinding' to not overwrite the tags on binding an SSL certificate to an app
+
+## 1.1.0 - January 2019
+#### Az.Accounts
+* Add 'Local' Scope to Enable-AzureRmAlias
+
+#### Az.Compute
+* Name is now optional in ID parameter set for Restart/Start/Stop/Remove/Set-AzVM and Save-AzVMImage
+* Updated the description of ID in help files
+* Fix backward compatibility issue with Az.Accounts module
+
+#### Az.DataLakeStore
+* Update the sdk version of dataplane to 1.1.14 for SDK fixes.
+    - Fix handling of negative acesstime and modificationtime for getfilestatus and liststatus, Fix async cancellation token
+
+#### Az.EventGrid
+* Updated to use the 2019-01-01 API version.
+* Update the following cmdlets to support new scenario in 2019-01-01 API version
+    - New-AzureRmEventGridSubscription: Add new optional parameters for specifying:
+        - Event Time-To-Live,
+        - Maximum number of delivery attempts for the events,
+        - Dead letter endpoint.
+    - Update-AzureRmEventGridSubscription: Add new optional parameters for specifying:
+        - Event Time-To-Live,
+        - Maximum number of delivery attempts for the events,
+        - Dead letter endpoint.
+* Add new enum values (namely, storageQueue and hybridConnection) for EndpointType option in New-AzureRmEventGridSubscription and Update-AzureRmEventGridSubscription cmdlets.
+* Show warning message if creating or updating the event subscription is expected to entail manual action from user.
+
+#### Az.IotHub
+* Updated to the latest version of the IotHub SDK
+
+#### Az.LogicApp
+* Get-AzLogicApp lists all without specified Name
+
+#### Az.Resources
+* Fix parameter set issue when providing '-ODataQuery' and '-ResourceId' parameters for 'Get-AzResource'
+    - More information here: https://github.com/Azure/azure-powershell/issues/7875
+* Fix handling of the -Custom parameter in New/Set-AzPolicyDefinition
+* Fix typo in New-AzDeployment documentation
+* Made '-MailNickname' parameter mandatory for 'New-AzADUser'
+    - More information here: https://github.com/Azure/azure-powershell/issues/8220
+
+#### Az.SignalR
+* Fix backward compatibility issue with Az.Accounts module
+
+#### Az.Sql
+* Converted the Storage management client dependency to the common SDK implementation.
+
+#### Az.Storage
+* Set the StorageAccountName of Storage context as the real Storage Account Name, when it's created with Sas Token, OAuth or Anonymous
+    - New-AzStorageContext
+* Create Sas Token of Blob Snapshot Object with '-FullUri' parameter, fix the returned Uri to be the sanpshot Uri
+    - New-AzStorageBlobSASToken
+
+#### Az.Websites
+* Fixed a date parsing bug in 'Get-AzDeletedWebApp'
+* Fix backward compatibility issue with Az.Accounts module
+
 ## Version 1.0.0 - December 2018
 
 #### General
 
 * General availability of `Az` module
-* For more information about the `Az` module, please visit the following: https://aka.ms/azps-announce
+* For more information about the `Az` module, please visit the following: https://aka.ms/azps-announce

CodeSign.targets

@@ -19,7 +19,10 @@
     <Message Text="----> Dlls signing section" Importance="high"/>
     <!-- Azure -->
     <ItemGroup Condition="'$(Scope)' == 'All' or '$(Scope)' == 'Latest' or '$(Scope)' == 'Netcore'">
-      <DelaySignedAssembliesToSign Include="$(PackageDirectory)\$(Configuration)\**\Microsoft*Azure*PowerShell*Cmdlets*.dll" />
+      <DelaySignedAssembliesToSign Include="$(PackageDirectory)\$(Configuration)\**\Microsoft*Azure*PowerShell*Cmdlets*.dll;
+        $(PackageDirectory)\$(Configuration)\Az.Accounts\Microsoft.Azure.PowerShell.Authentication.ResourceManager.dll;
+        $(PackageDirectory)\$(Configuration)\**\Microsoft.Azure.PowerShell.Authenticators.dll;
+        $(PackageDirectory)\$(Configuration)\Az.Accounts\Microsoft.Azure.PowerShell.Authentication.dll" />
     </ItemGroup>
 
     <Message Importance="high" Text="$(PackageDirectory)\$(Configuration) does not contains any files to sign. Code sign will skip." Condition="'@(DelaySignedAssembliesToSign)' == ''" />

README.md

@@ -89,7 +89,7 @@ To change the subscription that you are using for your current context, use the
 Set-AzContext -Subscription $SubscriptionName -Name 'MyContext'
 
 # Set the Azure PowerShell context using piping
-Get-AzSubscription -SubscriptionName $SubscriptionName | Set-AzureRmContext -Name 'MyContext'
+Get-AzSubscription -SubscriptionName $SubscriptionName | Set-AzContext -Name 'MyContext'
 ```
 
 For details on Azure PowerShell contexts, see our [persisted credentials guide][PersistedCredentialsGuide].

build.proj

@@ -218,8 +218,11 @@
     <!-- Build and create package content -->
     <Exec Command="dotnet --version" />
     <Exec Command="dotnet new sln -n Azure.PowerShell --force" />
+    <PropertyGroup Condition="'$(ExcludeAuthenticators)' == 'true'">
+      <AuthenticatorExclusions>$(LibraryRoot)src/**/Authenticators.csproj;</AuthenticatorExclusions>
+    </PropertyGroup>
     <ItemGroup>
-      <ModuleCsprojFiles Include="$(LibraryRoot)src/**/*.csproj" Exclude="$(LibraryRoot)src/**/*.Test.csproj" />
+      <ModuleCsprojFiles Include="$(LibraryRoot)src/**/*.csproj" Exclude="$(AuthenticatorExclusions)$(LibraryRoot)src/**/*.Test.csproj" />
     </ItemGroup>
     <!-- https://stackoverflow.com/a/35477012/294804 -->
     <ItemGroup>
@@ -228,7 +231,7 @@
       </ModuleCsprojFilesRelative>
     </ItemGroup>
     <Exec Command="dotnet sln Azure.PowerShell.sln add @(ModuleCsprojFilesRelative->'%(RelativePath)', ' ')" />
-    <Exec Command="dotnet sln Azure.PowerShell.sln add $(LibraryRoot)tools/Tools.Common/Tools.Common.Netcore.csproj $(LibraryRoot)tools/StaticAnalysis/StaticAnalysis.Netcore.csproj" />
+    <Exec Command="dotnet sln Azure.PowerShell.sln add $(LibraryRoot)tools/Tools.Common/Tools.Common.Netcore.csproj $(LibraryRoot)tools/StaticAnalysis/StaticAnalysis.Netcore.csproj $(LibraryRoot)tools/VersionController/VersionController.Netcore.csproj" />
     <Exec Command="dotnet publish Azure.PowerShell.sln -c $(Configuration)" />
 
 
@@ -372,14 +375,17 @@
 
   <Target Name="Test">
     <Message Importance="high" Text="Running check in tests..." />
+    <PropertyGroup Condition="'$(ExcludeAuthenticators)' == 'true'">
+      <AuthenticatorExclusions>$(LibraryRoot)src/**/Authenticators.csproj;</AuthenticatorExclusions>
+    </PropertyGroup>
 
     <CallTarget Targets="BinScopeCheck" Condition="'$(OnPremiseBuild)'" />
     <CallTarget Targets="RunPoliCheck" Condition="'$(OnPremiseBuild)'" />
 
     <MakeDir Directories="$(TestOutputDirectory)" ContinueOnError="false" />
     <Exec Command="dotnet new sln -n Azure.PowerShell --force" />
     <ItemGroup>
-      <ModuleCsprojFiles Include="$(LibraryRoot)src/**/*.csproj" />
+      <ModuleCsprojFiles Include="$(LibraryRoot)src/**/*.csproj" Exclude="$(AuthenticatorExclusions)"/>
     </ItemGroup>
     <!-- https://stackoverflow.com/a/35477012/294804 -->
     <ItemGroup>
@@ -388,6 +394,7 @@
       </ModuleCsprojFilesRelative>
     </ItemGroup>
     <Exec Command="dotnet sln Azure.PowerShell.sln add @(ModuleCsprojFilesRelative->'%(RelativePath)', ' ')" />
+    <Exec Command="dotnet sln Azure.PowerShell.sln add $(LibraryRoot)tools/ScenarioTest.ResourceManager/ScenarioTest.ResourceManager.csproj $(LibraryRoot)tools/TestFx/TestFx.csproj" />
     <!-- https://github.com/Microsoft/msbuild/issues/3157#issuecomment-378002971 -->
     <!-- https://github.com/xunit/xunit/issues/1154#issuecomment-405366373 -->
     <Exec Command="dotnet test Azure.PowerShell.sln --filter &quot;AcceptanceType=CheckIn%26RunType!=DesktopOnly&quot; --configuration $(Configuration) --framework netcoreapp2.0 --logger trx" />

documentation/development-docs/azure-powershell-developer-guide.md

@@ -174,13 +174,13 @@ After the solution file is updated, save and close it. Now, open the solution fi
     <PsModuleName>Cdn</PsModuleName>
   </PropertyGroup>
 ```
-- Remove the entry:
+- **Remove the entry**:
 ```xml
   <PropertyGroup>
     <RootNamespace>$(LegacyAssemblyPrefix)$(PsModuleName)</RootNamespace>
   </PropertyGroup>
 ```
-This is not needed since this is a new project.
+**Note**: This is not needed since this is a new project and does not use legacy namespace conventions.
 
 - Update this entry to use your SDK:
 ```xml
@@ -244,8 +244,11 @@ Please see our guide on [Using Azure TestFramework](../testing-docs/using-azure-
 
 ### Adding Scenario Tests
 
-- Create a new class in `Commands.<SERVICE>.Test`
-- Create a ps1 file in the same folder that contains the actual tests ([see sample](../../src/Media/Commands.Media.Test/ScenarioTests))
+- Create a new class in `<SERVICE>.Test`
+    - Add `[Fact]` as an attribute to every test
+    - Add `[Trait(Category.AcceptanceType, Category.CheckIn)]` as an attribute to any test that should be run during CI in Playback mode.
+    - Add `[Trait(Category.AcceptanceType, Category.LiveOnly)]` as an attribute to any test that cannot be run in Playback mode (for example, if a test depends on a Dataplane SDK).
+- Create a ps1 file in the same folder that contains the actual tests ([see sample](../../src/Media/Media.Test/ScenarioTests))
     - Use `Assert-AreEqual x y` to verify that values are the same
     - Use `Assert-AreNotEqual x y` to verify that values are not the same
     - Use `Assert-Throws scriptblock message` to verify an exception is being thrown
@@ -256,7 +259,7 @@ Please see our guide on [Using Azure TestFramework](../testing-docs/using-azure-
     - Use `Assert-Null object` to verify that an object is null
     - Use `Assert-NotNull object` to verify that an object is not null
     - Use `Assert-Exists path` to verify that a file exists
-    - Use `Assert-AreEqualArray a1 a2` to verify that arrays are the sam
+    - Use `Assert-AreEqualArray a1 a2` to verify that arrays are the same
 
 ### Using Active Directory
 
@@ -270,14 +273,8 @@ Please see our guide on [Using Azure TestFramework](../testing-docs/using-azure-
 
 ### AD Scenario Tests
 
-Create these environment variables for the AD scenario tests:
+Create this environment variables for the AD scenario tests:
 
-- `AZURE_LIVEID` should be UserId and Password for a valid LiveId account.
-  - `AZURE_LIVEID=UserId=<[email protected]>;Password=<Password>`
-- `AZURE_ORGID_FPO` should be an orgid and password for an account that does not have any subscriptions or role assignments associated with it. It is supposed to be a foreign principal in your current tenant.
-  - `AZURE_ORGID_FPO=UserId=<[email protected]>;Password=<Password>`
--  `AZURE_ORGID_ONE_TENANT_ONE_SUBSCRIPTION` should be an account that is in a single tenant and has access to the subscription managed by that AD tenant.
-  - `AZURE_ORGID_ONE_TENANT_ONE_SUBSCRIPTION=UserId=<[email protected]>;Password=<Password>;SubscriptionId=<SubscriptionId>;AADAuthEndpoint=https://login.windows.net/`
 - `AZURE_SERVICE_PRINCIPAL` should be a service principal - an application defined in the subscription's tenant - that has management access to the subscription (or at least to a resource group in the tenant)
   - `AZURE_SERVICE_PRINCIPAL=UserId=<UserGuid>;Password=<Password>;AADTenant=<TenantGuid>;SubscriptionId=<SubscriptionId>`
 
@@ -287,7 +284,7 @@ Create these environment variables for the AD scenario tests:
 - Run the test in Visual Studio in the Test Explorer window and make sure you got a generated JSON file that matches the test name in the bin folder under the `SessionRecords` folder
 - Copy this `SessionRecords` folder and place it inside the test project
   - Inside Visual Studio, add all of the generated JSON files, making sure to change the "Copy to Output Directory" property for each one to "Copy if newer"
-  -  Make sure that all of these JSON files appear in your `Commands.<SERVICE>.Test.csproj` file
+  -  Make sure that all of these JSON files appear in your `<SERVICE>.Test.csproj` file
 
 # After Development
 

documentation/testing-docs/using-azure-test-framework.md

@@ -6,10 +6,8 @@
 	- [New-TestCredential](#new-testcredential)
 		- [Create New Service Principal](#create-new-service-principal)
 		- [Use Existing Service Principal](#use-existing-service-principal)
-		- [UserId](#userid)
 	- [Set-TestEnvironment](#set-testenvironment)
 		- [Existing Service Principal](#existing-service-principal)
-		- [UserId](#userid)
 	- [Manually Set Environment Variables](#manually-set-environment-variables)
 		- [Environment Variables](#environment-variables)
 		- [Playback Test](#playback-test)
@@ -51,11 +49,11 @@ This cmdlet, located in the [`Repo-Tasks` module](/tools/Repo-Tasks.psd1), which
 Using a service principal is the preferred option for recording tests because it works with both .NET Framework and .NET Core.  In order to create a new service principal, run this command with an unused service principal display name:
 
 ```powershell
-New-TestCredential -ServicePrincipalDisplayName "ScenarioTestCredentials" -ServicePrincipalSecret `
-"testpassword" -SubscriptionId <subscriptionId> -TenantId <tenantId> -RecordMode "Record"
+New-TestCredential -ServicePrincipalDisplayName "ScenarioTestCredentials" -SubscriptionId `
+<subscriptionId> -TenantId <tenantId> -RecordMode "Record"
 ```
 
-This command will create a new service principal, set the correct role assignment for this service principal based upon the subscription provided, and place the service principal id and secret into the credentials file.
+This command will create a new service principal, set the correct role assignment for this service principal based upon the subscription provided, and place the service principal id and automatically generated secret into the credentials file.
 
 Alternatively, to create a service principal, follow the [Azure AD guide to create a Application Service Principal](https://docs.microsoft.com/en-us/azure/azure-resource-manager/resource-group-create-service-principal-portal#create-an-active-directory-application). The application type should be `Web app / API` and the sign-on URL value is irrelevant (you can set any value).
 
@@ -68,15 +66,6 @@ New-TestCredential -ServicePrincipalDisplayName "Existing Service Principal" -Se
 "testpassword" -SubscriptionId <subscriptionId> -TenantId <tenantId> -RecordMode "Record"
 ```
 
-#### UserId
-
-This is no longer the preferred option because it only works when running on .NET Framework. When running on .NET Core you may get an error like `Interactive Login is supported only in NET45 projects`.  Additionally, you will have to manually log in when running the scenario tests rather than being automatically validated.
-
-```powershell
-New-TestCredential -UserId "[email protected]" -SubscriptionId <subscriptionId> `
--TenantId <tenantId> -RecordMode "Record"
-```
-
 ### Set-TestEnvironment
 
 This cmdlet, located in the [`Repo-Tasks` module](/tools/Repo-Tasks.psd1), will directly set the environment variable for the session.
@@ -90,15 +79,6 @@ Set-TestEnvironment -ServicePrincipalId <servicePrincipalId> -ServicePrincipalSe
 "testpassword" -SubscriptionId <subscriptionId> -TenantId <tenantId> -RecordMode "Record"
 ```
 
-#### UserId
-
-This is no longer the preferred option because it only works when running on .NET Framework. When running on .NET Core you may get an error like `Interactive Login is supported only in NET45 projects`.
-
-```powershell
-Set-TestEnvironment -UserId "[email protected]" -SubscriptionId <subscriptionId> `
--TenantId <tenantId> -RecordMode "Record"
-```
-
 ### Manually Set Environment Variables
 
 #### Environment Variables
@@ -118,21 +98,8 @@ TEST_CSM_ORGID_AUTHENTICATION=
 AZURE_TEST_MODE=Playback
 ```
 
-#### Record Test with Interactive login using OrgId
-
-This is no longer the preferred option because it only works when running on .NET Framework. When running on .NET Core you may get an error like `Interactive Login is supported only in NET45 projects`.
-
-To use this option, set the following environment variables before starting Visual Studio:
-
-```
-TEST_CSM_ORGID_AUTHENTICATION=SubscriptionId={SubId};UserId={orgId};AADTenant={tenantId};Environment={env};HttpRecorderMode=Record;
-AZURE_TEST_MODE=Record
-```
-
 #### Record Test with service principal
 
-This is the preferred option for recording tests because it works with both .NET Framework and .NET Core.
-
 After the service principal is created, you will need to give it access to Azure resources. This can be done with the following PowerShell command, with the [Service Principal Application ID](https://docs.microsoft.com/en-us/azure/azure-resource-manager/resource-group-create-service-principal-portal#get-application-id-and-authentication-key) (this is a guid, not the display name of the service principal) substituted in for `{clientId}`.
 
 ```powershell

setup/azurecmd.wxs

@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="UTF-8"?>
+<?xml version="1.0" encoding="UTF-8"?>
 <Wix xmlns="http://schemas.microsoft.com/wix/2006/wi" xmlns:util="http://schemas.microsoft.com/wix/UtilExtension" >
   <Product Id="*"
            Name="$(var.productName)"