-
Notifications
You must be signed in to change notification settings - Fork 720
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
Support PullRequest builds from other repos #1354
Merged
pshipton
merged 2 commits into
eclipse-openj9:master
from
AdamBrousseau:support_extensions_prs
Mar 21, 2018
Merged
Changes from all commits
Commits
Show all changes
2 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -56,14 +56,16 @@ This folder contains Jenkins pipeline scripts that are used in the OpenJ9 Jenkin | |
- `Jenkins test all` | ||
- `Jenkins test all all all` | ||
|
||
You can also request a "Compile & Sanity" build from the extensions repos or openj9-omr | ||
You can also request a Pull Request build from the extensions repos or openj9-omr | ||
- [openj9-omr](https://github.com/eclipse/openj9-omr) | ||
- [openj9-openjdk-jdk8](https://github.com/ibmruntimes/openj9-openjdk-jdk8) | ||
- [openj9-openjdk-jdk9](https://github.com/ibmruntimes/openj9-openjdk-jdk9) | ||
- [openj9-omr](https://github.com/eclipse/openj9-omr) | ||
|
||
###### Note: When specifying a dependent change in an OpenJDK extensions repo, you can only build the SDK version that matches the repo where the dependent change lives. Eg. You cannot build JDK8 with a PR in openj9-openjdk-jdk9. | ||
|
||
##### Dependent Changes | ||
|
||
- If you have dependent change(s) in either eclipse/omr, eclipse/openj9-omr, or ibmruntimes/openj9-openjdk-jdk9, you can build & test with all needed changes | ||
- If you have dependent change(s) in either eclipse/omr, eclipse/openj9-omr, or ibmruntimes/openj9-openjdk-jdk\*, you can build & test with all needed changes | ||
- Request a build by including the PR ref in your trigger comment | ||
- Ex. Dependent change in OMR Pull Request `#123` | ||
- `Jenkins test sanity depends eclipse/omr#123` | ||
|
@@ -72,11 +74,9 @@ You can also request a "Compile & Sanity" build from the extensions repos or ope | |
- Ex. Dependent change in OpenJDK Pull Request `#789` | ||
- `Jenkins test sanity depends ibmruntimes/openj9-openjdk-jdk9#789` | ||
- Ex. Dependent changes in OMR and OpenJDK | ||
- `Jenkins test sanity depends eclipse/omr#123 ibmruntimes/openj9-openjdk-jdk9#789` | ||
- `Jenkins test sanity all jdk9 depends eclipse/omr#123 ibmruntimes/openj9-openjdk-jdk9#789` | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This fixes one of these - aren't other commands incorrect as well? |
||
- Ex. If you have a dependent change and only want one platform, depends comes last | ||
- `Jenkins test sanity zlinux depends eclipse/omr#123` | ||
|
||
**Note:** Dependent changes can only be requested from an OpenJ9 Pull Request | ||
- `Jenkins test sanity zlinux all depends eclipse/omr#123` | ||
|
||
##### Other Pull Requests builds | ||
|
||
|
@@ -248,143 +248,36 @@ You can also request a "Compile & Sanity" build from the extensions repos or ope | |
- Trigger: Triggered by `Mirror-OMR-to-OpenJ9-OMR` | ||
|
||
#### Pull Requests | ||
###### **REPO** indicates the job is configured for Pull Requests from that repository. Can be one of openj9, openj9-omr, openj9-openjdk-jdk8 etc. | ||
###### **VERSION** indicates the job is configured for each JDK version. Can be one of 8,9,10 etc. | ||
###### **SPEC** indicates the job is configured for each platform spec. Can be one of aix_ppc-64_cmprssptrs, linux_ppc-64_cmprssptrs_le, linux_390-64_cmprssptrs etc. | ||
|
||
- PullRequest-Compile-JDK8-aix_ppc-64_cmprssptrs-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Compile-JDK8-aix_ppc-64_cmprssptrs-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Compile-JDK8-aix_ppc-64_cmprssptrs-OpenJ9) | ||
- Description: | ||
- Compile on java8 aix_ppc-64_cmprssptrs | ||
- Trigger: | ||
- Github PR comment `Jenkins compile aix jdk8` | ||
|
||
- PullRequest-Compile-JDK9-aix_ppc-64_cmprssptrs-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Compile-JDK9-aix_ppc-64_cmprssptrs-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Compile-JDK9-aix_ppc-64_cmprssptrs-OpenJ9) | ||
- Description: | ||
- Compile on java9 aix_ppc-64_cmprssptrs | ||
- Trigger: | ||
- Github PR comment `Jenkins compile aix jdk9` | ||
|
||
- PullRequest-Compile-JDK8-linux_390-64_cmprssptrs-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Compile-JDK8-linux_390-64_cmprssptrs-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Compile-JDK8-linux_390-64_cmprssptrs-OpenJ9) | ||
- Description: | ||
- Compile on java8 linux_390-64_cmprssptrs | ||
- Trigger: | ||
- Github PR comment `Jenkins compile zlinux jdk8` | ||
|
||
- PullRequest-Compile-JDK9-linux_390-64_cmprssptrs-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Compile-JDK9-linux_390-64_cmprssptrs-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Compile-JDK9-linux_390-64_cmprssptrs-OpenJ9) | ||
- Description: | ||
- Compile on java9 linux_390-64_cmprssptrs | ||
- Trigger: | ||
- Github PR comment `Jenkins compile zlinux jdk9` | ||
|
||
- PullRequest-Compile-JDK8-linux_ppc-64_cmprssptrs_le-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Compile-JDK8-linux_ppc-64_cmprssptrs_le-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Compile-JDK8-linux_ppc-64_cmprssptrs_le-OpenJ9) | ||
- Description: | ||
- Compile on java8 linux_ppc-64_cmprssptrs_le | ||
- Trigger: | ||
- Github PR comment `Jenkins compile plinux jdk8` | ||
|
||
- PullRequest-Compile-JDK9-linux_ppc-64_cmprssptrs_le-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Compile-JDK9-linux_ppc-64_cmprssptrs_le-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Compile-JDK9-linux_ppc-64_cmprssptrs_le-OpenJ9) | ||
- Description: | ||
- Compile on java9 linux_ppc-64_cmprssptrs_le | ||
- Trigger: | ||
- Github PR comment `Jenkins compile plinux jdk9` | ||
|
||
- PullRequest-JDK8-Extended-aix_ppc-64_cmprssptrs-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Extended-JDK8-aix_ppc-64_cmprssptrs-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Extended-JDK8-aix_ppc-64_cmprssptrs-OpenJ9) | ||
- Description: | ||
- Compile and Extended tests on java8 aix_ppc-64_cmprssptrs | ||
- Trigger: | ||
- Github PR comment `Jenkins test extended aix jdk8` | ||
|
||
- PullRequest-JDK9-Extended-aix_ppc-64_cmprssptrs-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Extended-JDK9-aix_ppc-64_cmprssptrs-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Extended-JDK9-aix_ppc-64_cmprssptrs-OpenJ9) | ||
- Description: | ||
- Compile and Extended tests on java9 aix_ppc-64_cmprssptrs | ||
- Trigger: | ||
- Github PR comment `Jenkins test extended aix jdk9` | ||
|
||
- PullRequest-JDK8-Extended-linux_390-64_cmprssptrs-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Extended-JDK8-linux_390-64_cmprssptrs-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Extended-JDK8-linux_390-64_cmprssptrs-OpenJ9) | ||
- Description: | ||
- Compile and Extended tests on java8 linux_390-64_cmprssptrs | ||
- Trigger: | ||
- Github PR comment `Jenkins test extended zlinux jdk8` | ||
|
||
- PullRequest-JDK9-Extended-linux_390-64_cmprssptrs-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Extended-JDK9-linux_390-64_cmprssptrs-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Extended-JDK9-linux_390-64_cmprssptrs-OpenJ9) | ||
- Description: | ||
- Compile and Extended tests on java9 linux_390-64_cmprssptrs | ||
- Trigger: | ||
- Github PR comment `Jenkins test extended zlinux jdk9` | ||
|
||
- PullRequest-Extended-JDK8-linux_ppc-64_cmprssptrs_le-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Extended-JDK8-linux_ppc-64_cmprssptrs_le-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Extended-JDK8-linux_ppc-64_cmprssptrs_le-OpenJ9) | ||
- Description: | ||
- Compile and Extended tests on java8 linux_ppc-64_cmprssptrs_le | ||
- Trigger: | ||
- Github PR comment `Jenkins test extended plinux jdk8` | ||
|
||
- PullRequest-Extended-JDK9-linux_ppc-64_cmprssptrs_le-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Extended-JDK9-linux_ppc-64_cmprssptrs_le-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Extended-JDK9-linux_ppc-64_cmprssptrs_le-OpenJ9) | ||
- Description: | ||
- Compile and Extended tests on java9 linux_ppc-64_cmprssptrs_le | ||
- Trigger: | ||
- Github PR comment `Jenkins test extended plinux jdk9` | ||
|
||
- PullRequest-Sanity-JDK8-aix_ppc-64_cmprssptrs-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Sanity-JDK8-aix_ppc-64_cmprssptrs-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Sanity-JDK8-aix_ppc-64_cmprssptrs-OpenJ9) | ||
- Description: | ||
- Compile and Sanity tests on java8 aix_ppc-64_cmprssptrs | ||
- Trigger: | ||
- Github PR comment `Jenkins test sanity aix jdk8` | ||
|
||
- PullRequest-Sanity-JDK9-aix_ppc-64_cmprssptrs-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Sanity-JDK9-aix_ppc-64_cmprssptrs-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Sanity-JDK9-aix_ppc-64_cmprssptrs-OpenJ9) | ||
- Description: | ||
- Compile and Sanity tests on java9 aix_ppc-64_cmprssptrs | ||
- Trigger: | ||
- Github PR comment `Jenkins test sanity aix jdk9` | ||
|
||
- PullRequest-Sanity-JDK8-linux_390-64_cmprssptrs-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Sanity-JDK8-linux_390-64_cmprssptrs-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Sanity-JDK8-linux_390-64_cmprssptrs-OpenJ9) | ||
- Description: | ||
- Compile and Sanity tests on java8 linux_390-64_cmprssptrs | ||
- Trigger: | ||
- Github PR comment `Jenkins test sanity zlinux jdk8` | ||
|
||
- PullRequest-Sanity-JDK9-linux_390-64_cmprssptrs-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Sanity-JDK9-linux_390-64_cmprssptrs-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Sanity-JDK9-linux_390-64_cmprssptrs-OpenJ9) | ||
- PullRequest-Compile-JDK**VERSION**-**SPEC**-**REPO** | ||
- Description: | ||
- Compile and Sanity tests on java9 linux_390-64_cmprssptrs | ||
- Compile java **VERSION** on platform **SPEC** | ||
- Trigger: | ||
- Github PR comment `Jenkins test sanity zlinux jdk9` | ||
- Github PR comment example `Jenkins compile <SPEC> jdk<VERSION>` | ||
|
||
- PullRequest-Sanity-JDK8-linux_ppc-64_cmprssptrs_le-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Sanity-JDK8-linux_ppc-64_cmprssptrs_le-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Sanity-JDK8-linux_ppc-64_cmprssptrs_le-OpenJ9) | ||
- PullRequest-Extended-JDK**VERSION**-**SPEC**-**REPO** | ||
- Description: | ||
- Compile and Sanity tests on java8 linux_ppc-64_cmprssptrs_le | ||
- Compile and Extended tests java **VERSION** on platform **SPEC** | ||
- Trigger: | ||
- Github PR comment `Jenkins test sanity plinux jdk8` | ||
- Github PR comment `Jenkins test extended <SPEC> jdk<VERSION>` | ||
|
||
- PullRequest-Sanity-JDK9-linux_ppc-64_cmprssptrs_le-OpenJ9 | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-Sanity-JDK9-linux_ppc-64_cmprssptrs_le-OpenJ9)](https://ci.eclipse.org/openj9/job/PullRequest-Sanity-JDK9-linux_ppc-64_cmprssptrs_le-OpenJ9) | ||
- PullRequest-Sanity-JDK**VERSION**-**SPEC**-**REPO** | ||
- Description: | ||
- Compile and Sanity tests on java9 linux_ppc-64_cmprssptrs_le | ||
- Compile and Sanity tests java **VERSION** on platform **SPEC** | ||
- Trigger: | ||
- Github PR comment `Jenkins test sanity plinux jdk9` | ||
- Github PR comment `Jenkins test sanity <SPEC> jdk<VERSION>` | ||
|
||
- PullRequest-LineEndingsCheck | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-LineEndingsCheck)](https://ci.eclipse.org/openj9/job/PullRequest-LineEndingsCheck) | ||
- PullRequest-LineEndingsCheck-**REPO** | ||
- Description: | ||
- Checks the files modified in a pull request have correct line endings | ||
- Trigger: | ||
- Automatically builds on every PR | ||
- Retrigger with `Jenkins line endings check` | ||
|
||
- PullRequest-CopyrightCheck | ||
- [![Build Status](https://ci.eclipse.org/openj9/buildStatus/icon?job=PullRequest-CopyrightCheck)](https://ci.eclipse.org/openj9/job/PullRequest-CopyrightCheck) | ||
- PullRequest-CopyrightCheck-**REPO** | ||
- Description: | ||
- Checks the files modified in a pull request have copyright with current year | ||
- Trigger: | ||
|
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Isn't this line missing
all jdk9
as well?