> +
> +   @Inject
> +   public CreateUniqueKeyPair(CloudStackClient client) {
> +      this.client = checkNotNull(client, "client");
> +   }
> +
> +   @Override
> +   public SshKeyPair load(String input) {
> +      SshKeyPair keyPair = null;
> +      while (keyPair == null) {
> +         try {
> +            keyPair = client.getSSHKeyPairClient().createSSHKeyPair(input);
> +            logger.debug(">> creating SSH key pair with name %s", input);
> +         } catch (IllegalStateException e) {
> +            
> +         }

If this is intentional, please add a comment `// ignored` or name the exception 
`IllegalStateException ignored` or something like that. Should we add a logging 
statement?

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

Reply via email to