> +
> +   private Location createLocation(String input) {
> +      return new 
> LocationBuilder().id(input).scope(LocationScope.REGION).description(input).parent(
> +            
> Iterables.getOnlyElement(provider.get())).metadata(ImmutableMap.<String, 
> Object>of("name", input)).build();
> +   }
> +
> +   public static Function<String, OsFamily> osFamily() {
> +      return new Function<String, OsFamily>() {
> +         @Override
> +         public OsFamily apply(final String label) {
> +            if (label != null) {
> +               if (label.toUpperCase().contains("CENTOS"))
> +                  return OsFamily.CENTOS;
> +               if (label.toUpperCase().contains("OPENLOGIC"))
> +                  return OsFamily.CENTOS;
> +               else if (label.contains("SUSE"))

No need to call `toUpperCase()` here?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/117/files#r22192100

Reply via email to