-
Notifications
You must be signed in to change notification settings - Fork 729
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
Pull Request builds for z/p linux (test sanity) #91
Comments
Do we want to archive the sdk from the build so developers can download it if needed? |
OpenJ9 does not create binaries. Archiving the sdk and making it available violates that, although convenient. |
To be clear, archiving the artifacts of the build is temporary (done within the Jenkinsfile / pipeline script). What ever we choose to archive, will be available only for as long as we keep the Jenkins workspaces around, which is configurable (last 5 builds, last 3 days, etc). We plan to archive the test output to make it easier for developers to debug issues, should there be any... |
@pshipton is this a policy we decided on or a legal roadblock? |
It is a legal issue. As per the Eclipse project proposal https://projects.eclipse.org/proposals/eclipse-openj9 OpenJ9 is a source code only project that does not distribute binaries. The binaries include OpenJDK and other third party code which is available under different licenses from OpenJ9. |
Eclipse Bug resolved, we now have a webhook. I have added (am adding) the current OpenJ9 committers to the whitelist (admin list). See description for trigger phrases. Unfortunately the PR Builder plugin is not honouring the global config admin list, I therefore had to add the list to every PR job we have configured to use the plugin. Hopefully this can be resolved. Ideally we would whitelist the github team but there is no support for teams right now, only orgs |
@jduimovich has confirmed hosting the binaries is not an issue. AdoptOpenJDK is the official distribution channel for the binaries not OpenJ9. However, we are storing the artifacts temporarily for the purposes of build/test. We are only building using the same methods anyone else can do themselves. If there are no more objections I will add the bits to the job to archive the PR sdks back to Jenkins. |
Similar to #24, we need pull request builds setup on our Jenkins instance.
jenkins test sanity
case insensitive for all platformsjenkins test sanity plinux
jenkins test sanity zlinux
The text was updated successfully, but these errors were encountered: