> [...] would it be easier to commit
WHIRR-222 then do this one?
Yes. I've updated the patch for
WHIRR-222. +1 for committing that patch first.
> It would be better to add a blobstore provider property, rather than inferring it from the provider.
What if I we keep the current behaviour as default and make the new configuration parameters optional?
> ClusterSpec should really be little more than a configuration holder, with little to no logic.
I agree. I was going to move the code to a new class. Any ideas for a good name?
> Should uploaded files be removed at cluster destroy time?
Let's not worry about this now. We could handle this while solving the cluster state persistence problem.
> You can copy the remote resource to the local filesystem on each node using a SaveHttpResponseTo statement.
Sounds like a good approach.