> +      this.imageAvailablePredicate = checkNotNull(imageAvailablePredicate, 
> "imageAvailablePredicate");
> +   }
> +
> +   @Override
> +   public ImageTemplate buildImageTemplateFromNode(String name, final String 
> id) {
> +      Container container = api.getRemoteApi().inspectContainer(id);
> +      if (container == null)
> +         throw new NoSuchElementException("Cannot find container with id: " 
> + id);
> +      CloneImageTemplate template = new 
> ImageTemplateBuilder.CloneImageTemplateBuilder().nodeId(id).name(name).build();
> +      return template;
> +   }
> +
> +   @Override
> +   public ListenableFuture<Image> createImage(ImageTemplate template) {
> +      checkState(template instanceof CloneImageTemplate,
> +              " docker only currently supports creating images through 
> cloning.");

Should better use `checkArgument` to propagate an `IllegalargumentException`.

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

Reply via email to