OpenStack Dashboard (Horizon)

LBaaS VIP creation via horizon requires a mandatory "Connection Limit" argument although it's only optional in the cli command

Bug #1241209 reported by Rami Vaknin on 2013-10-17

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	OpenStack Dashboard (Horizon)	Fix Released	Low	Akihiro Motoki	OpenStack Dashboard (Horizon) 2014.1 "icehouse"
	Havana	Fix Released	Low	Akihiro Motoki	OpenStack Dashboard (Horizon) 2013.2.1

Bug Description

Version
=======
Havana on rhel

Description
===========
The "Connection Limit" argument in the Add VIP dialog should not be mandatory.

Here is the cli command tha performs the same:

$ neutron lb-vip-create
usage: neutron lb-vip-create [-h] [-f {shell,table}] [-c COLUMN]
                             [--variable VARIABLE] [--prefix PREFIX]
                             [--request-format {json,xml}]
                             [--tenant-id TENANT_ID] [--address ADDRESS]
                             [--admin-state-down]
                             [--connection-limit CONNECTION_LIMIT]
                             [--description DESCRIPTION] --name NAME
                             --protocol-port PROTOCOL_PORT --protocol PROTOCOL
                             --subnet-id SUBNET_ID
                             pool

Tags:

Revision history for this message

Rami Vaknin (rvaknin) wrote on 2013-10-17:

Screenshot Edit (41.8 KiB, image/png)

Akihiro Motoki (amotoki) on 2013-10-18

Changed in horizon:
status:	New → Confirmed
importance:	Undecided → Low
milestone:	none → icehouse-1
tags:	added: havana-backport-potential

Akihiro Motoki (amotoki) on 2013-10-18

tags:	added: neutron
tags:	added: lbaas

Akihiro Motoki (amotoki) on 2013-10-18

Changed in horizon:
assignee:	nobody → Akihiro Motoki (amotoki)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-10-20: Fix proposed to horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/52842

Changed in horizon:
status:	Confirmed → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-10-24: Fix merged to horizon (master)

Reviewed: https://review.openstack.org/52842
Committed: http://github.com/openstack/horizon/commit/c9f4e27aa5c60bc997e70b4982856bbf4a23372b
Submitter: Jenkins
Branch: master

commit c9f4e27aa5c60bc997e70b4982856bbf4a23372b
Author: Akihiro MOTOKI <email address hidden>
Date: Fri Oct 18 14:05:52 2013 +0900

Fix a bug some optional field in LBaaS are mandatory

* VIP connection_limit
* Pool Member weight

    Closes-Bug: #1241209
    Closes-Bug: #1241004
    Change-Id: I337215938645f6785393925f201567f2143d133e

Changed in horizon:
status:	In Progress → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-11-12: Fix proposed to horizon (stable/havana)

Fix proposed to branch: stable/havana
Review: https://review.openstack.org/56070

Akihiro Motoki (amotoki) on 2013-12-03

tags:

removed: havana-backport-potential

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-12-03: Fix merged to horizon (stable/havana)

Reviewed: https://review.openstack.org/56070
Committed: http://github.com/openstack/horizon/commit/8e003574a071dacf680387639c7f5c71a5a98200
Submitter: Jenkins
Branch: stable/havana

commit 8e003574a071dacf680387639c7f5c71a5a98200
Author: Akihiro MOTOKI <email address hidden>
Date: Fri Oct 18 14:05:52 2013 +0900

Fix a bug some optional field in LBaaS are mandatory

* VIP connection_limit
* Pool Member weight

Closes-Bug: #1241209
Closes-Bug: #1241004

    The change of create_stubs in openstack_dashboard/test/helpers.py
    is cherry-picked from commit fa3a039d0585c8c995504c09657070af4025305f.
    It allows us to create_stubs decorator for sub methods with arguments.

Change-Id: I337215938645f6785393925f201567f2143d133e
(cherry picked from commit c9f4e27aa5c60bc997e70b4982856bbf4a23372b)

Thierry Carrez (ttx) on 2013-12-04