Miscellaneous bugfixs from release

src/main/java/gyro/azure/containerservice/ClusterLoadBalancerProfile.java

@@ -66,10 +66,6 @@ public void setEffectiveOutboundIps(List<String> effectiveOutboundIps) {
      * If set to ``true`` enables multiple standard load balancer. Defaults to ``false``.
      */
     public Boolean getEnableMultipleStandardLoadBalancers() {
-        if (enableMultipleStandardLoadBalancers == null) {
-            enableMultipleStandardLoadBalancers = false;
-        }
-
         return enableMultipleStandardLoadBalancers;
     }
 
@@ -171,7 +167,9 @@ protected ManagedClusterLoadBalancerProfile toClusterLoadBalancerProfile() {
             profile.withAllocatedOutboundPorts(getAllocatedOutboundPorts());
         }
 
-        profile.withEnableMultipleStandardLoadBalancers(getEnableMultipleStandardLoadBalancers());
+        if (getEnableMultipleStandardLoadBalancers() != null) {
+            profile.withEnableMultipleStandardLoadBalancers(getEnableMultipleStandardLoadBalancers());
+        }
 
         if (getIdleTimeoutInMinutes() != null) {
             profile.withIdleTimeoutInMinutes(getIdleTimeoutInMinutes());

src/main/java/gyro/azure/network/NetworkResource.java

@@ -132,6 +132,7 @@ public void setAddressSpaces(Set<String> addressSpaces) {
      *
      * @subresource gyro.azure.network.SubnetResource
      */
+    @Required
     public Set<SubnetResource> getSubnet() {
         if (subnet == null) {
             subnet = new HashSet<>();
@@ -273,12 +274,13 @@ public void create(GyroUI ui, State state) {
             withAddressSpace = networkDefWithoutAddress.withAddressSpace(addressSpace);
         }
 
+        withAddressSpace = withAddressSpace.withSubnets(getSubnet().stream()
+            .collect(Collectors.toMap(SubnetResource::getName, SubnetResource::getAddressPrefix)));
+
         Network network = withAddressSpace
             .withTags(getTags())
             .create();
 
-        network = network.update().withoutSubnet("subnet1").apply();
-
         copyFrom(network);
     }
 

src/main/java/gyro/azure/network/SubnetResource.java

@@ -151,6 +151,7 @@ public void copyFrom(Subnet subnet) {
             RouteTableResource.class,
             subnet.routeTableId()) : null);
         setServiceEndpoints(toServiceEndpoints(subnet.servicesWithAccess()));
+        setId(subnet.innerModel().id());
     }
 
     @Override
@@ -186,6 +187,8 @@ public void create(GyroUI ui, State state) {
 
         Network response = updateWithAttach.attach().apply();
         setId(response.subnets().get(getName()).id());
+
+        parent.refresh();
     }
 
     @Override

src/main/java/gyro/azure/storage/CloudBlobContainerResource.java

@@ -18,13 +18,17 @@
 
 import java.util.HashMap;
 import java.util.Map;
+import java.util.Optional;
 import java.util.Set;
 import java.util.concurrent.TimeUnit;
 
+import com.azure.core.http.rest.Response;
 import com.azure.storage.blob.BlobContainerClient;
 import com.azure.storage.blob.BlobServiceClient;
 import com.azure.storage.blob.BlobServiceClientBuilder;
+import com.azure.storage.blob.models.BlobContainerAccessPolicies;
 import com.azure.storage.blob.models.PublicAccessType;
+import com.azure.storage.blob.options.BlobContainerCreateOptions;
 import gyro.azure.AzureResource;
 import gyro.azure.Copyable;
 import gyro.core.GyroUI;
@@ -77,7 +81,6 @@ public void setName(String name) {
     /**
      * The public access of the container.
      */
-    @Required
     @ValidStrings({ "blob", "container" })
     @Updatable
     public String getPublicAccess() {
@@ -127,7 +130,8 @@ public void setId(String id) {
 
     @Override
     public void copyFrom(BlobContainerClient container) {
-        setPublicAccess(container.getAccessPolicy().getBlobAccessType().toString());
+        setPublicAccess(Optional.ofNullable(container).map(BlobContainerClient::getAccessPolicy).map(
+            BlobContainerAccessPolicies::getBlobAccessType).map(PublicAccessType::toString).orElse(null));
         setName(container.getBlobContainerName());
         setMetadata(container.getProperties().getMetadata());
         setStorageAccount(findById(StorageAccountResource.class, container.getAccountName()));
@@ -150,16 +154,34 @@ public boolean refresh() {
     public void create(GyroUI ui, State state) {
         BlobContainerClient blobContainer = blobContainer();
 
-        blobContainer.create();
-
-        blobContainer = blobContainer();
+        BlobContainerCreateOptions blobContainerCreateOptions = new BlobContainerCreateOptions();
 
-        blobContainer.setAccessPolicy(PublicAccessType.fromString(getPublicAccess()), null);
+        if (getPublicAccess() != null) {
+            blobContainerCreateOptions.setPublicAccessType(PublicAccessType.fromString(getPublicAccess()));
+        }
 
         if (!getMetadata().isEmpty()) {
-            blobContainer.setMetadata(getMetadata());
+            blobContainerCreateOptions.setMetadata(getMetadata());
         }
 
+        // We add this wait to create as sometimes there is a delay for the StorageAccount to actually become public.
+        // This results in a 409 error that gets thrown
+        // Unfortunately th api doesn't really provide a good way to determine if the account is actually public.
+        Wait.atMost(2, TimeUnit.MINUTES)
+            .prompt(false)
+            .checkEvery(10, TimeUnit.SECONDS)
+            .until(() -> {
+                Response<Boolean> response =
+                    blobContainer.createIfNotExistsWithResponse(blobContainerCreateOptions, null, null);
+
+                if (response.getStatusCode() == 409) {
+                    return false;
+
+                } else {
+                    return true;
+                }
+            });
+
         copyFrom(blobContainer);
     }