RFR: JDK-8314121: test tools/jpackage/share/RuntimePackageTest.java#id0 fails on RHEL8 [v2]

[Matthias Baesken]

On Wed, 6 Sep 2023 07:29:20 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

>> on some RHEL Linux 8.X machines , we run into errors in test tools/jpackage/share/RuntimePackageTest.java#id0 , error can be seen below.
>> It looks like these errors occur when running the jtreg tests with higher concurrency, I did not see them when running just the single test.
>> 
>> When adding some test output , we see the 2 top install dirs below (1 is expected, not 2) :
>> ./test/unpacked-rpm/opt
>> ./test/unpacked-rpm/usr
>> 
>> error output :
>> 
>> java.lang.AssertionError: Expected [1]. Actual [2]: Check the package has 1 top installation directories
>>        at jdk.jpackage.test.TKit.error(TKit.java:273)
>>        at jdk.jpackage.test.TKit.assertEquals(TKit.java:576)
>>        at jdk.jpackage.test.PackageTest$Handler.verifyRootCountInUnpackedPackage(PackageTest.java:705)
>>        at jdk.jpackage.test.PackageTest$Handler.lambda$verifyPackageInstalled$3(PackageTest.java:635)
>>        at java.base/java.util.Optional.ifPresent(Optional.java:178)
>>        at jdk.jpackage.test.PackageTest$Handler.verifyPackageInstalled(PackageTest.java:633)
>>        at jdk.jpackage.test.PackageTest$Handler.accept(PackageTest.java:592)
>>        at jdk.jpackage.test.PackageTest$2.accept(PackageTest.java:504)
>>        at jdk.jpackage.test.PackageTest$2.accept(PackageTest.java:411)
>>        at jdk.jpackage.test.Functional$ThrowingConsumer.lambda$toConsumer$0(Functional.java:41)
>>        at java.base/java.lang.Iterable.forEach(Iterable.java:75)
>>        at jdk.jpackage.test.PackageTest.lambda$runActions$24(PackageTest.java:381)
>>        at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
>>        at jdk.jpackage.test.PackageTest.lambda$runActions$25(PackageTest.java:380)
>>        at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
>>        at jdk.jpackage.test.PackageTest.runActions(PackageTest.java:379)
>>        at jdk.jpackage.test.RunnablePackageTest.run(RunnablePackageTest.java:58)
>>        at RuntimePackageTest.test(RuntimePackageTest.java:83)
>>        at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
>>        at java.base/java.lang.reflect.Method.invoke(Method.java:580)
>>        at jdk.jpackage.test.MethodCall.accept(MethodCall.java:145)
>>        at jdk.jpackage.test.TestInstance.run(TestInstance.java:230)
>>        at jdk.jpackage.test.TKit.lambda$ignoreExceptions$5(TKit.java:141)
>>        at jdk.jpackage.test.TKit.lambda$runTests$3(TKit.java:126)
>>        at java.base/java.util.ArrayList$ArrayListSpl...
>
> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
> 
>   remove output from PackageTest.java

Hi Alexey,  thanks for the review !

-------------

PR Comment: https://git.openjdk.org/jdk/pull/15528#issuecomment-1709620631