External load balancer support is broken

Bug #1573583 reported by Aleksandr Didenko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Aleksandr Didenko
Mitaka
Fix Released
High
Aleksandr Didenko
Newton
Fix Released
High
Aleksandr Didenko

Bug Description

External LB and moving controllers into different racks is currently broken.

Steps to reproduce:
1) Build and install https://github.com/openstack/fuel-plugin-external-lb
2) Create multirack env with controllers in different racks
3) Deploy

Expected result:
deployment successful

Actual result:
deployment failed

Version info:
reproduced on fuel-9.0-191-2016-04-12_02-00-00.iso

Revision history for this message
Aleksandr Didenko (adidenko) wrote :

It's by this change:

https://review.openstack.org/#/c/224857/71/deployment/puppet/osnailyfacter/modular/virtual_ips/conntrackd.pp

The following conditional is missing after that patch:
if $network_metadata['vips']["vrouter_${vrouter_name}"]['namespace'] {

So we need to put it back in order to fix this bug and also add a noop tests to avoid such regressions in the future.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/309449
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=e0ef70d91a98225921b1111c76ad61559d5a1ba2
Submitter: Jenkins
Branch: master

commit e0ef70d91a98225921b1111c76ad61559d5a1ba2
Author: Aleksandr Didenko <email address hidden>
Date: Fri Apr 22 16:48:38 2016 +0200

    Fix conntrackd for external LB use case

    Do not configure conttrackd resource for non-existing VIP.

    Change-Id: Iee314c991aaeab84718d083b273090422ffc4ae0
    Closes-bug: #1573583

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/310132

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/mitaka)

Reviewed: https://review.openstack.org/310132
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=ae67dc251e3b23b2a77a37dda0c2852a1a76f08e
Submitter: Jenkins
Branch: stable/mitaka

commit ae67dc251e3b23b2a77a37dda0c2852a1a76f08e
Author: Aleksandr Didenko <email address hidden>
Date: Fri Apr 22 16:48:38 2016 +0200

    Fix conntrackd for external LB use case

    Do not configure conttrackd resource for non-existing VIP.

    Change-Id: Iee314c991aaeab84718d083b273090422ffc4ae0
    Closes-bug: #1573583
    (cherry picked from commit e0ef70d91a98225921b1111c76ad61559d5a1ba2)

Revision history for this message
Nastya Urlapova (aurlapova) wrote :
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.