yahoo-eng-team team mailing list archive

Thread
Date
[Bug 1582668] [NEW] healthmonitor-id is not an optional argument for neutron lbaas-pool-create

To: yahoo-eng-team@xxxxxxxxxxxxxxxxxxx
From: huanghuijin <huang.huijin@xxxxxxxxxx>
Date: Tue, 17 May 2016 11:33:18 -0000
Reply-to: Bug 1582668 <1582668@xxxxxxxxxxxxxxxxxx>
Sender: bounces@xxxxxxxxxxxxx
Public bug reported:

It is in kilo branch.

>From the help information of lbaas-pool-create,  --healthmonitor-id is
an optional argument, but actually it is not allowed to use, so should
it be removed from the help information?

Details:
1.From the following help information, --healthmonitor-id is an optional argument;
[root@opencos2 ~(keystone_admin)]# neutron lbaas-pool-create -h
usage: neutron lbaas-pool-create [-h] [-f {shell,table,value}] [-c COLUMN]
                                 [--max-width <integer>] [--prefix PREFIX]
                                 [--request-format {json,xml}]
                                 [--tenant-id TENANT_ID] [--admin-state-down]
                                 [--description DESCRIPTION]
                                 [--healthmonitor-id HEALTHMONITOR_ID]
                                 [--session-persistence TYPE:VALUE]
                                 --lb-algorithm
                                 {ROUND_ROBIN,LEAST_CONNECTIONS,SOURCE_IP}
                                 --listener LISTENER --protocol
                                 {HTTP,HTTPS,TCP}
                                 NAME

LBaaS v2 Create a pool.

positional arguments:
  NAME                  The name of the pool.

optional arguments:
  -h, --help            show this help message and exit
  --request-format {json,xml}
                        The XML or JSON request format.
  --tenant-id TENANT_ID
                        The owner tenant ID.
  --admin-state-down    Set admin state up to false.
  --description DESCRIPTION
                        Description of the pool.
  --healthmonitor-id HEALTHMONITOR_ID
                        ID of the health monitor to use.
  --session-persistence TYPE:VALUE
                        The type of session persistence to use.
  --lb-algorithm {ROUND_ROBIN,LEAST_CONNECTIONS,SOURCE_IP}
                        The algorithm used to distribute load between the
                        members of the pool.
  --listener LISTENER   The listener to associate with the pool
  --protocol {HTTP,HTTPS,TCP}
                        Protocol for balancing.

output formatters:
  output formatter options

  -f {shell,table,value}, --format {shell,table,value}
                        the output format, defaults to table
  -c COLUMN, --column COLUMN
                        specify the column(s) to include, can be repeated

table formatter:
  --max-width <integer>
                        Maximum display width, 0 to disable

shell formatter:
  a format a UNIX shell can parse (variable="value")

  --prefix PREFIX       add a prefix to all variable names

2. If I add the --healthmonitor-id, lbaas-pool-create fails.
[root@opencos2 ~(keystone_admin)]# neutron lbaas-pool-create --healthmonitor-id 3aeac49b-11a2-4d38-801a-33f5befa4c52 --lb-algorithm ROUND_ROBIN --listener 981a81b5-6b33-4a9d-a683-f64bf9904eea --protocol HTTP lbpool
Attribute 'healthmonitor_id' not allowed in POST

3. After removing the --healthmonitor-id, it creates successfully.
[root@opencos2 ~(keystone_admin)]# neutron lbaas-pool-create  --lb-algorithm ROUND_ROBIN --listener 981a81b5-6b33-4a9d-a683-f64bf9904eea --protocol HTTP lbpool
Created a new pool:
+---------------------+------------------------------------------------+
| Field               | Value                                          |
+---------------------+------------------------------------------------+
| admin_state_up      | True                                           |
| description         |                                                |
| healthmonitor_id    |                                                |
| id                  | a3582715-53af-42e2-8867-1f9608aafbea           |
| lb_algorithm        | ROUND_ROBIN                                    |
| listeners           | {"id": "981a81b5-6b33-4a9d-a683-f64bf9904eea"} |
| members             |                                                |
| name                | lbpool                                         |
| protocol            | HTTP                                           |
| session_persistence |                                                |
| tenant_id           | be58eaec789d44f296a65f96b944a9f5               |
+---------------------+------------------------------------------------+

