Merge pull request #5477 from xizhamsft/preview

add dns name label support for container instance

Author: cormacpayne

src/ResourceManager/ContainerInstance/ChangeLog.md

@@ -18,6 +18,8 @@
         - Additional information about change #1
 -->
 ## Current Release
+* Apply Azure Container Instance SDK 2018-02-01
+    - Support DNS name label
 
 ## Version 0.2.2
 * Added Location Completer to -Location parameters allowing tab completion through valid Locations

src/ResourceManager/ContainerInstance/Commands.ContainerInstance.Test/Commands.ContainerInstance.Test.csproj

@@ -47,7 +47,7 @@
       <HintPath>..\..\..\packages\Microsoft.Azure.Common.2.1.0\lib\net45\Microsoft.Azure.Common.NetFramework.dll</HintPath>
     </Reference>
     <Reference Include="Microsoft.Azure.Management.ContainerInstance, Version=0.0.0.0, Culture=neutral, PublicKeyToken=31bf3856ad364e35, processorArchitecture=MSIL">
-      <HintPath>..\..\..\packages\Microsoft.Azure.Management.ContainerInstance.1.0.1-preview\lib\net452\Microsoft.Azure.Management.ContainerInstance.dll</HintPath>
+      <HintPath>..\..\..\packages\Microsoft.Azure.Management.ContainerInstance.1.0.2-preview\lib\net452\Microsoft.Azure.Management.ContainerInstance.dll</HintPath>
       <Private>True</Private>
     </Reference>
     <Reference Include="Microsoft.Azure.Management.ResourceManager, Version=1.0.0.0, Culture=neutral, PublicKeyToken=31bf3856ad364e35, processorArchitecture=MSIL">
@@ -193,6 +193,9 @@
     <None Include="SessionRecords\Microsoft.Azure.Commands.ContainerInstance.Test.ScenarioTests.ContainerInstanceTests\TestCreateContainerGroup.json">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </None>
+    <None Include="SessionRecords\Microsoft.Azure.Commands.ContainerInstance.Test.ScenarioTests.ContainerInstanceTests\TestCreateContainerGroupWithDnsNameLabel.json" >
+      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+    </None>
     <None Include="SessionRecords\Microsoft.Azure.Commands.ContainerInstance.Test.ScenarioTests.ContainerInstanceTests\TestCreateContainerGroupWithVolume.json">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </None>

src/ResourceManager/ContainerInstance/Commands.ContainerInstance.Test/ScenarioTests/ContainerInstanceTests.cs

@@ -49,5 +49,12 @@ public void TestCreateContainerGroupWithVolume()
         {
             TestController.NewInstance.RunPowerShellTest("Test-AzureRmContainerGroupWithVolumeMount");
         }
+
+        [Fact]
+        [Trait(Category.AcceptanceType, Category.CheckIn)]
+        public void TestCreateContainerGroupWithDnsNameLabel()
+        {
+            TestController.NewInstance.RunPowerShellTest("Test-AzureRmContainerGroupWithDnsNameLabel");
+        }
     }
 }

src/ResourceManager/ContainerInstance/Commands.ContainerInstance.Test/ScenarioTests/ContainerInstanceTests.ps1

@@ -128,6 +128,57 @@ function Test-AzureRmContainerGroupWithVolumeMount
     }
 }
 
