ODL ML2 driver doesn't notify active/inactive ports

Bug #1301449 reported by Simon Pasquier on 2014-04-02
24
This bug affects 5 people
Affects Status Importance Assigned to Milestone
neutron
High
Robert Kukura

Bug Description

The nova-event-callback blueprint [1] implemented the notifications of active/down ports to Nova. Before effectively starting an instance, Nova compute waits for a "VIF plugged" notification from Neutron.

I'm running ODL ML2 driver in a devstack using the master branch and I notice that the ODL driver doesn't notify back the Nova API. Hence with the default settings, the VM creation always fails.

As a workaround, set the following parameters in your nova.conf:
vif_plugging_timeout = 10
vif_plugging_is_fatal = False

With this configuration, I'm able to boot and connect to the instances but the Neutron ports are always reported as DOWN [2].

[1] https://blueprints.launchpad.net/neutron/+spec/nova-event-callback
[2] http://paste.openstack.org/show/74861/

summary: - ODL ML2 driver doesn't notify active ports
+ ODL ML2 driver doesn't notify active/inactive ports
description: updated
Kyle Mestery (mestery) on 2014-04-02
Changed in neutron:
assignee: nobody → Kyle Mestery (mestery)
importance: Undecided → High
Akihiro Motoki (amotoki) on 2014-04-02
tags: added: icehouse-rc-potential opendaylight
tags: added: ml2
Simon Pasquier (simon-pasquier) wrote :

I pasted my configuration files:
- neutron.conf => http://paste.openstack.org/show/74866/
- ml2_conf.ini => http://paste.openstack.org/show/74868/
- nova.conf => http://paste.openstack.org/show/74867/

Kyle Mestery (mestery) wrote :

Per discussion with @amotoki in #openstack-neutron, I think we just need to set the port status to ACTIVE initially. I'll verify this and propose a patch soon.

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

Changed in neutron:
assignee: Kyle Mestery (mestery) → Robert Kukura (rkukura)
status: New → In Progress

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

commit a9e3074aa9f442c2fff1ba058ac8ed585c6caa24
Author: Robert Kukura <email address hidden>
Date: Thu Apr 3 17:01:00 2014 -0400

    ML2: ODL driver sets port status

    The OpenDaylight mechanism driver does not depend on an L2 agent to
    plug the port. Now that nova waits for notification that the port
    status is ACTIVE, the ML2 driver API is extended so that the mechanism
    driver that binds a port can optionally set the port status, and the
    OpenDaylight mechanism driver uses this to set the port status to
    ACTIVE.

    Closes-Bug: 1301449
    Change-Id: I171c405f36b4f2354d9585e8ae3dfa50ddaa9a7e

Changed in neutron:
status: In Progress → Fix Committed
Changed in neutron:
milestone: none → juno-1
tags: removed: icehouse-rc-potential
Changed in neutron:
milestone: juno-1 → icehouse-rc2
no longer affects: neutron/icehouse

Reviewed: https://review.openstack.org/85511
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=01380b7c67ba2ef1f923d0cf4265dd2d20e31093
Submitter: Jenkins
Branch: milestone-proposed

commit 01380b7c67ba2ef1f923d0cf4265dd2d20e31093
Author: Robert Kukura <email address hidden>
Date: Thu Apr 3 17:01:00 2014 -0400

    ML2: ODL driver sets port status

    The OpenDaylight mechanism driver does not depend on an L2 agent to
    plug the port. Now that nova waits for notification that the port
    status is ACTIVE, the ML2 driver API is extended so that the mechanism
    driver that binds a port can optionally set the port status, and the
    OpenDaylight mechanism driver uses this to set the port status to
    ACTIVE.

    Closes-Bug: 1301449
    Change-Id: I171c405f36b4f2354d9585e8ae3dfa50ddaa9a7e
    (cherry picked from commit a9e3074aa9f442c2fff1ba058ac8ed585c6caa24)

Changed in neutron:
status: Fix Committed → Fix Released
Simon Pasquier (simon-pasquier) wrote :

I've verified with a fresh devstack that the change fixes the problem.

Thierry Carrez (ttx) on 2014-04-17
Changed in neutron:
milestone: icehouse-rc2 → 2014.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers