l3 rpc handler not checking for UNBOUND with host_id

Bug #1648879 reported by Kevin Benton
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
Kevin Benton

Bug Description

If a binding fails to commit after a host_id is set on a port, the port will have a host_id with a vif type of 'unbound'. In this case the L3 RPC handler should be trying to update the port again as it would when the binding had changed the vif type to binding_failed. However, it currently doesn't due to a bad conditional[1].

1. https://github.com/openstack/neutron/blob/5254f0ab60d46cabdace6d59ce50ca05568a87d5/neutron/api/rpc/handlers/l3_rpc.py#L141-L142

Changed in neutron:
assignee: nobody → Kevin Benton (kevinbenton)
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/409314

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

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

commit a5a8a37344f6d193886ae8d047664aef76920ed8
Author: Kevin Benton <email address hidden>
Date: Fri Dec 9 11:49:09 2016 -0800

    Check for unbound ports in L3 RPC handler

    The handler was making the incorrect assumption that once a
    host_id was set, a port which failed to bind could only be in
    the 'binding_failed' state. So it would not try to rebind ports
    that encountered an exeption during port binding commit that left
    them in the unbound state.

    Change-Id: I28bbeda5fed4275ea38e27308518f89df9ab4eff
    Closes-Bug: #1648879

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

This issue was fixed in the openstack/neutron 10.0.0.0b2 development milestone.

tags: added: neutron-proactive-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/422035

Changed in neutron:
importance: Undecided → High
tags: removed: neutron-proactive-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/mitaka)

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

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

Reviewed: https://review.openstack.org/422037
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=c361055021e5f81142eee1d8aeb0bf786a5692e6
Submitter: Jenkins
Branch: stable/mitaka

commit c361055021e5f81142eee1d8aeb0bf786a5692e6
Author: Kevin Benton <email address hidden>
Date: Fri Dec 9 11:49:09 2016 -0800

    Check for unbound ports in L3 RPC handler

    The handler was making the incorrect assumption that once a
    host_id was set, a port which failed to bind could only be in
    the 'binding_failed' state. So it would not try to rebind ports
    that encountered an exeption during port binding commit that left
    them in the unbound state.

    Change-Id: I28bbeda5fed4275ea38e27308518f89df9ab4eff
    Closes-Bug: #1648879
    (cherry picked from commit a5a8a37344f6d193886ae8d047664aef76920ed8)

tags: added: in-stable-mitaka
tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/newton)

Reviewed: https://review.openstack.org/422035
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=51f9ea667af74bae7dd6c61fd3a9e6f8b2e1b862
Submitter: Jenkins
Branch: stable/newton

commit 51f9ea667af74bae7dd6c61fd3a9e6f8b2e1b862
Author: Kevin Benton <email address hidden>
Date: Fri Dec 9 11:49:09 2016 -0800

    Check for unbound ports in L3 RPC handler

    The handler was making the incorrect assumption that once a
    host_id was set, a port which failed to bind could only be in
    the 'binding_failed' state. So it would not try to rebind ports
    that encountered an exeption during port binding commit that left
    them in the unbound state.

    Change-Id: I28bbeda5fed4275ea38e27308518f89df9ab4eff
    Closes-Bug: #1648879
    (cherry picked from commit a5a8a37344f6d193886ae8d047664aef76920ed8)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 9.2.0

This issue was fixed in the openstack/neutron 9.2.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 8.4.0

This issue was fixed in the openstack/neutron 8.4.0 release.

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.