Hey Ilgiz,
I just played with the Libcloud REST and here are some issues / bugs I have
observed:
1. /<version>/compute/providers endpoint
Currently it returns this:
{"providers": ["RACKSPACE_NOVA_DFW", "IBM", "SOFTLAYER", "EC2_EU_WEST",
> "ELASTICHOSTS", "BRIGHTBOX", "BLUEBOX", "EC2_AP_SOUTHEAST", "EC2_US_EAST",
> "OPENNEBULA", "ELASTICHOSTS_UK2", "ELASTICHOSTS_UK1", "ELASTICHOSTS_US2",
> "VPSNET", "VOXEL", "RACKSPACE", "ELASTICHOSTS_US1", "RIMUHOSTING",
> "LINODE", "JOYENT", "NIMBUS", "GOGRID", "DREAMHOST", "ECP", "GANDI",
> "DUMMY", "EC2_US_WEST_OREGON", "EC2_US_WEST", "TERREMARK", "SERVERLOVE",
> "VCL", "NINEFOLD", "CLOUDSIGMA_US", "RACKSPACE_UK", "EC2_AP_NORTHEAST",
> "CLOUDSTACK", "EC2", "VCLOUD", "EC2_SA_EAST", "OPSOURCE", "EUCALYPTUS",
> "EC2_EU", "SKALICLOUD", "SLICEHOST", "LIBVIRT", "CLOUDSIGMA",
> "ELASTICHOSTS_CA1", "RACKSPACE_NOVA_BETA", "OPENSTACK"]}
It should return a flat array. "providers" object container is unnecessary.
Actually, I think we should return more information about every provider -
a list of dictionaries with the following keys:
- id - provider constant which is used in the URL (e.g. RACKSPACE)
- friendly_name (e.g. Rackspace Cloud)
- website - provider website (e.g. http://www.rackspace.com)
Friendly name is already available as a class attribute on the NodeDriver
class ("name"). For the website I'm fine with either adding a new
annotation to a docstring in the driver constructor or adding a new class
attribute to the NodeDriver class.
2. Need to add @requires annotation to constructors for all the compute
drivers.
3. "Unknown error" when trying to create node without passing any payload
in the body. Need to polish it up and return ValidationError.
4. We should return an error if user passes in invalid content type when
issuing POST or PUT request. Only valid content type should be
"application/json".
5. Typo in create_node handler - 'localtion_id'
6. Should return 201 (created) status code instead of 200 when node has
been sucesfully created (create_node).
7. Should return 204 (deleted) status code instead of 200 when a node has
been successfully destroyed (destroy_node).
8. For security reason (resource exhaustion, etc.) we need to limit a
request body size. We need to check Content-Length and return a
ValidationError if length is > MAX_BODY_LENGTH. For now we can set this
variable to 500 KB.
