Hi, Corey
On 2016年02月13日 07:17, Corey Bryant wrote:
> Hi,
>
> Has anyone come across this test failure? I'm seeing it on an i386 run.
>
> ======================================================================
> FAIL: test_parser (ryu.tests.unit.ofproto.test_ofproto_parser.TestMsgBase)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
> File "/«PKGBUILDDIR»/ryu/tests/unit/ofproto/test_ofproto_parser.py", line
> 204, in test_parser
> ok_(self._test_parser())
> File "/«PKGBUILDDIR»/ryu/tests/unit/ofproto/test_ofproto_parser.py", line
> 199, in _test_parser
> eq_(hex(xid), check['xid'])
> AssertionError: '0x822c6866L' != '2183948390'
Thank you for your report.
I think it maybe a bug in Ryu.
The value of xid(2183948390) in the above test is larger than 2147483647.
(With my understanding, 2147483647(=2^31-1) is the maximum positive value on
32bit plat-form.)
So, the insinstance(x, int) returns "False".
If it is possible, please try the following steps to resolve the issue.
1) Correct ryu/ofproto/ofproto_parser.py as follows
def __str__(self):
def hexify(x):
+ return hex(x) if x is not None else x
- return hex(x) if isinstance(x, int) else x
buf = 'version=%s,msg_type=%s,msg_len=%s,xid=%s,' %\
(hexify(self.version), hexify(self.msg_type),
hexify(self.msg_len), hexify(self.xid))
return buf + StringifyMixin.__str__(self)
2) Reinstall Ryu
3) Execute 'run_test.sh' script
thanks,
>
>
> --
> Regards,
> Corey
>
>
> ------------------------------------------------------------------------------
> Site24x7 APM Insight: Get Deep Visibility into Application Performance
> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
> Monitor end-to-end web transactions and take corrective actions now
> Troubleshoot faster and improve end-user experience. Signup Now!
> http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
>
>
>
> _______________________________________________
> Ryu-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/ryu-devel
>
------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
_______________________________________________
Ryu-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ryu-devel
