ML2 hierarchical port binding needs additional tests

Bug #1427291 reported by Robert Kukura
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Expired
Medium
Unassigned

Bug Description

Although the current unit tests cover the hierarchical port binding code reasonably well, additional tests are needed that verify the following:

* Binding loops are properly avoided
* Binding limit is detected
* Dead-end binding attempts are handled properly

Robert Kukura (rkukura)
Changed in neutron:
milestone: kilo-3 → kilo-rc1
Kyle Mestery (mestery)
Changed in neutron:
milestone: kilo-rc1 → liberty-1
Thierry Carrez (ttx)
Changed in neutron:
milestone: liberty-1 → liberty-2
Thierry Carrez (ttx)
Changed in neutron:
milestone: liberty-2 → liberty-3
Changed in neutron:
milestone: liberty-3 → none
Revision history for this message
Federico Ressi (federico-ressi-9) wrote :

This bug has been here for a long time. What is its state? Is still valid? Specifications doesn't look to clear to me.

* Binding loops are properly avoided
What do you mean with this? How can this happen?

* Binding limit is detected
What do you mean with this? How can this happen?

* Dead-end binding attempts are handled properly
What do you mean with this? How can this happen?

Revision history for this message
Federico Ressi (federico-ressi-9) wrote :

@Robert Kukura: you should update this, fix this or unassigne yourself from this to allow others taking it.

Changed in neutron:
assignee: Robert Kukura (rkukura) → Federico Ressi (federico-ressi-9)
status: New → In Progress
Revision history for this message
Federico Ressi (federico-ressi-9) wrote :

I have submitted a fix for this bug:https://review.openstack.org/#/c/289863/
Looking for reviewers.

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

Change abandoned by Federico Ressi (<email address hidden>) on branch: master
Review: https://review.openstack.org/289863
Reason: The effort of taking it to be accepted doesn't pay the amount of change I understood have been asked for.

Changed in neutron:
assignee: Federico Ressi (federico-ressi-9) → nobody
Sergey Belous (sbelous)
Changed in neutron:
assignee: nobody → Sergey Belous (sbelous)
Sergey Belous (sbelous)
Changed in neutron:
assignee: Sergey Belous (sbelous) → nobody
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

This bug is > 180 days without activity. We are unsetting assignee and milestone and setting status to Incomplete in order to allow its expiry in 60 days.

If the bug is still valid, then update the bug status.

Changed in neutron:
status: In Progress → Incomplete
tags: added: low-hanging-fruit
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for neutron because there has been no activity for 60 days.]

Changed in neutron:
status: Incomplete → Expired
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.