Set port state based on state from OVN

Bug #1457571 reported by Russell Bryant
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
networking-ovn
Fix Released
Critical
Numan Siddique

Bug Description

Set Neutron port state when OVN logical port 'up' state changes to true so that Nova will get notified that the port is ready. Right now ports are marked as up immediately on port binding instead of paying attention to state from OVN.

Revision history for this message
Russell Bryant (russellb) wrote :
Changed in networking-ovn:
status: New → In Progress
Changed in networking-ovn:
importance: Undecided → High
Revision history for this message
Russell Bryant (russellb) wrote :

The patch is stalled, so I'm setting this back to confirmed.

Changed in networking-ovn:
status: In Progress → Confirmed
importance: High → Critical
Changed in networking-ovn:
assignee: nobody → Numan Siddique (numansiddique)
Changed in networking-ovn:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to networking-ovn (master)

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

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

Change abandoned by Numan Siddique (<email address hidden>) on branch: master
Review: https://review.openstack.org/261306
Reason: @garyk - Thanks for the review.

I have addressed your comments in the original patch.

Abandoning this patch. I thought the original patch is abandoned, but its not.
Hence updated that.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to networking-ovn (master)

Reviewed: https://review.openstack.org/178826
Committed: https://git.openstack.org/cgit/openstack/networking-ovn/commit/?id=9c4560dc9932c9d9dddb4fe70a0b59eb6cec219a
Submitter: Jenkins
Branch: master

commit 9c4560dc9932c9d9dddb4fe70a0b59eb6cec219a
Author: Terry Wilson <email address hidden>
Date: Thu Dec 24 17:57:24 2015 +0530

    Add port 'up' and 'down' notification

    This adds the ability to watch for OVSDB events and take action
    when they match specific conditions.

    This patch handles the Logical_Port.up updates
     - At startup, it catches the initial 'create' events from
       ovsdb server and updates the port status if not in sync.
     - If the Logical_Port.up changes from False to True
       it sets the port status to 'ACTIVE' so that neutron server
       can send 'network-vif-plugged' event to nova.
     - If the Logical_Port.up changes from True to False
       it sets the port status to 'DOWN' so that neutron server
       can send 'network-vif-unplugged' event to nova.

    To ensure that only one neutron-server handles the OVSDB events
    it uses the ovsdb lock.

    When the port is created, it sets the status to 'DOWN' by default.

    Co-Authored-By: Numan Siddique <email address hidden>

    Change-Id: Ib7e1ebbbbd430a46aead2402e8b7543b5d73806e
    Closes-bug: #1457571

Changed in networking-ovn:
status: In Progress → Fix Released
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.