Hi Tomaz,

A small question on your comment about the naming of the
"create_volume" method inline below:

>>
>> # The following methods could be added to
>> libcloud.compute.base.NodeDriver:
>>
>>    def create_volume(**kwargs):
>> [SNIP]
>
>
> To be consistent with other method names it should just be called "create".
>


I named this method "create_volume" instead of just "create" because I
was thinking of adding it to libcloud.compute.base.NodeDriver, and
there is already a "create_node" method in that class.

Am I correct in assuming that you were thinking of having this method
in another class that is specific to storage volumes? Or does it seem
acceptable to leave it named as "create_volume" and keep it in the
NodeDriver interface?

- Gabriel

Reply via email to