cloudmesh / client

Deprecated: Version 3 of cloudmesh is a light weight cloud client to manage virtual clusters
http://cloudmesh.github.io/client
Other
3 stars 12 forks source link

Provider: self.kind vs self.cloud_type #182

Closed laszewsk closed 8 years ago

laszewsk commented 8 years ago

P:20

There may be a significant bug in the prg, but I Am not yet sure

originally we used kind to indicate the cloud type such as openstack, but we also used kind in other places to indicate which kind of information is represented in a provider such as

vm, image, flavor, list, quota

The reason we changed from type to kind is that type is a predefined word in python. But we forgot that we already used kind In the base class.

I suggest that MAngirish and Gourav look into this error and fix if needed. I am actually confused that openstack works ;-)

class CloudProviderOpenstackAPI(CloudProviderBase):
    kind = "openstack" # BUG this should be cloud_type
    cloud_pwd = {}

    def __init__(self, cloud_name, cloud_details, user=None, flat=True):
        super(CloudProviderOpenstackAPI, self).__init__(cloud_name, user=user)
        self.flat = flat
        self.kind = "openstack" # BUG this should be cloud_type
laszewsk commented 8 years ago

I labeled you all on this bug as this is real important