openstack/os-ken
Code Issues Proposed changes

app_manager: Stop checking h.callers in get_handlers()

Browse Source 
dynamically registered handlers (vs. ones registered via
set_ev_cls/handler decorators) do not necessarily have h.callers.

Signed-off-by: YAMAMOTO Takashi <yamamoto@valinux.co.jp>
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
This commit is contained in:
YAMAMOTO Takashi 2014-06-16 19:18:53 +09:00 committed by FUJITA Tomonori
parent ba9621627d
commit 4f19fc54cd
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
ryu/base/app_manager.py
View File

@ -226,9 +226,10 @@ class RyuApp(object):
return handlers return handlers
def test(h): def test(h):
if ev_cls not in h.callers: if not hasattr(h, 'callers') or ev_cls not in h.callers:
# this handler does not listen the event. # dynamically registered handlers does not have
return False # h.callers element for the event.
return True
states = h.callers[ev_cls].dispatchers states = h.callers[ev_cls].dispatchers
if not states: if not states:
# empty states means all states # empty states means all states