Re: [jclouds/jclouds] Fixes assert for IpPermissions collection (#1209)

2018-05-17 Thread Ignasi Barrera
Merged to 
[master](http://git-wip-us.apache.org/repos/asf/jclouds/commit/a918550e) and 
[2.1.x](http://git-wip-us.apache.org/repos/asf/jclouds/commit/6edb99c6).

-- 
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/1209#issuecomment-389768857

Re: [jclouds/jclouds] Fixes assert for IpPermissions collection (#1209)

2018-05-17 Thread Ignasi Barrera
Closed #1209.

-- 
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/1209#event-1631251748

Re: [jclouds/jclouds] Fixes assert for IpPermissions collection (#1209)

2018-05-16 Thread Daniel Estévez
Fail!... and fixed!

-- 
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/1209#issuecomment-389517489

Re: [jclouds/jclouds] Fixes assert for IpPermissions collection (#1209)

2018-05-16 Thread Ignasi Barrera
Thanks @danielestevez! Can you check and fix the checkstyle violations in the 
ARM provider?

-- 
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/1209#issuecomment-389469592

[jclouds/jclouds] Fixes assert for IpPermissions collection (#1209)

2018-05-15 Thread Daniel Estévez
Azure does not return IpPermissions in order so this test fails randomly
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds/pull/1209

-- Commit Summary --

  * Fixes assert for IpPermissions collection

-- File Changes --

M 
providers/azurecompute-arm/src/test/java/org/jclouds/azurecompute/arm/compute/extensions/AzureComputeSecurityGroupExtensionLiveTest.java
 (4)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/1209.patch
https://github.com/jclouds/jclouds/pull/1209.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/pull/1209