LBaaS: haproxy driver should respect vip/pool admin state

Bug #1297142 reported by Oleg Bondarev
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Oleg Bondarev

Bug Description

On vip/pool update when admin_state_up becomes False haproxy driver should reflect it in the config.

Tags: lbaas
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

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

Changed in neutron:
status: New → In Progress
summary: - LBaaS: haproxy driver doesn't undeploy loadbalancer when needed
+ LBaaS: haproxy driver should respect vip/pool admin state
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron-lbaas (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (master)

Change abandoned by Oleg Bondarev (<email address hidden>) on branch: master
Review: https://review.openstack.org/82749
Reason: Was proposed for neutron-lbaas: https://review.openstack.org/#/c/142471/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron-lbaas (master)

Reviewed: https://review.openstack.org/142471
Committed: https://git.openstack.org/cgit/openstack/neutron-lbaas/commit/?id=a627100d510d58d558982d2d5f5bebbe35634b18
Submitter: Jenkins
Branch: master

commit a627100d510d58d558982d2d5f5bebbe35634b18
Author: Oleg Bondarev <email address hidden>
Date: Wed Dec 17 19:26:51 2014 +0400

    Haproxy driver should respect vip/pool admin state

    On vip/pool update when admin_state_up becomes False
    haproxy driver should reflect it in the config.
    Currently there may be only one vip in the config and
    if it is disabled, haproxy process fails to restart with
    "[ALERT] 084/045122 (11407) : [haproxy.main()] No enabled
    listener found (check the <listen> keywords) ! Exiting.",
    and continues running and balancing with old config -
    so for this case we need to undeploy loadbalancer.
    The patch also moves namespace deletion to delete_pool()
    as there is no need to delete/recreate namespace each time
    the vip is removed/added or disabled/enabled

    Closes-Bug: #1155092
    Closes-Bug: #1297142
    Change-Id: I11e2bd3185328ba47ba1aaede932e3114263bed8

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

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