l2population mechanism driver should not prevent creation of a vlan transparent network

Bug #1611920 reported by Erik Colnick
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Erik Colnick

Bug Description

The l2population mechanism driver does not override the MechanismDriver base class check_vlan_transparent method. This results in the inability to create vlan-transparent networks when the l2populiation mechanism driver is included in the mechanism_drivers configuration list. The l2population mechanism driver has no impact on vlan transparency and thus it should not prevent the creation of a
vlan-transparent network.

Tags: l2-pop
Changed in neutron:
status: New → Confirmed
importance: Undecided → Medium
assignee: nobody → Erik Colnick (erikcolnick)
tags: added: l2-pop
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/353704

Changed in neutron:
status: Confirmed → In Progress
summary: - l2population mechanism driver should report vlan transparent support
+ l2population mechanism driver should not prevent creation of a vlan
+ transparent network
description: updated
Changed in neutron:
assignee: Erik Colnick (erikcolnick) → Armando Migliaccio (armando-migliaccio)
Changed in neutron:
assignee: Armando Migliaccio (armando-migliaccio) → Erik Colnick (erikcolnick)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/353704
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=676770ba759c9ab41461b337847d1b47e2319fae
Submitter: Jenkins
Branch: master

commit 676770ba759c9ab41461b337847d1b47e2319fae
Author: Erik Colnick <email address hidden>
Date: Wed Aug 10 06:58:59 2016 -0600

    Implement check_vlan_transparency to return True in L2population mech driver

    As the L2population mechanism driver does not impact VLAN transparency
    support, this patch overrides the check_vlan_transparency method from
    the MechanismDriver base class and returns 'True', In this way,
    inclusion of the L2population mechanism driver in the list of configured
    mechanism_drivers does not prevent the creation of VLAN transparent
    networks.

    Change-Id: I8b26e899180544e2ad49ef470b9624015fd4d5de
    Closes-Bug: #1611920

Changed in neutron:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 9.0.0.0b3

This issue was fixed in the openstack/neutron 9.0.0.0b3 development milestone.

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.