[teststabilization] RFR: 8223856: Replace wildcard address with loopback or local host in tests - part 8

Chris Hegarty chris.hegarty at oracle.com
Mon May 20 09:50:51 UTC 2019


Daniel,

On 16/05/2019 13:23, Daniel Fuchs wrote:
> Hi,
> 
> Please find below a fix for [1]:
> 8223856: Replace wildcard address with loopback or local host in
>           tests - part 8
> 
> http://cr.openjdk.java.net/~dfuchs/webrev_8223856/webrev.00/index.html

Looks good.

> Some of the test failures weren't all attributable to the use
> of the wildcard: in some cases tests make assumptions that
> are not guaranteed to be true - e.g. given two different
> interfaces `i1` and `i2` you may not assume that because you
> can bind to i1 with port `p`, ephemerally allocated, you should
> also be able to bind to `i2` with the same port `p`.
> 
> In such a case I have taken the liberty to add a retry mechanism,
> (that is: try again the whole thing) in the hope
> of decreasing the probability that someone else might
> be again bound to the new ephemeral port on the other interface.

It's unfortunate that we have to do such things, but I agree
with the pragmatic approach. It should improve the reliability
of these few tests.

-Chris.


More information about the net-dev mailing list