-
-
Notifications
You must be signed in to change notification settings - Fork 73
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
JDK22+ Mac x64 builds cannot be tested on MacOS versions below 11 #949
Comments
Draft PR created here, which was based on this fix here. Being tested here: https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk22/job/jdk22-mac-x64-temurin/18/console |
The additional test label isn't being passed down to the smoke test job. I think this is because we pass the point where the modified code is called before the pipeline reaches the stage where we can modify it via jenkins job parameter. Here's a run where we do use the modified code: https://ci.adoptium.net/job/build-scripts/job/openjdk22-pipeline/128/console |
To resolve the issue described in #949 (comment), try adding |
Thanks Shelley. I think it's already there, I just wasn't running my modified code due to the hard-coded pipeline repo/branch value for ci-jenkins-pipeline. I've hit this before, I just forgot about it. My rerun here definitely uses my modified code. |
Ok, that works! Requesting reviews now. |
PR merged, issue closed. |
What are you trying to do?
Ensure all JDK22+ Mac x64 tests are run with a node label that excludes the one MacOS test node using a version under 11.
Expected behaviour:
The node label for all testing should include a "!sw.os.osx.10_14".
Observed behaviour:
https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk22/job/jdk22-mac-x64-temurin/15/
We fail when we get to the smoke tests on a 10.14 machine. Error message:
Any other comments:
Discovered here, discussed here, and documented here.
The text was updated successfully, but these errors were encountered: