Loadbalancing: Missing validation in Vip creation

Bug #1123114 reported by Avishay Balderman
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Ryan O'Hara

Bug Description

Both Vip and Pool has a field named protocol. Valid values for this field are : "HTTP", "HTTPS", "TCP"
The code needs to validate the combinations of Vip(protocol) and Pool(protocol)
Valid combinations are:
1) Vip(TCP) -- Pool(TCP)
2) Vip(HTTP) -- Pool(HTTP)
3) Vip(HTTPS) -- Pool(HTTPS)

Tags: lbaas
Revision history for this message
Avishay Balderman (avishayb) wrote :

For cases #1 and #3 Stickiness can be "SOURCE_IP" only.

tags: added: lbaas
Changed in quantum:
importance: Undecided → Low
Ryan O'Hara (rohara)
Changed in quantum:
assignee: nobody → Ryan O'Hara (rohara)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to quantum (master)

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

Revision history for this message
Ryan O'Hara (rohara) wrote :

It was not obvious to me how to set session persistence when creating a VIP, so I'm not sure how to validate "stickyness" based on protocol. If I'm missing something, please advise.

Changed in quantum:
milestone: none → havana-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to quantum (master)

Reviewed: https://review.openstack.org/24224
Committed: http://github.com/openstack/quantum/commit/9a15b11794dbf945f0e9b9705ae510d3b378287d
Submitter: Jenkins
Branch: master

commit 9a15b11794dbf945f0e9b9705ae510d3b378287d
Author: Ryan O'Hara <email address hidden>
Date: Tue Mar 12 13:01:23 2013 -0500

    Validate protocol when creating VIP.

    Fixes: bug #1123114

    When creating a VIP, check that the protocol matches the protocol
    for the associated pool. If not, raise an exception.

    Change-Id: Iba318eda935ccc89dbe8244e00f36ebdfcce65e0

Changed in quantum:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in quantum:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: havana-1 → 2013.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.