RFR(S): 8222769: [TESTBUG] TestJFRNetworkEvents should not rely on hostname command

[mikhailo.seledtsov at oracle.com]

Here is the updated webrev: 
http://cr.openjdk.java.net/~mseledtsov/8222769.01/


On 4/25/19 6:38 PM, mikhailo.seledtsov at oracle.com wrote:
> Thank you Leonid,
>
>
> On 4/25/19 6:25 PM, Leonid Mesnik wrote:
>> Hi
>>
>> The overall fix looks food. But I expect that there are might be 
>> different docker environments.
>> Would not be more robust to verify that address exist in getLocalIp() 
>> list and fails if not?
> Sounds good. I will update the code to match the address reported by 
> JFR to all the addresses returned by getLocalIp(). If at least a 
> single match found, test continues. If not, test fails.
>
> Misha
>>
>> Leonid
>>
>>> On Apr 25, 2019, at 5:51 PM, mikhailo.seledtsov at oracle.com wrote:
>>>
>>> Please review this change that uses a better platform independent 
>>> way of obtaining IP address inside a docker container.
>>> This test worked for Oracle Linux, but did not work on Fedora, which 
>>> is now fixed.
>>> The code for obtaining IP address in getLocalIp() was contributed by 
>>> Severin, thank you Severin.
>>>
>>>      JBS: https://bugs.openjdk.java.net/browse/JDK-8222769
>>>      Webrev: http://cr.openjdk.java.net/~mseledtsov/8222769.00/
>>>      Testing:
>>>          Ran the affected test on Linux-x64 (Oracle Linux 7.3, 7.6) 
>>> - Passed
>>>
>>>
>>> Thank you,
>>> Misha
>>>
>