Yeah, this is a stupid oversight. Adding a fix for this would be fairly simple. I don't have time to get to it right away but the patch would be fairly straightforward:

- add xid to ../include/aggregate-stats-in.hh (set in constructor)
- update coreapps/pyrt/pyrt.cc convert_aggregate_stats_in() to add to python

- in coreapps/pyrt/pycontext.cc:

change send stats_request to create a unique xid and return it
reflect this change through send_aggregate_stats_request

I'll add it on the TODO but will happily accept a patch ;-)
.martin

Hi all,

Has anyone thought about how to access OpenFlow headers via the Python
API? I am writing a component that collects aggregate statistics from
switches. To pair up the request with the response that generated it,
I need the transaction id field from the OpenFlow headers. (Note that
without a way to do this pairing aggregate statistics are pretty much
useless!) Ideally I would be able to do the following:

  xid = send_aggregate_stats_request(...)

  def aggregate_stats_handler(xid,dpid,stats):
     ...

Is something like this is in the works or if there is another way to
achieve the same effect?

Thanks,
Nate

_______________________________________________
nox-dev mailing list
[email protected]
http://noxrepo.org/mailman/listinfo/nox-dev_noxrepo.org


_______________________________________________
nox-dev mailing list
[email protected]
http://noxrepo.org/mailman/listinfo/nox-dev_noxrepo.org

Reply via email to