nacx commented on this pull request.
> - vmLocations.addAll(m.locations()); - break; - } - } - - Iterable<Location> result = Iterables.filter(locations, new Predicate<Location>() { - @Override - public boolean apply(Location input) { - return vmLocations.contains(input.displayName()); - } - }); - - return result; + private Iterable<String> findWhiteListOfRegions() { + if (providerMetadata.getDefaultProperties().get(LocationConstants.PROPERTY_REGIONS) == null) return null; + return Splitter.on(",").trimResults().split((CharSequence) providerMetadata.getDefaultProperties().get(LocationConstants.PROPERTY_REGIONS)); Is this cast to CharSequence needed? -- 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/308#pullrequestreview-564604