neutron

ML2 : network filters for provider attributes don't work

Bug #1333475 reported by Manish Godara
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Manish Godara
neutron 2015.1.0 "kilo"

Bug Description

If you filter networks based on any of the provider-network attributes then the output doesn't seem to honor them. Other filters work fine.

https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/plugin.py#L207

TBD line item.

See original description
Manish Godara (manishatyhoo)
Changed in neutron:
assignee: nobody → Manish Godara (manishatyhoo)
yong sheng gong (gongysh)
Changed in neutron:
importance: Undecided → Low
milestone: none → juno-2
status: New → Confirmed
Kyle Mestery (mestery)
Changed in neutron:
milestone: juno-2 → juno-3
Thierry Carrez (ttx)
Changed in neutron:
milestone: juno-3 → juno-rc1
Revision history for this message
Shiv Haris (shh) wrote :

Manish: Any update?

Revision history for this message
Kyle Mestery (mestery) wrote :

Moving low priority bug out of Juno-RC1.

Changed in neutron:
milestone: juno-rc1 → kilo-1
Revision history for this message
Manish Godara (manishatyhoo) wrote :

@shh checkin in patch shortly.

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/124917

Changed in neutron:
status: Confirmed → In Progress
Manish Godara (manishatyhoo)
summary: - ML2 : network filters for provider attributes not implemented
+ ML2 : network filters for provider attributes don't work
description: updated
Kyle Mestery (mestery)
Changed in neutron:
milestone: kilo-1 → kilo-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (master)

Reviewed: https://review.openstack.org/149088
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=9213a5a9f89f9b0833e9d0dc088aaf3ce99a4d8f
Submitter: Jenkins
Branch: master

commit 9213a5a9f89f9b0833e9d0dc088aaf3ce99a4d8f
Author: Manish Godara <email address hidden>
Date: Wed Jan 21 15:49:43 2015 -0800

    Refactor ml2 manager

    Refactor code to facilitate other changes. Keeping the
    refactored code in different patch for ease of review.
    This patch mainly consolidates the keys that are referenced
    together most of the time in reference to providernet
    extension. By doing this, the subsequent changes are less
    repetitive and makes the code little cleaner as well.

    Change-Id: Idc0648d5c4688c8f797cc5427b71c2a3919ce722
    Related-Bug: #1333475

Kyle Mestery (mestery)
Changed in neutron:
milestone: kilo-2 → kilo-3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

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

commit d7d2f1daedbafa7952185d14a0df584c03a6cf06
Author: Manish Godara <email address hidden>
Date: Wed Jan 21 16:24:15 2015 -0800

    Add filter for provider network attributes in ML2

    Added code to implement filters for provider network attributes
    in ML2 plugin. There was a todo item in code but never got done.
    It is needed to lookup networks based on provider network attributes.

    Change-Id: I9eaa8bd8fa35cfe2b94551f9c27336503f118ce6
    Closes-Bug: 1333475

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

Other bug subscribers

Remote bug watches

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