jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Phillips <notificati...@github.com>
Subject Re: [jclouds] Prefer OpenStack Regions (#463)
Date Wed, 13 Aug 2014 17:42:16 GMT
> @@ -160,19 +161,19 @@ public FlavorInZone apply(Flavor arg0) {
>     }
>  
>     @Override
> -   public Iterable<ImageInZone> listImages() {
> -      Builder<ImageInZone> builder = ImmutableSet.builder();
> -      Set<String> zones = zoneIds.get();
> -      checkState(zones.size() > 0, "no zones found in supplier %s", zoneIds);
> -      for (final String zoneId : zones) {
> -         Set<? extends Image> images = novaApi.getImageApiForZone(zoneId).listInDetail().concat().toSet();
> +   public Iterable<ImageInRegion> listImages() {
> +      Builder<ImageInRegion> builder = ImmutableSet.builder();
> +      Set<String> regions = regionIds.get();
> +      checkState(regions.size() > 0, "no regions found in supplier %s", regionIds);
> +      for (final String regionId : regions) {
> +         Set<? extends Image> images = novaApi.getImageApi(regionId).listInDetail().concat().toSet();
>           if (images.size() == 0) {

Same as above: `images.isEmpty()`

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/463/files#r16191667
Mime
View raw message