** Affects: neutron
     Importance: Undecided
         Status: New


** Tags: lbaas

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1582668

Title:
  healthmonitor-id is not an optional argument for neutron lbaas-pool-
  create

Status in neutron:
  New

Bug description:
  It is in kilo branch.

  From the help information of lbaas-pool-create,  --healthmonitor-id is
  an optional argument, but actually it is not allowed to use, so should
  it be removed from the help information?

  Details:
  1.From the following help information, --healthmonitor-id is an optional argument;
  [root@opencos2 ~(keystone_admin)]# neutron lbaas-pool-create -h
  usage: neutron lbaas-pool-create [-h] [-f {shell,table,value}] [-c COLUMN]
                                   [--max-width <integer>] [--prefix PREFIX]
                                   [--request-format {json,xml}]
                                   [--tenant-id TENANT_ID] [--admin-state-down]
                                   [--description DESCRIPTION]
                                   [--healthmonitor-id HEALTHMONITOR_ID]
                                   [--session-persistence TYPE:VALUE]
                                   --lb-algorithm
                                   {ROUND_ROBIN,LEAST_CONNECTIONS,SOURCE_IP}
                                   --listener LISTENER --protocol
                                   {HTTP,HTTPS,TCP}
                                   NAME

  LBaaS v2 Create a pool.

  positional arguments:
    NAME                  The name of the pool.

  optional arguments:
    -h, --help            show this help message and exit
    --request-format {json,xml}
                          The XML or JSON request format.
    --tenant-id TENANT_ID
                          The owner tenant ID.
    --admin-state-down    Set admin state up to false.
    --description DESCRIPTION
                          Description of the pool.
    --healthmonitor-id HEALTHMONITOR_ID
                          ID of the health monitor to use.
    --session-persistence TYPE:VALUE
                          The type of session persistence to use.
    --lb-algorithm {ROUND_ROBIN,LEAST_CONNECTIONS,SOURCE_IP}
                          The algorithm used to distribute load between the
                          members of the pool.
    --listener LISTENER   The listener to associate with the pool
    --protocol {HTTP,HTTPS,TCP}
                          Protocol for balancing.

  output formatters:
    output formatter options

    -f {shell,table,value}, --format {shell,table,value}
                          the output format, defaults to table
    -c COLUMN, --column COLUMN
                          specify the column(s) to include, can be repeated

  table formatter:
    --max-width <integer>
                          Maximum display width, 0 to disable

  shell formatter:
    a format a UNIX shell can parse (variable="value")

    --prefix PREFIX       add a prefix to all variable names

  2. If I add the --healthmonitor-id, lbaas-pool-create fails.
  [root@opencos2 ~(keystone_admin)]# neutron lbaas-pool-create --healthmonitor-id 3aeac49b-11a2-4d38-801a-33f5befa4c52 --lb-algorithm ROUND_ROBIN --listener 981a81b5-6b33-4a9d-a683-f64bf9904eea --protocol HTTP lbpool
  Attribute 'healthmonitor_id' not allowed in POST

  3. After removing the --healthmonitor-id, it creates successfully.
  [root@opencos2 ~(keystone_admin)]# neutron lbaas-pool-create  --lb-algorithm ROUND_ROBIN --listener 981a81b5-6b33-4a9d-a683-f64bf9904eea --protocol HTTP lbpool
  Created a new pool:
  +---------------------+------------------------------------------------+
  | Field               | Value                                          |
  +---------------------+------------------------------------------------+
  | admin_state_up      | True                                           |
  | description         |                                                |
  | healthmonitor_id    |                                                |
  | id                  | a3582715-53af-42e2-8867-1f9608aafbea           |
  | lb_algorithm        | ROUND_ROBIN                                    |
  | listeners           | {"id": "981a81b5-6b33-4a9d-a683-f64bf9904eea"} |
  | members             |                                                |
  | name                | lbpool                                         |
  | protocol            | HTTP                                           |
  | session_persistence |                                                |
  | tenant_id           | be58eaec789d44f296a65f96b944a9f5               |
  +---------------------+------------------------------------------------+

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1582668/+subscriptions
Follow ups

[Bug 1582668] Re: healthmonitor-id is not an optional argument for neutron lbaas-pool-create
From: Hirofumi Ichihara, 2016-05-20