-
-
Notifications
You must be signed in to change notification settings - Fork 72
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
Issues with logics in openjdk*-pipeline publish process #1148
Comments
I'm thinking we should try and get to the position of not publishing if the build job status is not SUCCESSFUL|UNSTABLE, |
Another choice is not publishing if build itself failed ( build, any other build related jobs - sign, installer,.., jobs/steps before test jobs). This can be done by check job status ( set by related jobs) before tests. If it's non successful do the artifacts deletion so no artifacts available. Hence no artifacts will be publish for EA and no release link will be generated for release. |
The discussion point needs to be had about whether we still publish EA builds regardless of test results. |
Yes, there are two cases needed to discuss.
|
Currently the openjdk*-pipeline can be triggered manually with releaseType='weekly' and enableTests=false. That is with no tests at all all build results will be published to temurin binary repo. When run tests query by nightly or url directly tests failed without workable jdk. https://github.com/adoptium/temurin8-binaries/releases/download/jdk8u442-b02-ea-beta/OpenJDK8U-jdk_x64_mac_hotspot_8u442b02-ea.tar.gz.
Another thing is in openjdk*-pipeline build it's clearly build job failed, we will still publish archived artifacts. https://ci.adoptium.net/job/build-scripts/job/openjdk8-pipeline/2947/consoleFull. Four platforms build jobs failed, artifacts published regardless. This is common to release build, as it's difficult to tell if the build job itself (build process ) fails or it fails because the downstream test jobs fail. Need to think about discriminating those two situation and decide publish or not.
The text was updated successfully, but these errors were encountered: