-
Notifications
You must be signed in to change notification settings - Fork 740
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
Reformat readme for Jenkins pipelines #2557
Conversation
Jenkins compile all jdknext |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
@vsebe I'm surprised to see a compile requested on this as it doesn't modify code (only doc). Is there something I'm missing? |
@DanHeidinga - testing java next PRs |
jenkins compile all jdknext |
jenkins compile aix jdknext |
jenkins compile aix jdkn |
Sorry @vsebe not sure I'm following. Did you mean testing things unrelated to this PR? |
Jenkins test sanity aix jdknext |
@DanHeidinga - correct. |
Jenkins compile xlinux jdkn |
Dismissing review as marked WIP after initial review
Jenkins compile xlinux jdkn |
Jenkins compile xlinux jdkn |
Jenkins compile xlinux jdkn |
Jenkins test sanity aix,plinux jdknext |
Jenkins test sanity zlinux jdknext |
jenkins compile zlinux jdkn |
Jenkins compile win jdkn |
a81166b
to
7985c0e
Compare
@jdekonin @AdamBrousseau please review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks fine for now. I have more changes coming in #2728.
buildenv/jenkins/README.md
Outdated
- This job is used in other pipelines but can be launched manually | ||
- Compile and run sanity tests against new OMR content | ||
- Triggers: | ||
- Build-JDK`<version>`-`<platform>` and Test-Sanity-JDK`<version>`-`<platform>` across all platforms and JDK versions 8, 10 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is version 11 now instead of 10. In the future it may include 12 or newer versions. Probably want to avoid specifying here the exact versions since it can change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pshipton - updated line - removed 8, 10
new line:
- Build-JDK\<version\>-\<platform\> and Test-Sanity-JDK\<version\>-\<platform\> across all platforms and versions
- reformat pipelines sections for a broader coverage - update regexp for Pull Request triggers [ci skip] Signed-off-by: Violeta Sebe <[email protected]>
@jdekonin @pshipton @DanHeidinga ready for review |
[ci skip]
Signed-off-by: Violeta Sebe [email protected]