On 07/12/2012 07:40 AM, pblaho(a)redhat.com wrote:
Looks good overall. I think we do need to test attributes specific to
different provider types, notably ec2. You indicated this too as a TODO.
But I think we should implement it now. This is something that is easy
to forget and could bite us in the future.
Conditional ack.