Pull Request: 7013: AIX: InetAddress.getByName(addr) does not work as expected
Hello all, This is a test that fails in some environments - enough so that has been added to the ignore-list at AdoptOpenJDK (now Adoptium). And it will stay there until it gets fixed upstream. I researched this and came up with differences that are behave differently depending on the environment, e.g., my server, and partitions hosted at IBM garage respond - at CLI - ``` aixtools@x064:[/data/prj/aixtools/tests]ping 0.0.0.0 PING 0.0.0.0: (0.0.0.0): 56 data bytes --- 0.0.0.0 ping statistics --- 5 packets transmitted, 0 packets received, 100% packet loss ``` While at OSUOSL environment: ``` aixtools@nim.bak:[/home/aixtools]ping 0.0.0.0 PING 0.0.0.0: (0.0.0.0): 56 data bytes 64 bytes from 140.211.9.1: icmp_seq=0 ttl=255 time=0 ms 64 bytes from 140.211.9.1: icmp_seq=1 ttl=255 time=0 ms 64 bytes from 140.211.9.1: icmp_seq=2 ttl=255 time=0 ms --- 0.0.0.0 ping statistics --- 3 packets transmitted, 3 packets received, 0% packet loss round-trip min/avg/max = 0/0/0 ms ``` The patch is to make this test work in both environments - without any need to change the `java.net.InetAddress` or `java.net.Inet6Address` definitions. Regards, Michael
Hi Micheal, welcome, and thank you for your contribution! I opened a bug id for you to track this: https://bugs.openjdk.java.net/browse/JDK-8280498 . I can sponsor this patch for you. You need one reviewer, I think. I don't have the cycles right now to think this through. I cc core-libs dev since this is a networking issue and one of their tests. Cheers, Thomas On Thu, Jan 20, 2022 at 2:50 PM Michael Felt <aixtools@felt.demon.nl> wrote:
Hello all,
This is a test that fails in some environments - enough so that has been added to the ignore-list at AdoptOpenJDK (now Adoptium). And it will stay there until it gets fixed upstream.
I researched this and came up with differences that are behave differently depending on the environment, e.g., my server, and partitions hosted at IBM garage respond - at CLI -
```
aixtools@x064:[/data/prj/aixtools/tests]ping 0.0.0.0 PING 0.0.0.0: (0.0.0.0): 56 data bytes
--- 0.0.0.0 ping statistics --- 5 packets transmitted, 0 packets received, 100% packet loss ```
While at OSUOSL environment: ```
aixtools@nim.bak:[/home/aixtools]ping 0.0.0.0 PING 0.0.0.0: (0.0.0.0): 56 data bytes 64 bytes from 140.211.9.1: icmp_seq=0 ttl=255 time=0 ms 64 bytes from 140.211.9.1: icmp_seq=1 ttl=255 time=0 ms 64 bytes from 140.211.9.1: icmp_seq=2 ttl=255 time=0 ms
--- 0.0.0.0 ping statistics --- 3 packets transmitted, 3 packets received, 0% packet loss round-trip min/avg/max = 0/0/0 ms ```
The patch is to make this test work in both environments - without any need to change the `java.net.InetAddress` or `java.net.Inet6Address` definitions.
Regards,
Michael
On 22/01/2022 08:40, Thomas Stüfe wrote:
Hi Micheal,
welcome, and thank you for your contribution!
I opened a bug id for you to track this: https://bugs.openjdk.java.net/browse/JDK-8280498 . I can sponsor this patch for you.
You need one reviewer, I think. I don't have the cycles right now to think this through. I cc core-libs dev since this is a networking issue and one of their tests.
The networking tests are maintained on net-dev so if there is a PR then it should go to that list. -Alan
Redirecting to net-dev On Sat, Jan 22, 2022 at 11:00 AM Alan Bateman <Alan.Bateman@oracle.com> wrote:
On 22/01/2022 08:40, Thomas Stüfe wrote:
Hi Micheal,
welcome, and thank you for your contribution!
I opened a bug id for you to track this: https://bugs.openjdk.java.net/browse/JDK-8280498 . I can sponsor this patch for you.
You need one reviewer, I think. I don't have the cycles right now to think this through. I cc core-libs dev since this is a networking issue and one of their tests.
The networking tests are maintained on net-dev so if there is a PR then it should go to that list.
-Alan
Thank you Thomas! From: Thomas Stüfe <thomas.stuefe@gmail.com> Sent: Saturday, 22 January 2022 11:23 To: Alan Bateman <Alan.Bateman@oracle.com> Cc: Michael Felt <aixtools@felt.demon.nl>; ppc-aix-port-dev <ppc-aix-port-dev@openjdk.java.net>; net-dev <net-dev@openjdk.java.net> Subject: Re: Pull Request: 7013: AIX: InetAddress.getByName(addr) does not work as expected Redirecting to net-dev On Sat, Jan 22, 2022 at 11:00 AM Alan Bateman <Alan.Bateman@oracle.com <mailto:Alan.Bateman@oracle.com> > wrote: On 22/01/2022 08:40, Thomas Stüfe wrote:
Hi Micheal,
welcome, and thank you for your contribution!
I opened a bug id for you to track this: https://bugs.openjdk.java.net/browse/JDK-8280498 . I can sponsor this patch for you.
You need one reviewer, I think. I don't have the cycles right now to think this through. I cc core-libs dev since this is a networking issue and one of their tests.
The networking tests are maintained on net-dev so if there is a PR then it should go to that list. -Alan
Hi, I went to the ticket you opened – and it seems I need to do something additional – as I cannot login to comment (should I need to). Thx again for opening the ticket. From: Thomas Stüfe <thomas.stuefe@gmail.com> Sent: Saturday, 22 January 2022 11:23 To: Alan Bateman <Alan.Bateman@oracle.com> Cc: Michael Felt <aixtools@felt.demon.nl>; ppc-aix-port-dev <ppc-aix-port-dev@openjdk.java.net>; net-dev <net-dev@openjdk.java.net> Subject: Re: Pull Request: 7013: AIX: InetAddress.getByName(addr) does not work as expected Redirecting to net-dev On Sat, Jan 22, 2022 at 11:00 AM Alan Bateman <Alan.Bateman@oracle.com <mailto:Alan.Bateman@oracle.com> > wrote: On 22/01/2022 08:40, Thomas Stüfe wrote:
Hi Micheal,
welcome, and thank you for your contribution!
I opened a bug id for you to track this: https://bugs.openjdk.java.net/browse/JDK-8280498 . I can sponsor this patch for you.
You need one reviewer, I think. I don't have the cycles right now to think this through. I cc core-libs dev since this is a networking issue and one of their tests.
The networking tests are maintained on net-dev so if there is a PR then it should go to that list. -Alan
Thanks Alan - for opening the ticket. (Open my eyes). Still need to know if I am expected to repond to the ticket, and if so - how. -----Original Message----- From: Alan Bateman <Alan.Bateman@oracle.com> Sent: Saturday, 22 January 2022 11:00 To: Thomas Stüfe <thomas.stuefe@gmail.com>; Michael Felt <aixtools@felt.demon.nl> Cc: ppc-aix-port-dev <ppc-aix-port-dev@openjdk.java.net>; Java Core Libs <core-libs-dev@openjdk.java.net> Subject: Re: Pull Request: 7013: AIX: InetAddress.getByName(addr) does not work as expected On 22/01/2022 08:40, Thomas Stüfe wrote:
Hi Micheal,
welcome, and thank you for your contribution!
I opened a bug id for you to track this: https://bugs.openjdk.java.net/browse/JDK-8280498 . I can sponsor this patch for you.
You need one reviewer, I think. I don't have the cycles right now to think this through. I cc core-libs dev since this is a networking issue and one of their tests.
The networking tests are maintained on net-dev so if there is a PR then it should go to that list. -Alan
participants (3)
-
Alan Bateman
-
Michael Felt
-
Thomas Stüfe