> +              .defaultProperties(ProfitBricksApiMetadata.defaultProperties())
> +              .defaultModules(ImmutableSet.<Class<? extends 
> Module>>of(ProfitBricksHttpApiModule.class));
> +      }
> +
> +      @Override
> +      public ProfitBricksApiMetadata build() {
> +      return new ProfitBricksApiMetadata(this);
> +      }
> +
> +      @Override
> +      protected Builder self() {
> +      return this;
> +      }
> +   }
> +
> +}

As ProfitBricks is a provider, not only a generic API, you should add the 
ProviderMetadata too.
When it comes to implement the ComputeService abstraction, you can put there 
the default locations and other default properties for the provider, but the 
ProviderMetadata class should be added now.

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

Reply via email to