Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Smoke tests are getting JDK_IMPL=temurin unlike other jobs #250

Closed
sxa opened this issue Feb 21, 2022 · 8 comments
Closed

Smoke tests are getting JDK_IMPL=temurin unlike other jobs #250

sxa opened this issue Feb 21, 2022 · 8 comments
Assignees

Comments

@sxa
Copy link
Member

sxa commented Feb 21, 2022

What are you trying to do? Run smoke test jobs successfully in the nightly pipelines

Expected behaviour: Builds success

Observed behaviour: Builds fail as theyt are being passed JDK_IMPL=temurin instead of the other test jobs which are getting JDK_IMPL=hotspot - basically the smoke tests do not seem to be subject to

if (buildConfig.VARIANT == "temurin") {
, which is odd since that code is in getCommonTestJobParams which is called from getSmokeTestJobParams and getAQATestJobParams

Any other comments:

@sxa
Copy link
Member Author

sxa commented Feb 21, 2022

@sophia-guo Do you know why the temurin->hotspot parameter switch isn't taking effect for the Smoke tests but is for the other test jobs?

@sxa
Copy link
Member Author

sxa commented Feb 22, 2022

OK I'm now even more confused by what's happening ...

If I run https://ci.adoptopenjdk.net/view/work-in-progress/job/SXA-LinuxARMbuildTest/, which is a straight copy of https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-linux-aarch64-temurin/, it passed JVM_IMPL=hotspot into the generated smoke test job which passed, but it doesn't for the smoke test on the original pipeline

@sophia-guo Any idea why this is happening?

@sophia-guo
Copy link
Contributor

if (buildConfig.VARIANT == "temurin") {
is in PR #245, which is to fix this. The failed smoke tests were generated before this fix and need to be regenerated.

@sophia-guo
Copy link
Contributor

Note, recent AQA_AUTO_GEN only work for aqa test jobs, which doesn't work for smoke test job.

@sxa
Copy link
Member Author

sxa commented Feb 22, 2022

if (buildConfig.VARIANT == "temurin") {

is in PR #245, which is to fix this. The failed smoke tests were generated before this fix and need to be regenerated.

Hmmm OK I'm a bit confused as to how this works then as I thought that code was in the pipeline code that sets the parameters that are used to call the smoke test job - I'd run the regeneration of the build pipeline so thought that would change the input to the smoke test job, but I'll try deleting the smoke test job and retrying ...

@sxa
Copy link
Member Author

sxa commented Feb 23, 2022

LGTM - I'll delete all the temurin_SmokeTest jobs and let them be regenerated. Backup created as temurin_SmokeTests.tar.xz in the jobs directory on the jenkins master.

@sxa
Copy link
Member Author

sxa commented Feb 26, 2022

Now working.

@sxa sxa closed this as completed Feb 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants