Re: [jclouds/jclouds] Adds method checkIpAvailability to VirtualNetworkApi (#1231)

2018-07-13 Thread Ignasi Barrera
Closed #1231. -- 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/pull/1231#event-1732679475

Re: [jclouds/jclouds] Adds method checkIpAvailability to VirtualNetworkApi (#1231)

2018-07-13 Thread Ignasi Barrera
Pushed to [master](http://git-wip-us.apache.org/repos/asf/jclouds/commit/92076d6b) and [2.1.x](http://git-wip-us.apache.org/repos/asf/jclouds/commit/4d1f64d7). Thanks! -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [jclouds/jclouds] Adds method checkIpAvailability to VirtualNetworkApi (#1231)

2018-07-12 Thread Daniel Estévez
Squashed and ready to merge, thanks @nacx ! -- 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/pull/1231#issuecomment-404673071

Re: [jclouds/jclouds] Adds method checkIpAvailability to VirtualNetworkApi (#1231)

2018-07-12 Thread Ignasi Barrera
nacx approved this pull request. -- 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/pull/1231#pullrequestreview-136854963

Re: [jclouds/jclouds] Adds method checkIpAvailability to VirtualNetworkApi (#1231)

2018-07-12 Thread Daniel Estévez
@danielestevez pushed 1 commit. e71ce96 Fixed formatter maximum line width. Removed redundantTest -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [jclouds/jclouds] Adds method checkIpAvailability to VirtualNetworkApi (#1231)

2018-07-12 Thread Ignasi Barrera
nacx commented on this pull request. > + public void checkIpAvailable() throws InterruptedException { + server.enqueue(jsonResponse("/ipavailable.json")); + + final VirtualNetworkApi vnApi = api.getVirtualNetworkApi(resourcegroup); + IpAddressAvailabilityResult checkResult =

Re: [jclouds/jclouds] Adds method checkIpAvailability to VirtualNetworkApi (#1231)

2018-07-12 Thread Daniel Estévez
danielestevez commented on this pull request. > + public void checkIpAvailable() throws InterruptedException { + server.enqueue(jsonResponse("/ipavailable.json")); + + final VirtualNetworkApi vnApi = api.getVirtualNetworkApi(resourcegroup); + IpAddressAvailabilityResult

Re: [jclouds/jclouds] Adds method checkIpAvailability to VirtualNetworkApi (#1231)

2018-07-12 Thread Ignasi Barrera
nacx commented on this pull request. > + public void checkIpAvailable() throws InterruptedException { + server.enqueue(jsonResponse("/ipavailable.json")); + + final VirtualNetworkApi vnApi = api.getVirtualNetworkApi(resourcegroup); + IpAddressAvailabilityResult checkResult =

Re: [jclouds/jclouds] Adds method checkIpAvailability to VirtualNetworkApi (#1231)

2018-07-12 Thread Daniel Estévez
@danielestevez pushed 1 commit. 37ce4a7 Addresses @nacx comments -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/jclouds/jclouds/pull/1231/files/427fcdb34f6e8698e2c0ad218f65aac4a8964ef8..37ce4a7d29ad54d9b9436e668336bed07a998a7f

Re: [jclouds/jclouds] Adds method checkIpAvailability to VirtualNetworkApi (#1231)

2018-07-12 Thread Daniel Estévez
danielestevez commented on this pull request. > VirtualNetwork vn = api().createOrUpdate(virtualNetworkName, LOCATION, null, virtualNetworkProperties); + resourceAvailable.apply(new Supplier() { + @Override + public Provisionable get() { +

Re: [jclouds/jclouds] Adds method checkIpAvailability to VirtualNetworkApi (#1231)

2018-07-12 Thread Daniel Estévez
danielestevez commented on this pull request. > + */ +package org.jclouds.azurecompute.arm.domain; + +import java.util.List; + +import org.jclouds.javax.annotation.Nullable; +import org.jclouds.json.SerializedNames; + +import com.google.auto.value.AutoValue; +import

Re: [jclouds/jclouds] Adds method checkIpAvailability to VirtualNetworkApi (#1231)

2018-07-12 Thread Daniel Estévez
danielestevez commented on this pull request. > @@ -73,4 +74,11 @@ VirtualNetwork > createOrUpdate(@PathParam("virtualnetworkname") String virtualnet @DELETE @ResponseParser(FalseOn204.class) boolean delete(@PathParam("virtualnetworkname") String virtualnetworkname); + +

Re: [jclouds/jclouds] Adds method checkIpAvailability to VirtualNetworkApi (#1231)

2018-07-12 Thread Ignasi Barrera
nacx commented on this pull request. > + */ +package org.jclouds.azurecompute.arm.domain; + +import java.util.List; + +import org.jclouds.javax.annotation.Nullable; +import org.jclouds.json.SerializedNames; + +import com.google.auto.value.AutoValue; +import