When configuring locations from properties, the `ZoneIdsSupplier` is [already 
filtered](https://github.com/jclouds/jclouds/blob/master/core/src/main/java/org/jclouds/location/config/LocationModule.java#L129)
 by the value of the `jclouds.regions` property. However, the [default 
implementation](https://github.com/jclouds/jclouds/blob/master/core/src/main/java/org/jclouds/location/suppliers/LocationsSupplier.java#L27)
 of the `LocationsSupplier` does not take that into account and returns 
inconsistent results: it already filters the regions according to the region 
filters, but always returns all the zones.

This patch adds the zone filter so all suppliers return the locations 
consistently.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Consider the zone id filter in the location supplier

-- File Changes --

    M 
core/src/main/java/org/jclouds/location/suppliers/all/ZoneToRegionToProviderOrJustProvider.java
 (5)
    A 
core/src/test/java/org/jclouds/location/suppliers/all/ZoneToRegionToProviderOrJustProviderTest.java
 (82)

-- Patch Links --

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

Reply via email to