Repository: jclouds Updated Branches: refs/heads/master ff93565b1 -> e4f1823fd
Fixes/adds fields primary for NetworkInterfaceCardProperties and misplaced in IpConfiguration Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/e4f1823f Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/e4f1823f Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/e4f1823f Branch: refs/heads/master Commit: e4f1823fd533812f40313543f34a14e841972391 Parents: ff93565 Author: Dani Estevez <[email protected]> Authored: Thu Jun 14 15:06:46 2018 -0400 Committer: Ignasi Barrera <[email protected]> Committed: Fri Jun 15 08:56:27 2018 +0200 ---------------------------------------------------------------------- .../arm/domain/IpConfiguration.java | 13 +++--- .../arm/domain/IpConfigurationProperties.java | 22 +++++---- .../domain/NetworkInterfaceCardProperties.java | 19 +++++--- .../NetworkInterfaceCardApiLiveTest.java | 12 ++--- .../NetworkInterfaceCardApiMockTest.java | 39 ++++++++++------ .../arm/features/VirtualMachineApiLiveTest.java | 18 ++++---- .../VirtualMachineScaleSetApiLiveTest.java | 32 ++++++------- .../VirtualMachineScaleSetApiMockTest.java | 31 ++++++------- .../resources/createnetworkinterfacecard.json | 47 +++++++++++++------- .../test/resources/getnetworkinterfacecard.json | 3 +- 10 files changed, 137 insertions(+), 99 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/jclouds/blob/e4f1823f/providers/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/IpConfiguration.java ---------------------------------------------------------------------- diff --git a/providers/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/IpConfiguration.java b/providers/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/IpConfiguration.java index e523622..3d84f8e 100644 --- a/providers/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/IpConfiguration.java +++ b/providers/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/IpConfiguration.java @@ -17,10 +17,11 @@ package org.jclouds.azurecompute.arm.domain; -import com.google.auto.value.AutoValue; import org.jclouds.javax.annotation.Nullable; import org.jclouds.json.SerializedNames; +import com.google.auto.value.AutoValue; + @AutoValue public abstract class IpConfiguration { @@ -34,18 +35,15 @@ public abstract class IpConfiguration { public abstract String etag(); @Nullable - public abstract Boolean primary(); - - @Nullable public abstract IpConfigurationProperties properties(); - @SerializedNames({"name", "id", "etag", "primary", "properties"}) - public static IpConfiguration create(final String name, final String id, final String etag, final Boolean primary, final IpConfigurationProperties properties) { + @SerializedNames({ "name", "id", "etag", "properties" }) + public static IpConfiguration create(final String name, final String id, final String etag, + final IpConfigurationProperties properties) { return builder() .name(name) .id(id) .etag(etag) - .primary(primary) .properties(properties) .build(); } @@ -61,7 +59,6 @@ public abstract class IpConfiguration { public abstract Builder name(String name); public abstract Builder id(String id); public abstract Builder etag(String etag); - public abstract Builder primary(Boolean primary); public abstract Builder properties(IpConfigurationProperties properties); public abstract IpConfiguration build(); } http://git-wip-us.apache.org/repos/asf/jclouds/blob/e4f1823f/providers/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/IpConfigurationProperties.java ---------------------------------------------------------------------- diff --git a/providers/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/IpConfigurationProperties.java b/providers/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/IpConfigurationProperties.java index 21f494f..c17e1ec 100644 --- a/providers/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/IpConfigurationProperties.java +++ b/providers/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/IpConfigurationProperties.java @@ -18,12 +18,12 @@ package org.jclouds.azurecompute.arm.domain; import java.util.List; -import com.google.auto.value.AutoValue; -import com.google.common.collect.ImmutableList; - import org.jclouds.javax.annotation.Nullable; import org.jclouds.json.SerializedNames; +import com.google.auto.value.AutoValue; +import com.google.common.collect.ImmutableList; + @AutoValue public abstract class IpConfigurationProperties implements Provisionable { @@ -48,11 +48,15 @@ public abstract class IpConfigurationProperties implements Provisionable { @Nullable public abstract List<IdReference> loadBalancerInboundNatRules(); - @SerializedNames({ "provisioningState", "privateIPAddress", "privateIPAllocationMethod", "subnet", - "publicIPAddress", "loadBalancerBackendAddressPools", "loadBalancerInboundNatRules" }) + @Nullable + public abstract Boolean primary(); + + @SerializedNames({ "provisioningState", "privateIPAddress", "privateIPAllocationMethod", "subnet", "publicIPAddress", + "loadBalancerBackendAddressPools", "loadBalancerInboundNatRules", "primary" }) public static IpConfigurationProperties create(final String provisioningState, final String privateIPAddress, final String privateIPAllocationMethod, final IdReference subnet, final IdReference publicIPAddress, - List<IdReference> loadBalancerBackendAddressPools, List<IdReference> loadBalancerInboundNatRules) { + List<IdReference> loadBalancerBackendAddressPools, List<IdReference> loadBalancerInboundNatRules, + final Boolean primary) { return builder() .provisioningState(provisioningState) @@ -61,7 +65,7 @@ public abstract class IpConfigurationProperties implements Provisionable { .subnet(subnet) .publicIPAddress(publicIPAddress) .loadBalancerBackendAddressPools(loadBalancerBackendAddressPools) - .loadBalancerInboundNatRules(loadBalancerInboundNatRules) + .loadBalancerInboundNatRules(loadBalancerInboundNatRules).primary(primary) .build(); } @@ -82,10 +86,12 @@ public abstract class IpConfigurationProperties implements Provisionable { public abstract Builder subnet(IdReference subnet); public abstract Builder publicIPAddress(IdReference publicIPAddress); - + public abstract Builder loadBalancerBackendAddressPools(List<IdReference> loadBalancerBackendAddressPools); public abstract Builder loadBalancerInboundNatRules(List<IdReference> loadBalancerInboundNatRules); + + public abstract Builder primary(Boolean primary); abstract List<IdReference> loadBalancerBackendAddressPools(); http://git-wip-us.apache.org/repos/asf/jclouds/blob/e4f1823f/providers/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/NetworkInterfaceCardProperties.java ---------------------------------------------------------------------- diff --git a/providers/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/NetworkInterfaceCardProperties.java b/providers/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/NetworkInterfaceCardProperties.java index 680d0a2..e6a9755 100644 --- a/providers/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/NetworkInterfaceCardProperties.java +++ b/providers/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/domain/NetworkInterfaceCardProperties.java @@ -16,12 +16,13 @@ */ package org.jclouds.azurecompute.arm.domain; -import com.google.auto.value.AutoValue; -import com.google.common.collect.ImmutableList; +import java.util.List; + import org.jclouds.javax.annotation.Nullable; import org.jclouds.json.SerializedNames; -import java.util.List; +import com.google.auto.value.AutoValue; +import com.google.common.collect.ImmutableList; @AutoValue public abstract class NetworkInterfaceCardProperties implements Provisionable { @@ -32,17 +33,20 @@ public abstract class NetworkInterfaceCardProperties implements Provisionable { @Nullable public abstract List<IpConfiguration> ipConfigurations(); @Nullable public abstract IdReference networkSecurityGroup(); + @Nullable + public abstract Boolean primary(); + @SerializedNames({"provisioningState", "resourceGuid", "enableIPForwarding", "ipConfigurations", - "networkSecurityGroup"}) + "networkSecurityGroup", "primary" }) public static NetworkInterfaceCardProperties create(final String provisioningState, final String resourceGuid, final Boolean enableIPForwarding, final List<IpConfiguration> ipConfigurations, - final IdReference networkSecurityGroup) { + final IdReference networkSecurityGroup, final Boolean primary) { NetworkInterfaceCardProperties.Builder builder = NetworkInterfaceCardProperties.builder() .provisioningState(provisioningState) .resourceGuid(resourceGuid) .enableIPForwarding(enableIPForwarding) .ipConfigurations(ipConfigurations == null ? null : ImmutableList.copyOf(ipConfigurations)) - .networkSecurityGroup(networkSecurityGroup); + .networkSecurityGroup(networkSecurityGroup).primary(primary); return builder.build(); } @@ -65,6 +69,9 @@ public abstract class NetworkInterfaceCardProperties implements Provisionable { public abstract Builder ipConfigurations(List<IpConfiguration> ipConfigurations); public abstract Builder networkSecurityGroup(IdReference networkSecurityGroup); + public abstract Builder primary(Boolean primary); + + abstract List<IpConfiguration> ipConfigurations(); abstract NetworkInterfaceCardProperties autoBuild(); http://git-wip-us.apache.org/repos/asf/jclouds/blob/e4f1823f/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/NetworkInterfaceCardApiLiveTest.java ---------------------------------------------------------------------- diff --git a/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/NetworkInterfaceCardApiLiveTest.java b/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/NetworkInterfaceCardApiLiveTest.java index cd95a2c..d7caab0 100644 --- a/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/NetworkInterfaceCardApiLiveTest.java +++ b/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/NetworkInterfaceCardApiLiveTest.java @@ -16,6 +16,7 @@ */ package org.jclouds.azurecompute.arm.features; +import static java.lang.Boolean.TRUE; import static org.testng.Assert.assertEquals; import static org.testng.Assert.assertNotNull; import static org.testng.Assert.assertTrue; @@ -66,12 +67,10 @@ public class NetworkInterfaceCardApiLiveTest extends BaseAzureComputeApiLiveTest @Test public void createNetworkInterfaceCard() { //Create properties object - final NetworkInterfaceCardProperties networkInterfaceCardProperties = - NetworkInterfaceCardProperties.builder().ipConfigurations( - Arrays.asList(IpConfiguration.builder() - .name("myipconfig") - .properties(IpConfigurationProperties.builder() - .privateIPAllocationMethod("Dynamic") + final NetworkInterfaceCardProperties networkInterfaceCardProperties = NetworkInterfaceCardProperties.builder() + .ipConfigurations(Arrays.asList(IpConfiguration.builder().name("myipconfig") + .properties(IpConfigurationProperties.builder().privateIPAllocationMethod("Dynamic").primary + (TRUE) .subnet(IdReference.create(subnetId)).build() ).build() )).build(); @@ -84,6 +83,7 @@ public class NetworkInterfaceCardApiLiveTest extends BaseAzureComputeApiLiveTest assertTrue(nic.properties().ipConfigurations().size() > 0); assertEquals(nic.properties().ipConfigurations().get(0).name(), "myipconfig"); assertEquals(nic.properties().ipConfigurations().get(0).properties().privateIPAllocationMethod(), "Dynamic"); + assertTrue(nic.properties().ipConfigurations().get(0).properties().primary()); assertEquals(nic.properties().ipConfigurations().get(0).properties().subnet().id(), subnetId); assertEquals(nic.tags().get("jclouds"), "livetest"); } http://git-wip-us.apache.org/repos/asf/jclouds/blob/e4f1823f/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/NetworkInterfaceCardApiMockTest.java ---------------------------------------------------------------------- diff --git a/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/NetworkInterfaceCardApiMockTest.java b/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/NetworkInterfaceCardApiMockTest.java index dd10046..5aae6bd 100644 --- a/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/NetworkInterfaceCardApiMockTest.java +++ b/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/NetworkInterfaceCardApiMockTest.java @@ -16,8 +16,15 @@ */ package org.jclouds.azurecompute.arm.features; -import com.google.common.collect.ImmutableMap; -import com.squareup.okhttp.mockwebserver.MockResponse; +import static org.testng.Assert.assertEquals; +import static org.testng.Assert.assertNotNull; +import static org.testng.Assert.assertNull; +import static org.testng.Assert.assertTrue; + +import java.util.Arrays; +import java.util.List; +import java.util.Map; + import org.jclouds.azurecompute.arm.domain.IdReference; import org.jclouds.azurecompute.arm.domain.IpConfiguration; import org.jclouds.azurecompute.arm.domain.IpConfigurationProperties; @@ -26,14 +33,8 @@ import org.jclouds.azurecompute.arm.domain.NetworkInterfaceCardProperties; import org.jclouds.azurecompute.arm.internal.BaseAzureComputeApiMockTest; import org.testng.annotations.Test; -import java.util.Arrays; -import java.util.List; -import java.util.Map; - -import static org.testng.Assert.assertNotNull; -import static org.testng.Assert.assertNull; -import static org.testng.Assert.assertTrue; -import static org.testng.Assert.assertEquals; +import com.google.common.collect.ImmutableMap; +import com.squareup.okhttp.mockwebserver.MockResponse; @Test(groups = "unit", testName = "NetworkInterfaceCardApiMockTest", singleThreaded = true) public class NetworkInterfaceCardApiMockTest extends BaseAzureComputeApiMockTest { @@ -54,7 +55,9 @@ public class NetworkInterfaceCardApiMockTest extends BaseAzureComputeApiMockTest assertSent(server, "GET", path); assertNotNull(nic); assertEquals(nic.name(), nicName); + assertTrue(nic.properties().primary()); assertEquals(nic.properties().ipConfigurations().get(0).name(), "myip1"); + assertTrue(nic.properties().ipConfigurations().get(0).properties().primary()); assertEquals(nic.tags().get("mycustomtag"), "foobar"); } @@ -104,16 +107,24 @@ public class NetworkInterfaceCardApiMockTest extends BaseAzureComputeApiMockTest final String SubnetID = "/subscriptions/" + subscriptionid + "/resourceGroups/azurearmtesting/providers/Microsoft.Network/virtualNetworks/myvirtualnetwork/subnets/mysubnet"; //Create properties object - final NetworkInterfaceCardProperties networkInterfaceCardProperties = NetworkInterfaceCardProperties.create(null, - null, null, Arrays.asList(IpConfiguration.create("myipconfig", null, null, null, IpConfigurationProperties - .create(null, null, "Dynamic", IdReference.create(SubnetID), null, null, null))), null); + final NetworkInterfaceCardProperties networkInterfaceCardProperties = NetworkInterfaceCardProperties + .create(null, null, null, Arrays.asList(IpConfiguration.create("myipconfig", null, null, + IpConfigurationProperties + .create(null, null, "Dynamic", IdReference.create(SubnetID), null, null, null, Boolean.TRUE))), + null, Boolean.TRUE); final Map<String, String> tags = ImmutableMap.of("mycustomtag", "foobar"); NetworkInterfaceCard nic = nicApi.createOrUpdate(nicName, location, networkInterfaceCardProperties, tags); + String path = String.format("/subscriptions/%s/resourcegroups/%s/providers/Microsoft.Network/networkInterfaces/%s?%s", subscriptionid, resourcegroup, nicName, apiVersion); - String json = String.format("{ \"location\":\"%s\", \"tags\": { \"mycustomtag\": \"foobar\" }, \"properties\":{ \"ipConfigurations\":[ { \"name\":\"%s\", \"properties\":{ \"subnet\":{ \"id\": \"%s\" }, \"privateIPAllocationMethod\":\"%s\" } } ] } }", location, "myipconfig", SubnetID, "Dynamic"); + String json = String.format( + "{\"location\":\"northeurope\",\"properties\":{\"ipConfigurations\":[{\"name\":\"myipconfig\"," + + "\"properties\":{\"privateIPAllocationMethod\":\"Dynamic\"," + + "\"subnet\":{\"id\":\"/subscriptions/SUBSCRIPTIONID/resourceGroups/azurearmtesting/providers" + + "/Microsoft" + ".Network/virtualNetworks/myvirtualnetwork/subnets/mysubnet\"},\"primary\":true}}]," + + "\"primary\":true}," + "\"tags\":{\"mycustomtag\":\"foobar\"}}"); assertSent(server, "PUT", path, json); assertEquals(nic.tags().get("mycustomtag"), "foobar"); } http://git-wip-us.apache.org/repos/asf/jclouds/blob/e4f1823f/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/VirtualMachineApiLiveTest.java ---------------------------------------------------------------------- diff --git a/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/VirtualMachineApiLiveTest.java b/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/VirtualMachineApiLiveTest.java index 34b289e..0154a27 100644 --- a/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/VirtualMachineApiLiveTest.java +++ b/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/VirtualMachineApiLiveTest.java @@ -21,6 +21,7 @@ import static org.jclouds.util.Predicates2.retry; import static org.testng.Assert.assertEquals; import static org.testng.Assert.assertNotNull; import static org.testng.Assert.assertTrue; +import static org.testng.util.Strings.isNullOrEmpty; import java.net.URI; import java.util.ArrayList; @@ -39,8 +40,12 @@ import org.jclouds.azurecompute.arm.domain.ManagedDiskParameters; import org.jclouds.azurecompute.arm.domain.NetworkInterfaceCard; import org.jclouds.azurecompute.arm.domain.NetworkInterfaceCardProperties; import org.jclouds.azurecompute.arm.domain.NetworkProfile; +import org.jclouds.azurecompute.arm.domain.NetworkProfile.NetworkInterface; +import org.jclouds.azurecompute.arm.domain.NetworkProfile.NetworkInterface.NetworkInterfaceProperties; import org.jclouds.azurecompute.arm.domain.OSDisk; import org.jclouds.azurecompute.arm.domain.OSProfile; +import org.jclouds.azurecompute.arm.domain.OSProfile.WindowsConfiguration.WinRM.Protocol; +import org.jclouds.azurecompute.arm.domain.OSProfile.WindowsConfiguration.WinRM.ProtocolListener; import org.jclouds.azurecompute.arm.domain.ResourceDefinition; import org.jclouds.azurecompute.arm.domain.Secrets; import org.jclouds.azurecompute.arm.domain.StorageAccountType; @@ -53,15 +58,10 @@ import org.jclouds.azurecompute.arm.domain.VirtualMachine; import org.jclouds.azurecompute.arm.domain.VirtualMachineInstance; import org.jclouds.azurecompute.arm.domain.VirtualMachineInstance.PowerState; import org.jclouds.azurecompute.arm.domain.VirtualMachineProperties; -import org.jclouds.azurecompute.arm.domain.NetworkProfile.NetworkInterface; -import org.jclouds.azurecompute.arm.domain.NetworkProfile.NetworkInterface.NetworkInterfaceProperties; -import org.jclouds.azurecompute.arm.domain.OSProfile.WindowsConfiguration.WinRM.Protocol; -import org.jclouds.azurecompute.arm.domain.OSProfile.WindowsConfiguration.WinRM.ProtocolListener; import org.jclouds.azurecompute.arm.functions.ParseJobStatus; import org.jclouds.azurecompute.arm.internal.BaseAzureComputeApiLiveTest; import org.testng.annotations.BeforeClass; import org.testng.annotations.Test; -import static org.testng.util.Strings.isNullOrEmpty; import com.beust.jcommander.internal.Lists; import com.google.common.base.Predicate; @@ -291,10 +291,10 @@ public class VirtualMachineApiLiveTest extends BaseAzureComputeApiLiveTest { private NetworkInterfaceCard createNetworkInterfaceCard(final String resourceGroupName, String networkInterfaceCardName, String locationName, String ipConfigurationName) { //Create properties object final NetworkInterfaceCardProperties networkInterfaceCardProperties = NetworkInterfaceCardProperties - .builder() - .ipConfigurations( - Arrays.asList(IpConfiguration.create(ipConfigurationName, null, null, null, IpConfigurationProperties - .create(null, null, "Dynamic", IdReference.create(subnetId), null, null, null)))).build(); + .builder().ipConfigurations(Arrays.asList(IpConfiguration.create(ipConfigurationName, null, null, + IpConfigurationProperties + .create(null, null, "Dynamic", IdReference.create(subnetId), null, null, null, Boolean.TRUE)))) + .build(); final Map<String, String> tags = ImmutableMap.of("jclouds", "livetest"); return api.getNetworkInterfaceCardApi(resourceGroupName).createOrUpdate(networkInterfaceCardName, locationName, networkInterfaceCardProperties, tags); http://git-wip-us.apache.org/repos/asf/jclouds/blob/e4f1823f/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/VirtualMachineScaleSetApiLiveTest.java ---------------------------------------------------------------------- diff --git a/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/VirtualMachineScaleSetApiLiveTest.java b/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/VirtualMachineScaleSetApiLiveTest.java index ff93998..4d17967 100644 --- a/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/VirtualMachineScaleSetApiLiveTest.java +++ b/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/VirtualMachineScaleSetApiLiveTest.java @@ -16,7 +16,18 @@ */ package org.jclouds.azurecompute.arm.features; -import com.google.common.collect.ImmutableMap; +import static org.testng.Assert.assertEquals; +import static org.testng.Assert.assertNotNull; +import static org.testng.Assert.assertTrue; + +import java.net.URI; +import java.util.ArrayList; +import java.util.Arrays; +import java.util.Collections; +import java.util.HashMap; +import java.util.List; +import java.util.Map; + import org.jclouds.azurecompute.arm.domain.Extension; import org.jclouds.azurecompute.arm.domain.ExtensionProfile; import org.jclouds.azurecompute.arm.domain.ExtensionProfileSettings; @@ -51,17 +62,7 @@ import org.jclouds.azurecompute.arm.internal.BaseAzureComputeApiLiveTest; import org.testng.annotations.BeforeClass; import org.testng.annotations.Test; -import java.net.URI; -import java.util.ArrayList; -import java.util.Collections; -import java.util.List; -import java.util.Map; -import java.util.HashMap; -import java.util.Arrays; - -import static org.testng.Assert.assertEquals; -import static org.testng.Assert.assertNotNull; -import static org.testng.Assert.assertTrue; +import com.google.common.collect.ImmutableMap; @Test(groups = "live", testName = "VirtualMachineScaleSetApiLiveTest") public class VirtualMachineScaleSetApiLiveTest extends BaseAzureComputeApiLiveTest { @@ -249,10 +250,9 @@ public class VirtualMachineScaleSetApiLiveTest extends BaseAzureComputeApiLiveTe private NetworkInterfaceCard createNetworkInterfaceCard(final String resourceGroupName, String networkInterfaceCardName, String locationName, String ipConfigurationName) { //Create properties object final NetworkInterfaceCardProperties networkInterfaceCardProperties = NetworkInterfaceCardProperties - .builder() - .ipConfigurations( - Arrays.asList(IpConfiguration.create(ipConfigurationName, null, null, null, IpConfigurationProperties - .create(null, null, "Dynamic", IdReference.create(subnetId), null, null, null)))).build(); + .builder().ipConfigurations(Arrays.asList(IpConfiguration.create(ipConfigurationName, null, null, + IpConfigurationProperties + .create(null, null, "Dynamic", IdReference.create(subnetId), null, null, null, null)))).build(); final Map<String, String> tags = ImmutableMap.of("jclouds", "livetest"); return api.getNetworkInterfaceCardApi(resourceGroupName).createOrUpdate(networkInterfaceCardName, locationName, networkInterfaceCardProperties, tags); http://git-wip-us.apache.org/repos/asf/jclouds/blob/e4f1823f/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/VirtualMachineScaleSetApiMockTest.java ---------------------------------------------------------------------- diff --git a/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/VirtualMachineScaleSetApiMockTest.java b/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/VirtualMachineScaleSetApiMockTest.java index e0420fa..a87bfe7 100644 --- a/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/VirtualMachineScaleSetApiMockTest.java +++ b/providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/features/VirtualMachineScaleSetApiMockTest.java @@ -16,7 +16,18 @@ */ package org.jclouds.azurecompute.arm.features; -import com.google.common.collect.ImmutableMap; +import static com.google.common.collect.Iterables.isEmpty; +import static org.testng.Assert.assertEquals; +import static org.testng.Assert.assertNotNull; +import static org.testng.Assert.assertNull; +import static org.testng.Assert.assertTrue; + +import java.util.ArrayList; +import java.util.Arrays; +import java.util.HashMap; +import java.util.List; +import java.util.Map; + import org.jclouds.azurecompute.arm.domain.DataDisk; import org.jclouds.azurecompute.arm.domain.Extension; import org.jclouds.azurecompute.arm.domain.ExtensionProfile; @@ -49,17 +60,7 @@ import org.jclouds.azurecompute.arm.domain.VirtualMachineScaleSetVirtualMachineP import org.jclouds.azurecompute.arm.internal.BaseAzureComputeApiMockTest; import org.testng.annotations.Test; -import java.util.ArrayList; -import java.util.Arrays; -import java.util.HashMap; -import java.util.List; -import java.util.Map; - -import static com.google.common.collect.Iterables.isEmpty; -import static org.testng.Assert.assertEquals; -import static org.testng.Assert.assertNotNull; -import static org.testng.Assert.assertNull; -import static org.testng.Assert.assertTrue; +import com.google.common.collect.ImmutableMap; @Test(groups = "unit", testName = "VirtualMachineScaleSetAPIMockTest", singleThreaded = true) @@ -349,9 +350,9 @@ public class VirtualMachineScaleSetApiMockTest extends BaseAzureComputeApiMockTe IdReference.create( "/subscriptions/xxxxx-xxxx-xxxx-xxxx-xxxxxx/resourceGroups/" + "jcloud-eastus/providers/" + - "Microsoft.Network/virtualNetworks/" + - "jclouds-eastus-virtualNetworkName/subnets/" + - "jclouds-eastus-subnet") + "Microsoft.Network/virtualNetworks/" + "jclouds-eastus-virtualNetworkName" + + "/subnets/" + + "jclouds-eastus-subnet"), Boolean.TRUE ); Map<String, String> tags = ImmutableMap.of("jclouds", "livetest"); http://git-wip-us.apache.org/repos/asf/jclouds/blob/e4f1823f/providers/azurecompute-arm/src/test/resources/createnetworkinterfacecard.json ---------------------------------------------------------------------- diff --git a/providers/azurecompute-arm/src/test/resources/createnetworkinterfacecard.json b/providers/azurecompute-arm/src/test/resources/createnetworkinterfacecard.json index d08b8f6..15c3f20 100644 --- a/providers/azurecompute-arm/src/test/resources/createnetworkinterfacecard.json +++ b/providers/azurecompute-arm/src/test/resources/createnetworkinterfacecard.json @@ -1,35 +1,50 @@ { - "name": "myNic", - "id": "/subscriptions/12345678-2749-4e68-9dcf-123456789abc/resourceGroups/azurearmtesting/providers/Microsoft.Network/networkInterfaces/myNic", - "etag": "W/\"6b51f6e7-232b-4289-b740-04a996929f5e\"", - "type": "Microsoft.Network/networkInterfaces", - "location": "northeurope", + "name": "myusualvm607", + "id": "/subscriptions/bd81406c-6028-4037-9f03-9a3af4ff725d/resourceGroups/abiquo-eastus/providers/Microsoft.Network/networkInterfaces/myusualvm607", + "etag": "W/\"01f72f0f-a12f-426a-bf9a-25d09be49cee\"", + "location": "eastus", "tags": { "mycustomtag": "foobar" }, "properties": { "provisioningState": "Succeeded", - "resourceGuid": "f3465472-536f-49e7-9e9c-fa91b971a618", + "resourceGuid": "40a46008-5368-4e9f-ba39-a87b76ea047d", "ipConfigurations": [ { - "name": "myip1", - "id": "/subscriptions/12345678-2749-4e68-9dcf-123456789abc/resourceGroups/azurearmtesting/providers/Microsoft.Network/networkInterfaces/myNic/ipConfigurations/myip1", - "etag": "W/\"6b51f6e7-232b-4289-b740-04a996929f5e\"", + "name": "ipconfig1", + "id": "/subscriptions/bd81406c-6028-4037-9f03-9a3af4ff725d/resourceGroups/abiquo-eastus/providers/Microsoft.Network/networkInterfaces/myusualvm607/ipConfigurations/ipconfig1", + "etag": "W/\"01f72f0f-a12f-426a-bf9a-25d09be49cee\"", "properties": { "provisioningState": "Succeeded", - "privateIPAddress": "10.2.0.4", + "privateIPAddress": "192.168.0.4", "privateIPAllocationMethod": "Dynamic", + "publicIPAddress": { + "id": "/subscriptions/bd81406c-6028-4037-9f03-9a3af4ff725d/resourceGroups/abiquo-eastus/providers/Microsoft.Network/publicIPAddresses/myusualvm-ip" + }, "subnet": { - "id": "/subscriptions/12345678-2749-4e68-9dcf-123456789abc/resourceGroups/azurearmtesting/providers/Microsoft.Network/virtualNetworks/myvirtualnetwork/subnets/mysubnet" + "id": "/subscriptions/bd81406c-6028-4037-9f03-9a3af4ff725d/resourceGroups/abiquo-eastus/providers/Microsoft.Network/virtualNetworks/abqvnet-bf0tuznt0x/subnets/abqsubnet-reauuik6qx" }, - "primary": true + "primary": true, + "privateIPAddressVersion": "IPv4", + "isInUseWithService": false } } ], "dnsSettings": { "dnsServers": [], - "appliedDnsServers": [] + "appliedDnsServers": [], + "internalDomainNameSuffix": "zqh4yoheybzejmruwtgyl2semg.bx.internal.cloudapp.net" + }, + "macAddress": "00-0D-3A-17-A4-C9", + "enableAcceleratedNetworking": false, + "enableIPForwarding": false, + "networkSecurityGroup": { + "id": "/subscriptions/bd81406c-6028-4037-9f03-9a3af4ff725d/resourceGroups/abiquo-eastus/providers/Microsoft.Network/networkSecurityGroups/myusualvm-nsg" }, - "enableIPForwarding": false - } -} \ No newline at end of file + "primary": true, + "virtualMachine": { + "id": "/subscriptions/bd81406c-6028-4037-9f03-9a3af4ff725d/resourceGroups/abiquo-eastus/providers/Microsoft.Compute/virtualMachines/myusualvm" + } + }, + "type": "Microsoft.Network/networkInterfaces" +} http://git-wip-us.apache.org/repos/asf/jclouds/blob/e4f1823f/providers/azurecompute-arm/src/test/resources/getnetworkinterfacecard.json ---------------------------------------------------------------------- diff --git a/providers/azurecompute-arm/src/test/resources/getnetworkinterfacecard.json b/providers/azurecompute-arm/src/test/resources/getnetworkinterfacecard.json index f613291..041b575 100644 --- a/providers/azurecompute-arm/src/test/resources/getnetworkinterfacecard.json +++ b/providers/azurecompute-arm/src/test/resources/getnetworkinterfacecard.json @@ -26,10 +26,11 @@ } } ], + "primary": true, "dnsSettings": { "dnsServers": [], "appliedDnsServers": [] }, "enableIPForwarding": false } -} \ No newline at end of file +}
