> +        super(locations, images, sizes, defaultLocation, optionsProvider, 
> defaultTemplateProvider, getImageStrategy);
> +        imageChooser = imageChooser(IMAGE_ORDERING);
> +    }
> +
> +    Function<Iterable<? extends Image>, Image> imageChooser(final 
> Ordering<Image> ordering) {
> +        return new Function<Iterable<? extends Image>, Image>() {
> +            @Override
> +            public Image apply(Iterable<? extends Image> input) {
> +                // filter private images
> +                Predicate<Image> statusPredicate = new Predicate<Image>() {
> +
> +                    @Override
> +                    public boolean apply(Image image) {
> +                        return image.getStatus() == Image.Status.AVAILABLE;
> +                    }
> +                };

Is this predicate needed? According to [your image transformation 
function](https://github.com/MSOpenTech/jclouds-labs/blob/azure/azure/compute/src/main/java/org/jclouds/azurecompute/compute/functions/OSImageToImage.java#L83)
 images are always available. If that's not true, then that function should be 
improved to populate the right status.

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

Reply via email to