jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Everett Toews <notificati...@github.com>
Subject Re: [jclouds] Adds a top-level zone filter. (#283)
Date Wed, 05 Feb 2014 20:06:09 GMT
> @@ -74,6 +77,20 @@ protected String setIfTestSystemPropertyPresent(Properties overrides,
String key
>        return null;
>     }
>  
> +   /**
> +    * This helps live testing against specific zones only.
> +    * @param zones A list of zones, usually from getConfiguredZones()
> +    * @return a set of zones in the given set that are also contained in the set specified
by the test.live.region system property
> +    */
> +   protected Set<String> filterZones(Set<String> zones) {
> +      String zonesToList = System.getProperty("test.live.region");

It's odd to have all of the variables and method with "zone" in them but the property is `test.live.region`.
Is there a reason it's `test.live.region` could it just as easily be `test.live.zone`?

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