Open Stack

Thanks for comments Youcef!

I've updated document with proposed changes. "tenant-id" attribute is added
for all creation commands. Also I've noticed that there is "description"
attribute in REST API pool update command, so I've added it for both
"lb-pool-update" and "lb-pool-create" commands.

As of keystone interaction - the client already supports both token and
env. variables, so there is nothing to change here.

Thanks,
Ilya

2012/11/14 Youcef Laribi <Youcef.Laribi at eu.citrix.com>

> 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****
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20121115/fc07c621/attachment.html>