+<#
+.SYNOPSIS
+Test New-AzureRmContainerGroup with DNS name label.
+#>
+function Test-AzureRmContainerGroupWithDnsNameLabel
+{
+    $resourceGroupName = Get-RandomResourceGroupName
+    $containerGroupName = Get-RandomContainerGroupName
+	$fqdn = $containerGroupName + ".westus.azurecontainer.io"
+    $location = Get-ProviderLocation "Microsoft.ContainerInstance/ContainerGroups"
+    $image = "nginx"
+    $osType = "Linux"
+    $restartPolicy = "Never"
+    $port1 = 8000
+    $port2 = 8001
+
+    try
+    {
+        New-AzureRmResourceGroup -Name $resourceGroupName -Location $location
+        $containerGroupCreated = New-AzureRmContainerGroup -ResourceGroupName $resourceGroupName -Name $containerGroupName -Image $image -OsType $osType -RestartPolicy $restartPolicy -DnsNameLabel $containerGroupName -Port @($port1, $port2) -Cpu 1 -Memory 1.5
+
+        Assert-AreEqual $containerGroupCreated.ResourceGroupName $resourceGroupName
+        Assert-AreEqual $containerGroupCreated.Name $containerGroupName
+        Assert-AreEqual $containerGroupCreated.Location $location
+        Assert-AreEqual $containerGroupCreated.OsType $osType
+        Assert-AreEqual $containerGroupCreated.RestartPolicy $restartPolicy
+        Assert-NotNull $containerGroupCreated.IpAddress
+        Assert-AreEqual $containerGroupCreated.DnsNameLabel $containerGroupName
+        Assert-AreEqual $containerGroupCreated.Fqdn $fqdn
+        Assert-AreEqual $containerGroupCreated.Ports.Count 2
+        Assert-NotNull $containerGroupCreated.Containers
+        Assert-AreEqual $containerGroupCreated.Containers[0].Image $image
+        Assert-AreEqual $containerGroupCreated.Containers[0].Cpu 1
+        Assert-AreEqual $containerGroupCreated.Containers[0].MemoryInGb 1.5
+
+        $retrievedContainerGroup = Get-AzureRmContainerGroup -ResourceGroupName $resourceGroupName -Name $containerGroupName
+        Assert-ContainerGroup $containerGroupCreated $retrievedContainerGroup
+
+        $retrievedContainerGroupList = Get-AzureRmContainerGroup -ResourceGroupName $resourceGroupName
+        Assert-AreEqual $retrievedContainerGroupList.Count 1
+        Assert-ContainerGroup $containerGroupCreated $retrievedContainerGroupList[0]
+
+        $retrievedContainerGroup | Remove-AzureRmContainerGroup
+    }
+    finally
+    {
+        # Cleanup
+        Clean-ResourceGroup $resourceGroupName
+    }
+}
+
 <#
 .SYNOPSIS
 Assert a container group object.
@@ -155,6 +206,7 @@ function Assert-ContainerGroup
     Assert-AreEqual $Actual.RestartPolicy $Expected.RestartPolicy
     Assert-NotNull $Actual.IpAddress
     Assert-AreEqual $Actual.Ports.Count $Expected.Ports.Count
+	Assert-AreEqual $Actual.DnsNameLabel $Expected.DnsNameLabel
     Assert-NotNull $Actual.Containers
     Assert-AreEqual $Actual.Containers[0].Image $Expected.Containers[0].Image
     Assert-AreEqual $Actual.Containers[0].Cpu $Expected.Containers[0].Cpu

src/ResourceManager/ContainerInstance/Commands.ContainerInstance.Test/SessionRecords/Microsoft.Azure.Commands.ContainerInstance.Test.ScenarioTests.ContainerInstanceTests/TestContainerInstanceLog.json

@@ -65,7 +65,7 @@
   </ItemGroup>
   <ItemGroup>
     <Reference Include="Microsoft.Azure.Management.ContainerInstance, Version=0.0.0.0, Culture=neutral, PublicKeyToken=31bf3856ad364e35, processorArchitecture=MSIL">
-      <HintPath>..\..\..\packages\Microsoft.Azure.Management.ContainerInstance.1.0.1-preview\lib\net452\Microsoft.Azure.Management.ContainerInstance.dll</HintPath>
+      <HintPath>..\..\..\packages\Microsoft.Azure.Management.ContainerInstance.1.0.2-preview\lib\net452\Microsoft.Azure.Management.ContainerInstance.dll</HintPath>
     </Reference>
   </ItemGroup>
   <ItemGroup>

src/ResourceManager/ContainerInstance/Commands.ContainerInstance.Test/SessionRecords/Microsoft.Azure.Commands.ContainerInstance.Test.ScenarioTests.ContainerInstanceTests/TestCreateContainerGroup.json

@@ -143,10 +143,13 @@ public ContainerGroup CreateContainerGroup(ContainerGroupCreationParameters para
                 RestartPolicy = parameters.RestartPolicy
             };
 
-            if (string.Equals(IpAddress.Type, parameters.IpAddressType, StringComparison.OrdinalIgnoreCase))
+            if (string.Equals(IpAddress.Type, parameters.IpAddressType, StringComparison.OrdinalIgnoreCase) || 
+                !string.IsNullOrEmpty(parameters.DnsNameLabel))
             {
                 container.Ports = parameters.Ports.Select(p => new ContainerPort(p)).ToList();
-                containerGroup.IpAddress = new IpAddress(parameters.Ports.Select(p => new Port(p)).ToList());
+                containerGroup.IpAddress = new IpAddress(
+                    ports: parameters.Ports.Select(p => new Port(p)).ToList(),
+                    dnsNameLabel: parameters.DnsNameLabel);
             }
 
             if (!string.IsNullOrEmpty(parameters.RegistryServer))