Re: [jclouds/jclouds-labs] JCLOUDS-1254 Filters storage account by current available locations (#370)

2017-03-28 Thread Ignasi Barrera 
Closed #370.

-- 
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/370#event-1017962865

Re: [jclouds/jclouds-labs] JCLOUDS-1254 Filters storage account by current available locations (#370)

2017-03-18 Thread Ignasi Barrera 
rebuild please

-- 
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/370#issuecomment-287524079

Re: [jclouds/jclouds-labs] JCLOUDS-1254 Filters storage account by current available locations (#370)

2017-03-17 Thread Ignasi Barrera 
The build failed due to a connection timeout. Let's restart it: rebuild please

-- 
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/370#issuecomment-287498371

Re: [jclouds/jclouds-labs] JCLOUDS-1254 Filters storage account by current available locations (#370)

2017-03-17 Thread Daniel Estévez 
danielestevez commented on this pull request.



> @@ -16,10 +16,23 @@
  */
 package org.jclouds.azurecompute.arm.compute;
 
+import static com.google.common.base.Preconditions.checkState;
+import static com.google.common.collect.Iterables.contains;
+import static com.google.common.collect.Iterables.filter;
+import static com.google.common.collect.Iterables.find;
+import static com.google.common.collect.Iterables.getOnlyElement;
+import static 
org.jclouds.azurecompute.arm.compute.extensions.AzureComputeImageExtension.CONTAINER_NAME;
+import static 
org.jclouds.azurecompute.arm.compute.extensions.AzureComputeImageExtension.CUSTOM_IMAGE_OFFER;
+import static 
org.jclouds.azurecompute.arm.compute.functions.VMImageToImage.decodeFieldsFromUniqueId;
+import static 
org.jclouds.azurecompute.arm.compute.functions.VMImageToImage.encodeFieldsToUniqueIdCustom;
+import static 
org.jclouds.azurecompute.arm.compute.functions.VMImageToImage.getMarketplacePlanFromImageMetadata;
+import static 
org.jclouds.azurecompute.arm.config.AzureComputeProperties.IMAGE_PUBLISHERS;
+import static 
org.jclouds.compute.util.ComputeServiceUtils.metadataAndTagsAsCommaDelimitedValue;
+import static org.jclouds.util.Closeables2.closeQuietly;

My IDE is configured to put all static imports first. Not sure if there is an 
established style convention for this issue in jclouds.

-- 
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/370#pullrequestreview-27702778

[jclouds/jclouds-labs] JCLOUDS-1254 Filters storage account by current available locations (#370)

2017-03-17 Thread Daniel Estévez 

You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds-labs/pull/370

-- Commit Summary --

  * JCLOUDS-1254 Filters storage account by current available locations

-- File Changes --

M 
azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/compute/AzureComputeServiceAdapter.java
 (42)

-- Patch Links --

https://github.com/jclouds/jclouds-labs/pull/370.patch
https://github.com/jclouds/jclouds-labs/pull/370.diff

-- 
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/370