> @@ -85,7 +86,13 @@ protected ComputeService getComputeService() throws 
> IOException {
>        String providerValue = EnvHelper.getComputeProvider(provider);
>        String apiValue = EnvHelper.getComputeApi(api);
>        String identityValue = EnvHelper.getComputeIdentity(identity);
> -      String credentialValue = EnvHelper.getComputeCredential(credential);
> +      String credentialValue = null;
> +      if (providerValue.equals("google-compute-engine")) {
> +         credentialValue = 
> EnvHelper.getGoogleCredentialFromJsonFile(credential);
> +      } else {
> +         credentialValue = EnvHelper.getComputeCredential(credential);
> +      }

You should expand the value of credential since the user may provide property 
or environment variable instead of a command line flag:

```java
String credentialValue = EnvHelper.getComputeCredential(credential);
if (providerValue.equals("google-compute-engine")) {
    credentialValue = 
EnvHelper.getGoogleCredentialFromJsonFile(credentialValue);
}
```

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

Reply via email to