> +      //return api.getRemoteApi().listImages();
> +      Set<Image> images = Sets.newHashSet();
> +      for (Image image : api.getRemoteApi().listImages()) {
> +         // less efficient than just listNodes but returns richer json that 
> needs repoTags coming from listImages
> +         Image inspected = api.getRemoteApi().inspectImage(image.getId());
> +         if(image.getRepoTags() != null) {
> +            inspected = 
> Image.builder().fromImage(inspected).repoTags(image.getRepoTags()).build();
> +         }
> +         images.add(inspected);
> +      }
> +      return images;
> +   }
> +
> +   @Override
> +   public Image getImage(final String imageId) {
> +      return find(listImages(), new Predicate<Image>() {

This will call the api to inspect every image, when you only want to retrieve 
one. Change this to directly call inspectImage.

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

Reply via email to