On Wed, 6 Mar 2013 13:15:31 +0900 Isaku Yamahata <yamah...@valinux.co.jp> wrote:
> On Wed, Mar 06, 2013 at 11:54:00AM +0900, OHMURA Kei wrote: >> This gets a netflow packet and sends it to the other >> applications. Each application can use this to get netflow >> messages as an event. >> >> Signed-off-by: OHMURA Kei <ohmura....@lab.ntt.co.jp> >> --- >> ryu/flags.py | 8 +++++- >> ryu/lib/xflow/netflow_collector.py | 54 >> ++++++++++++++++++++++++++++++++++++ >> 2 files changed, 61 insertions(+), 1 deletion(-) >> create mode 100644 ryu/lib/xflow/netflow_collector.py >> >> diff --git a/ryu/flags.py b/ryu/flags.py >> index 533f7de..4a0a2d7 100644 >> --- a/ryu/flags.py >> +++ b/ryu/flags.py >> @@ -40,5 +40,11 @@ CONF.register_cli_opts([ >> 'context'), >> cfg.StrOpt('quantum-controller-addr', default=None, >> help='openflow method:address:port to set controller of' >> - 'ovs bridge') >> + 'ovs bridge'), >> + >> + # lib/xflow/netflow_collector >> + cfg.StrOpt('netflow-listen-host', default='', >> + help='netflow listen host'), >> + cfg.IntOpt('netflow-listen-port', default=2055, >> + help='netflow listen port') >> ]) > > Now this can be in ryu/lib/xflow/netflow_collector.py. > openstack-derived cfg is capable of reevaluating options when loading module. Yeah, please don't add new stuff to the file. I think that we should move the options in the current flags.py to the quantum code. ------------------------------------------------------------------------------ Symantec Endpoint Protection 12 positioned as A LEADER in The Forrester Wave(TM): Endpoint Security, Q1 2013 and "remains a good choice" in the endpoint security space. For insight on selecting the right partner to tackle endpoint security challenges, access the full report. http://p.sf.net/sfu/symantec-dev2dev _______________________________________________ Ryu-devel mailing list Ryu-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ryu-devel