neutron

neutron-openvswitch-agent: Trace on SIGTERM with of_interface=native

Bug #1589746 reported by Inessa Vasilevskaya
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Expired
Low
Unassigned

Bug Description

Configuration: devstack, of_interface = native.

Steps to reproduce:
kill -TERM PID

The following will be seen in the logs:

2016-06-07 01:05:37.088 ERROR ryu.lib.hub [-] hub: uncaught exception: Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/ryu/lib/hub.py", line 54, in _launch
    return func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/ryu/base/app_manager.py", line 547, in close
    self.uninstantiate(app_name)
  File "/usr/local/lib/python2.7/dist-packages/ryu/base/app_manager.py", line 528, in uninstantiate
    app = self.applications.pop(name)
KeyError: 'ofctl_service'

Inessa Vasilevskaya (ivasilevskaya)
Changed in neutron:
assignee: nobody → Inessa Vasilevskaya (ivasilevskaya)
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/326437

Changed in neutron:
status: New → In Progress
Kevin Benton (kevinbenton)
Changed in neutron:
importance: Undecided → Low
Revision history for this message
YAMAMOTO Takashi (yamamoto) wrote :

fixed in ryu: https://github.com/osrg/ryu/commit/b0ab4f16028c452374c5a0f22bd970038194f142

Revision history for this message
YAMAMOTO Takashi (yamamoto) wrote :

requirements change: https://review.openstack.org/#/c/336799/

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

Change abandoned by Inessa Vasilevskaya (<email address hidden>) on branch: master
Review: https://review.openstack.org/326437
Reason: Fixed in ryu

Revision history for this message
Alan Pevec (apevec) wrote :

ryu version update was reverted in https://review.openstack.org/341057 so this is still open?

Revision history for this message
Inessa Vasilevskaya (ivasilevskaya) wrote :

Hm, I guess so. I'll revive my patch, though imho solving this in ryu is a proper solution.

Revision history for this message
IWAMOTO Toshihiro (iwamoto) wrote :

ryu 4.5 is in global requirements. I think it's fixed now.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Armando Migliaccio (<email address hidden>) on branch: master
Review: https://review.openstack.org/326437
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Armando Migliaccio (armando-migliaccio)
Changed in neutron:
status: In Progress → Incomplete
assignee: Inessa Vasilevskaya (ivasilevskaya) → nobody
Revision history for this message
Jakub Libosvar (libosvar) wrote :

Shouldn't we limit ryu requirement to some minimal version like mentioned 4.5 then?

Revision history for this message
IWAMOTO Toshihiro (iwamoto) wrote :

Maybe we can require ryu>=4.7 and remove a ryu patch in ovs_ryuapp.py altogether.

Revision history for this message
IWAMOTO Toshihiro (iwamoto) wrote :

requirments now has ryu>=4.9, so this bug should be closed.

Revision history for this message
Filippo DiNoto (fdinoto) wrote :

In the RDO newton release currently when you install the Neutron packages, it installs python2-ryu-4.3-2.el7.noarch

Any advice on how to deal with this since the solution to the bug is to require a higher version?

I'm trying set up a BGP speaker and so far when I configure neutron to load the Ryu BGP driver per the docs neutron will not load and errors that it doesn't know of that driver. Interestingly the RH OpenStack docs say the Ryu driver is deprecated and to use an SDN module instead but offers no examples. Although I think I am misunderstanding the RH docs so I'm sort of ignoring them for now.

Revision history for this message
IWAMOTO Toshihiro (iwamoto) wrote :

Hi Filippo,

1. Most likely you can ignore this bug and use ryu-4.3, as this bug is about extra error messages on agent shutdown and doesn't affect its functionality.
2. I guess the RDO document is talking about the old Ryu plugin, which is dead long ago and doesn't ship with Newton.

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.
Subscribing...

Other bug subscribers

Remote bug watches

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