Re: [openj9-dev] Pull Request builds are changing

["Daniel Heidinga" <Daniel_Heidinga@xxxxxxxxxx>]

Adam, I'm a little concerned by this:

 

> - Since there is only 1 top level job, you can only run 1 "build" at a time (per PR). Launching another build will cancel the previous one.

 

I often want to launch builds on different platforms / jdk levels that vary in the amount of testing.  From simple compiles on some releases to deep testing on other platforms.  I"m not sure how to do that with this scheme without having to wait for each build to complete before launching the next.  

 

ie:

jenkins compile win jdk8

jenkins test sanity plinux jdk10

jenkins test extended xlinux jdk11

 

Hopefully I've overlooked something here and the system doesn't turn our parallel build launches into a serial process.

 

--Dan

----- Original message -----
From: Adam Brousseau <adam.brousseau88@xxxxxxxxx>
Sent by: openj9-dev-bounces@xxxxxxxxxxx
To: openj9-dev@xxxxxxxxxxx
Cc:
Subject: [openj9-dev] Pull Request builds are changing
Date: Fri, Sep 21, 2018 3:59 PM
 

Coming soon to a Pull Request near you...

 

Joe sent an email last week proposing the Pull Request (PR) triggers be adjusted so that 'all' would no longer be valid for test level or jdk version (only for platform). Since there has been no feedback or objection to this, we will go ahead and implement this change in #2836.

PLEASE NOTE there are many additional changes that are coming with #2836. Since there is no major change to how the PRs are launched, I have not set in stone a "cut-over" date, and will likely announce on the OpenJ9 slack once the change is in place.

 

The main purpose of this change is to align the PR builds with the other builds (Nightly, OMR Acceptance, etc) in terms of how the Jenkins jobs are structured. The two most notable changes are:

- Pull Request builds will now be a "pipeline of jobs" instead of a single job

- Pull request builds will now use the AdoptOpenJDK Test framework 

 

Other notable changes:

- Test target syntax change

      - "Jenkins test sanity.functional <platform> <version>"

      - Note: "sanity" and "extended" are still supported but will map to sanity.functional and extended.functional respectively.

- System test support

      - Jenkins test sanity.system, or extended.system

      - Please continue to be mindful of the amount of testing you run, how long each test bucket takes to execute, and how busy the farm is.

- Cmake platform support

      - Jenkins compile xlinuxcmake jdk9

      - No testing yet

      - Only supported on jdk9 at the moment

- Separated Compile and Test jobs means 1 compile feeds an sdk into many test jobs

- Github PR status updates per job

      - Very similar to how it is already. The difference being PRs will receive separate updates for compiles and tests.

- Only 1 top-level job will trigger. This job will trigger all the necessary downstream jobs.

      - You will get a PR status update for this top-level job as well as all the downstream jobs.

- Since there is only 1 top level job, you can only run 1 "build" at a time (per PR). Launching another build will cancel the previous one.

      - For example if you run "Jenkins compile zlinux,plinux jdk8" and then realize you also needed xlinux, you will have to issue another comment "Jenkins compile zlinux,plinux,xlinux jdk8". Ie. you cannot simply launch another build with just xlinux because it will cancel the first build.

 

If you have any questions or concerns please let me know. Otherwise I expect to be making the switch sometime next week.

 

Thanks,

Adam

 

_______________________________________________
openj9-dev mailing list
openj9-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/openj9-dev