RE: Review Request: Fix bug: unittest NetUtilsTest failed sometimes

2012-08-12 Thread Mice Xia
Alex, Is it Ok for you to remove this unittest and related caller methods? Regards Mice From: Darren Shepherd [mailto:nore...@reviews.apache.org] On Behalf Of Darren Shepherd Sent: Monday, August 13, 2012 3:12 AM To: Alex Huang Cc: cloudstack; Mice Xia; Darren Shepherd Subject: Re: Review Reques

Re: Review Request: Fix bug: unittest NetUtilsTest failed sometimes

2012-08-12 Thread Darren Shepherd
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6558/#review10181 --- I think this code change might actually inadvertantly introduce a sma

Re: Review Request: Fix bug: unittest NetUtilsTest failed sometimes

2012-08-12 Thread Darren Shepherd
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6558/#review10180 --- It would be slightly cleaner if instead of changing the method signat