> +                      "    \"template\" : " + template + ", " +
> +                      "    \"mode\" : \"" + mode + "\", " +
> +                      "    \"parameters\" : " + parameters + " " +
> +                      "  } " +
> +                      "}";
> +        return body;
> +    }
> +
> +    @Test(groups = "live")
> +    public void testCreate() {
> +        Long now = System.currentTimeMillis();
> +        resourceName = "jcres" + now;
> +        deploymentName = "jcdep" + now;
> +        String template = 
> "{\"$schema\":\"https://schema.management.azure.com/schemas/2015-01-01/deploymentTemplate.json#\",\"contentVersion\":\"1.0.0.0\",\"parameters\":{\"newStorageAccountName\":{\"type\":\"string\",\"metadata\":{\"description\":\"Name
>  of the Storage 
> Account\"}},\"storageAccountType\":{\"type\":\"string\",\"defaultValue\":\"Standard_LRS\",\"allowedValues\":[\"Standard_LRS\",\"Standard_GRS\",\"Standard_ZRS\"],\"metadata\":{\"description\":\"Storage
>  Account type\"}},\"location\":{\"type\":\"string\",\"allowedValues\":[\"East 
> US\",\"West US\",\"West Europe\",\"East Asia\",\"Southeast 
> Asia\"],\"metadata\":{\"description\":\"Location of storage 
> account\"}}},\"resources\":[{\"type\":\"Microsoft.Storage/storageAccounts\",\"name\":\"[parameters('newStorageAccountName')]\",\"apiVersion\":\"2015-05-01-preview\",\"location\":\"[parameters('location')]\",\"properties\":{\"accountType\":\"[parameters('storageAccountType')]\"}}]}";
> +        String parameters = "{\"newStorageAccountName\":{\"value\":\"" + 
> resourceName + 
> "\"},\"storageAccountType\":{\"value\":\"Standard_LRS\"},\"location\":{\"value\":\"West
>  US\"}}";
> +        String properties = getPutBody(template, "Incremental", parameters);

Ok, let's keep the current API signature then. Thanks for taking your tine to 
discuss and dissgree! :)

>WDYT? The current implementation allows users to use the DeploymentApi 
>directly with the raw json template as intended by the original api, but it 
>also supports abstracting the raw json template with a builder if users use 
>jclouds apis

That sounds good. Does the builder already generate the json?

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/273/files/74f8e9bf66e7427298c5c3c9567aa7d8284e0987#r65033690

Reply via email to