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

Reformat readme for Jenkins pipelines #2557

Merged
merged 1 commit into from
Oct 24, 2018
Merged

Conversation

vsebe
Copy link
Contributor

@vsebe vsebe commented Aug 7, 2018

  • reformat pipelines sections for a broader coverage
  • update regexp for Pull Request triggers

[ci skip]

Signed-off-by: Violeta Sebe [email protected]

@vsebe
Copy link
Contributor Author

vsebe commented Aug 7, 2018

Jenkins compile all jdknext

DanHeidinga
DanHeidinga previously approved these changes Aug 7, 2018
Copy link
Member

@DanHeidinga DanHeidinga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@DanHeidinga
Copy link
Member

@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 DanHeidinga self-assigned this Aug 7, 2018
@vsebe
Copy link
Contributor Author

vsebe commented Aug 7, 2018

@DanHeidinga - testing java next PRs

@vsebe vsebe changed the title Update supported java version WIP: Update supported java version Aug 7, 2018
@vsebe
Copy link
Contributor Author

vsebe commented Aug 7, 2018

jenkins compile all jdknext

@vsebe
Copy link
Contributor Author

vsebe commented Aug 7, 2018

jenkins compile aix jdknext

@vsebe
Copy link
Contributor Author

vsebe commented Aug 7, 2018

jenkins compile aix jdkn

@DanHeidinga
Copy link
Member

testing java next PRs

Sorry @vsebe not sure I'm following. Did you mean testing things unrelated to this PR?

@vsebe
Copy link
Contributor Author

vsebe commented Aug 7, 2018

Jenkins test sanity aix jdknext

@vsebe
Copy link
Contributor Author

vsebe commented Aug 7, 2018

@DanHeidinga - correct.
I intend to test the pull requests triggers on all platforms for java next.
Please do not merge this until I ensure all PRs triggers work.

@vsebe
Copy link
Contributor Author

vsebe commented Aug 7, 2018

Jenkins compile xlinux jdkn
Jenkins test sanity xlinux jdknext
Jenkins test extended xlinux jdknext
Jenkins compile plinux jdkn
Jenkins test sanity plinux jdknext
Jenkins test extended plinux jdknext
Jenkins compile zlinux jdkn
Jenkins test sanity zlinux jdknext
Jenkins test extended zlinux jdknext
Jenkins compile xlinuxlargeheap jdkn
Jenkins test sanity xlinuxlargeheap jdknext
Jenkins test extended xlinuxlargeheap jdknext
Jenkins compile win jdkn
Jenkins test sanity win jdknext
Jenkins test extended win jdknext

@DanHeidinga DanHeidinga dismissed their stale review August 7, 2018 17:03

Dismissing review as marked WIP after initial review

@vsebe
Copy link
Contributor Author

vsebe commented Aug 7, 2018

Jenkins compile xlinux jdkn
Jenkins test sanity xlinux jdknext
Jenkins test extended xlinux jdknext

@vsebe
Copy link
Contributor Author

vsebe commented Aug 7, 2018

Jenkins compile xlinux jdkn

@vsebe
Copy link
Contributor Author

vsebe commented Aug 7, 2018

Jenkins compile xlinux jdkn
Jenkins test sanity xlinux jdknext
Jenkins test extended xlinux jdknext

@vsebe
Copy link
Contributor Author

vsebe commented Aug 7, 2018

Jenkins test sanity aix,plinux jdknext
Jenkins compile aix,plinux jdkn
Jenkins test extended plinux,aix jdknext

@vsebe
Copy link
Contributor Author

vsebe commented Aug 7, 2018

Jenkins test sanity zlinux jdknext
Jenkins test extended zlinux jdknext
Jenkins compile xlinuxlargeheap jdkn
Jenkins test sanity xlinuxlargeheap jdknext
Jenkins test extended xlinuxlargeheap jdknext

@vsebe
Copy link
Contributor Author

vsebe commented Aug 7, 2018

jenkins compile zlinux jdkn

@vsebe
Copy link
Contributor Author

vsebe commented Aug 7, 2018

Jenkins compile win jdkn
Jenkins test sanity win jdknext
Jenkins test extended win jdknext

@vsebe vsebe force-pushed the openj9 branch 7 times, most recently from a81166b to 7985c0e Compare August 8, 2018 14:50
@vsebe vsebe changed the title WIP: Reformat readme for Jenkins pipelines Reformat readme for Jenkins pipelines Sep 12, 2018
@vsebe
Copy link
Contributor Author

vsebe commented Oct 15, 2018

@jdekonin @AdamBrousseau please review

Copy link
Contributor

@AdamBrousseau AdamBrousseau left a 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.

- 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
Copy link
Member

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.

Copy link
Contributor Author

@vsebe vsebe Oct 24, 2018

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]>
@vsebe
Copy link
Contributor Author

vsebe commented Oct 24, 2018

@jdekonin @pshipton @DanHeidinga ready for review

@pshipton pshipton merged commit 196ee16 into eclipse-openj9:master Oct 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants