Open Stack

Looks great Ilya!

A few small comments below:


*         On lb-pool-create command, the arguments are not optional, they have to be specified.

lb-pool-create [--network-id <network-id>] [--lb-method <lb-method>] [--protocol <protocol>] <name>


*         The type of the health-monitor cannot be updated.


lb-healthmonitor-update [--type <type>] [--delay <delay>] [--max_retries <max_retries>] [--timeout <timeout>] [--name <name>] [--http_method <http method>] [--path <path>] [--expected-codes <expected codes>] <healthmonitor-id>


*         There is also the "description" field missing in the lb-vip-create command.

Also for all the create commands, we should support supplied a "tenant_id" attribute when an admin is creating resources for another tenant (API supports supplying this).

BTW, how is the tenant keystone token specified? are we going to use env. variables ?

Thanks
Youcef

From: Ilya Shakhat [mailto:ishakhat at mirantis.com]
Sent: Wednesday, November 14, 2012 8:51 AM
To: OpenStack Development Mailing List (openstack-dev at lists.openstack.org)
Subject: [openstack-dev] [Quantum][LBaaS] LBaaS extension for python-quantumclient

Hi!

Please review the draft specification on LBaaS CLI (http://wiki.openstack.org/Quantum/LBaaS/CLI). It is based on service REST API, all new commands start with "lb_' prefix.

Thanks,
Ilya
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20121114/71d2a7f6/attachment-0001.html>