adoptium / aqa-tests

Home of test infrastructure for Adoptium builds
https://adoptium.net/aqavit
Apache License 2.0
135 stars 314 forks source link

[aix]: java/lang/String/concat/IntegerMinValue.java jdk21 failed #4789

Open sophia-guo opened 1 year ago

sophia-guo commented 1 year ago

java/lang/String/concat/IntegerMinValue.java - https://bugs.openjdk.org/browse/JDK-8306007 , which ask XLC17 compiler. Current xlc version in adoptium: Version: 13.01.0003.0007.

Should we update the xlc version @sxa or we will exclude the test @smlambert ?

See https://github.com/adoptium/aqa-tests/issues/4773#issuecomment-1726762777

smlambert commented 11 months ago

This likely ends up being excluded until our AIX 7.2 machines pick up the xlc17 compiler.

@sxa - are there any plans to update the xlc version on AIX machines?

sxa commented 11 months ago

Short answer: No Longer answer: We have been looking at installing the newer compiler on some machines (See https://github.com/adoptium/infrastructure/issues/3208) although it hasn't had the full playbooks run on it so is likely not yet functional. There have been some upstream discussions relating to updating to xlc17 for the builds but they were not concluded last time I looked, so we're not deploying it by default yet.

On that basis I'm in agreement with something you mentioned on our chat earlier that would suggest we disable that test for now.

smlambert commented 10 months ago

Test is disabled and linked to this issue. The ProblemList entry should really point to an issue in a different repository where a future fix to the issue would be landed, infrastructure, as this test will not be re-enabled until machine updates occur, which is currently not in plan.

All that to be said, this issue has to stay open until an infra issue is created and the ProblemList is updated to point to the open issue. Alternatively, if this will be a permanent exclusion, then can exclude via an umbrella issue for all permanent exclusions.