TKG icon indicating copy to clipboard operation
TKG copied to clipboard
verified publisher icon adoptium

Improved getDependency workflow

Open Amrutha-Kanhirathingal opened this issue 2 months ago • 8 comments

  • getDependencies.pl script run twice.Once with customUrl. If it failed, retry with 3rd party URL.
  • Closes: https://github.com/adoptium/aqa-tests/issues/6461

Amrutha-Kanhirathingal avatar Oct 24 '25 14:10 Amrutha-Kanhirathingal

Logs :

  • System jars are already present in machine

https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55715/console

  • Added wrong url to fail downloading jar using custom url (FYI -the log and snapshot has old log statements which may not be matching with the current pr logs)
image

https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55707/consoleFull

  • Extended.Perf : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55703/ : completed
  • sanity.functional : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55739/consoleFull: failed
  • extended.functional : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55723/: UNSTABLE
  • special.functional : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55724 : completed
  • sanity.perf : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55722 : completed

Amrutha-Kanhirathingal avatar Oct 24 '25 15:10 Amrutha-Kanhirathingal

I do not see that a customUrl is set and used for non-system tests. Please check descriptions in https://github.com/adoptium/aqa-tests/issues/6461#issuecomment-3377796139

00:03:06.008  Custom URL: https://openj9-jenkins.osuosl.org/job/test.getDependency/lastSuccessfulBuild/artifact/

00:03:34.431       [exec] Downloading jar from third-party URL for json-simple.jar: https://repo1.maven.org/maven2/com/googlecode/json-simple/json-simple/1.1.1/json-simple-1.1.1.jar

sanity.functional : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55718/

Only ran one test. It did not run entire sanity.functional.

llxia avatar Oct 24 '25 20:10 llxia

Hi @llxia ,

As per discussion i have updated the script to consider non-system tests as well, Please find the logs

special function : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55803/ extended perf : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55802/ sanity.function : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55801/parameters/ special function -failed case : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55812/console

Thanks

Amrutha-Kanhirathingal avatar Oct 28 '25 18:10 Amrutha-Kanhirathingal

Please also run extended.functional. Since we are using a new library Try::Tiny, please run sanity.functional on zos.

llxia avatar Oct 29 '25 21:10 llxia

Extended.functional : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55868/ Sanity.functional on ZOS : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55895/console : failed since zos don't have Try::Tiny module hence updated script by using "eval" instead of "try-catch"

Logs : sanity.functional -zos : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55914/console Extended functional : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55918/. special function : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55919/ Extended perf : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55920 sanity.perf : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55921/ extended perf ( failed - case): https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55909/ system : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/55923/consoleFull

Amrutha-Kanhirathingal avatar Oct 30 '25 11:10 Amrutha-Kanhirathingal

Could you fix Closes : #6461 in both description and commit message? It should be using the full linkCloses: https://github.com/adoptium/aqa-tests/issues/6461

llxia avatar Oct 30 '25 17:10 llxia

tested in parallel: extended.functional and sanity.functional

@Amrutha-Kanhirathingal, please fix failed cases in sanity.functional.

llxia avatar Oct 30 '25 19:10 llxia

@llxia Issue fixed : https://github.com/eclipse-openj9/openj9/pull/22877

Amrutha-Kanhirathingal avatar Oct 31 '25 07:10 Amrutha-Kanhirathingal

This PR should run on the master branch (not the release branch) to ensure all dependent tests are updated accordingly. @smlambert, I will leave it for you to merge.

llxia avatar Dec 01 '25 16:12 llxia

@Amrutha-Kanhirathingal, Could you please rerun the sanity.functional and sanity.system tests to confirm everything is still working? Since a month has passed, I just want to make sure nothing has broken..

annaibm avatar Dec 03 '25 14:12 annaibm

@Amrutha-Kanhirathingal, Could you please rerun the sanity.functional and sanity.system tests to confirm everything is still working? Since a month has passed, I just want to make sure nothing has broken..

sanity.functional : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/56773/console sanity.system : https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/56781/

Amrutha-Kanhirathingal avatar Dec 09 '25 17:12 Amrutha-Kanhirathingal