Open Stack

Hi Oleg,

I took in your change on LBaas extension and working the CRUD operation
(database access).  I met some issues during the implementation.  Would
you please take a look?

 

1.       The REST url to "deassociate a health_monitor from a pool"
contains both pool_id and health_monitor_id.  By looking at the API
defined in LoadBalancerPluginBase as below, how I got the
health_monitor_id?  I think "body" parameter is referring to the request
content in JSON rather than the url parameter.  Correct me if I'm wrong.

def delete_health_monitors(self, context, pool_id, body):

 

2.       In ATTRIBUTE_MAP, the "vip_id" attribute of "pools" is validate
as uuid.  The question is user can create a pool without specifying the
VIP, which means vip_id is allowed to be empty (None).  However, in the
implementation I found that None is unacceptable for type::uuid validator.
The same for "pool_id" attribute of "members".

 

3.       I think we need to add "pool_id" attribute as part of
"health_monitors".  At least set "is_invisible" to False.  Otherwise,
there is no way to associate pool and health_monitor?

 

 

Thanks

Leon

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20121122/ce5cbd4d/attachment.html>