mgmt, bug fix, container group without ports (#24418)

Azure · Sep 29, 2021 · 7441a50 · 7441a50
1 parent 95c0df8
commit 7441a50
Show file tree

Hide file tree

Showing 3 changed files with 294 additions and 295 deletions.
diff --git a/...n/java/com/azure/resourcemanager/containerinstance/implementation/ContainerGroupImpl.java b/...n/java/com/azure/resourcemanager/containerinstance/implementation/ContainerGroupImpl.java
@@ -459,10 +459,9 @@ public ContainerGroupImpl withExistingNetworkProfile(
     @Override
     public ContainerGroupImpl withExistingNetworkProfile(String networkProfileId) {
         this.innerModel().withNetworkProfile(new ContainerGroupNetworkProfile().withId(networkProfileId));
-        if (this.innerModel().ipAddress() == null) {
-            this.innerModel().withIpAddress(new IpAddress());
+        if (this.innerModel().ipAddress() != null) {
+            this.innerModel().ipAddress().withType(ContainerGroupIpAddressType.PRIVATE);
         }
-        this.innerModel().ipAddress().withType(ContainerGroupIpAddressType.PRIVATE);
         return this;
     }
 

diff --git a/...nstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java b/...nstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java
@@ -66,7 +66,7 @@ public void testContainerGroupWithVirtualNetwork() {
             .withLinux()
             .withPublicImageRegistryOnly()
             .withoutVolume()
-            .withContainerInstance("nginx", 80)
+            .withContainerInstance("nginx")
             .withExistingNetworkProfile(containerGroup1.networkProfileId())
             .create();