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

Sync original repo #5

Merged
merged 63 commits into from
Mar 4, 2021
Merged

Sync original repo #5

merged 63 commits into from
Mar 4, 2021

Conversation

patkarns
Copy link
Owner

@patkarns patkarns commented Mar 4, 2021

No description provided.

LongyuZhang and others added 30 commits January 28, 2021 14:25
- Add add-exports to HCRLateAttachWorkload_previewEnabled

Signed-off-by: Longyu Zhang <longyu.zhang@ibm.com>
Signed-off-by: Mesbah_Alam@ca.ibm.com <Mesbah_Alam@ca.ibm.com>
- related: #2218 (comment)

Signed-off-by: GitHub <noreply@github.com>

Co-authored-by: smlambert <smlambert@users.noreply.github.com>
eclipse-openj9/openj9#11135 refers to an uber issue, which will be closed soon.
So, the references to eclipse-openj9/openj9#11135 have been replaced with
stand-alone trackers for the respective issues.

Also, a duplicate for SuperMethodTest.java has been removed.

Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>
Issue eclipse-openj9/openj9#11689

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
* Apply #2188 to OpenJ9 JDK17 exclude list

Apply #2188 to OpenJ9 JDK17 exclude list.

Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>

* Apply #2192 to OpenJ9 JDK17 exclude list

Apply #2192 to OpenJ9 JDK17 exclude list.

Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>

* Apply #2215 to OpenJ9 JDK17 exclude list

Apply #2215 to OpenJ9 JDK17 exclude list.

Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>
…new 4 test cases (#2214)

Co-authored-by: Kentaro Kuramochi <aa324442@KentaronoMacBook-Air.local>
The benefit of having lightweight checkout is to save space on Jenkins master.
However, Jenkins does not support using Repository URL and Branch build parameters with lightweight checkout together (see https://issues.jenkins.io/browse/JENKINS-48431).

In this PR, we set LIGHT_WEIGHT_CEHCKOUT to false and use Repository URL and Branch as build parameters by default. If LIGHT_WEIGHT_CEHCKOUT is set to true, set Repository URL and Branch to its explicit value

Signed-off-by: lanxia <lan_xia@ca.ibm.com>
Signed-off-by: Simon Rushton <srushton@redhat.com>
Signed-off-by: Simon Rushton <srushton@redhat.com>
…ava (#2230)

This was forgot after resolving openj9 issue #7623

Signed-off-by: Jason Feng <fengj@ca.ibm.com>
- Remove openj9_custom to merge it into jdk_custom

Signed-off-by: Longyu Zhang <longyu.zhang@ibm.com>
Signed-off-by: Simon Rushton <srushton@redhat.com>
Issue eclipse-openj9/openj9#11689

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
Signed-off-by: renfeiw <renfeiw@ca.ibm.com>
Signed-off-by: renfeiw <renfeiw@ca.ibm.com>
Signed-off-by: Mesbah_Alam@ca.ibm.com <Mesbah_Alam@ca.ibm.com>
This commit enables the following test, which used to fail with timeouts
on AArch64:

- java/math/BigInteger/LargeValueExceptions.java (eclipse-openj9/openj9#9082)

Signed-off-by: KONNO Kazuhiro <konno@jp.ibm.com>
The following test is known to timeout with OpenJ9 AArch64.
This commit excludes it.

- java/util/stream/test/org/openjdk/tests/java/util/stream/CountLargeTest.java eclipse-openj9/openj9#9040

Signed-off-by: KONNO Kazuhiro <konno@jp.ibm.com>
- Param PARALLEL_DEFAULT will be set as default
  PARALLEL option
- Param NUM_MACHINES will be set if passed

Signed-off-by: Adam Brousseau <adam.brousseau88@gmail.com>
Issue eclipse-openj9/openj9#11390

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
Fixes a command injection vulnerability with the auto test PR GitHub Workflow.

See https://securitylab.github.com/research/github-actions-untrusted-input

Also removes the unecessary `&& github.event_name != 'pull_request'` condition from the autoTestPR job because it will always be satisfied. (This workflow only triggers upon issue_comment, so `github.event_name` will never be 'pull_request')
- add a one line to cover the special case of auto disable

Signed-off-by: renfeiw <renfeiw@ca.ibm.com>
* Add certain directories trigger for PR/push testing

* use run-aqa with the target specified by the changed dirs

* remove unnecessary push trigger, leave only PR trigger

* merge openj9 and hotspot, deal with duplication situation

* Combine getChangeLocation and getBuildLists into one getBuildLists, remove unnecessary code

Co-authored-by: xichen1 <pan.chstu@gmail.com>
Avoid long hanging builds when copyArtifacts hangs

Signed-off-by: Adam Brousseau <adam.brousseau88@gmail.com>
Pass along the LIGHT_WEIGHT_CHECKOUT parameter
to the generater in order to have children
with the same value. Generally this will be false
for Ginders and true eveywhere else

Fixes #2250

Signed-off-by: Adam Brousseau <adam.brousseau88@gmail.com>
- related: #2256 (comment)

Signed-off-by: GitHub <noreply@github.com>

Co-authored-by: smlambert <smlambert@users.noreply.github.com>
LongyuZhang and others added 29 commits February 17, 2021 07:46
- exclude jdk_net failed tests temporarily
- for jdk 11, 15, and 16 on related  platforms

Signed-off-by: Longyu Zhang <longyu.zhang@ibm.com>
* add suffix to ignore native tests

* remove unnecessary matrix.bulid_list in target list
Signed-off-by: Mesbah_Alam@ca.ibm.com <Mesbah_Alam@ca.ibm.com>
* Add comment-triggered PR build action (#2199)

Add workflow `action.yml` that triggers on a PR comment starting with the word `action` then runs AQA tests based on the parameters.
Add `action_argparse.py` that parses PR comments to determine the parameters for AQA tests in the `action.yml` workflow.

Co-authored-by: Xichen Pan <xichen.pan@gmail.com>
Co-authored-by: patkarns <30379111+patkarns@users.noreply.github.com>
Signed-off-by: renfeiw <renfeiw@ca.ibm.com>
Prevents reportStatus from running on every comment by checking for the same conditions as parseComment:
- The comment must begin with `run aqa` keyword
- The comment must be a PR comment
Resolve: #2262

Signed-off-by: lanxia <lan_xia@ca.ibm.com>
Signed-off-by: Simon Rushton <srushton@redhat.com>
Signed-off-by: renfeiw <renfeiw@ca.ibm.com>
Signed-off-by: renfeiw <renfeiw@ca.ibm.com>
* test error

* test error

* remove parentheses

* use github.head_ref

* remove test file
Signed-off-by: renfeiw <renfeiw@ca.ibm.com>
* Trial external_custom target

Signed-off-by: Shelley Lambert <slambert@gmail.com>

* Update build.xml

Signed-off-by: Shelley Lambert <slambert@gmail.com>

* Add alpine-linux platform label

Signed-off-by: Shelley Lambert <slambert@gmail.com>

* add bits

Signed-off-by: Shelley Lambert <slambert@gmail.com>

* Add x86_solaris

Signed-off-by: Shelley Lambert <slambert@gmail.com>
This step is unnecessary now that we run
on any node. Also remove function as it
was only used for this step.

Follow up to #2295

Signed-off-by: Adam Brousseau <adam.brousseau88@gmail.com>
…#2315)

Modes 113 and 614 take too long to run across all platforms, and
contribute to special.system timeout problems on Windows.

(5) 113: -Xgcpolicy:gencon
-Xjit:count=0,optlevel=warm,gcOnResolve,rtResolve -Xmn512k
-Xnocompressedrefs
aix 8: 23min, 11: 31min
osx 8: 10min, 11: 20min
plinux 8: 18min, 11: 23min
xlinux 8: 25min, 11: 40min
zlinux 8: 12min, 11: 14min
Windows 11: 30min

(20) 614: -Xcompressedrefs -Xgcpolicy:gencon -Xjit:counts="- - - - - - 1
1 1 1000 250 250 - - - 10000 100000
10000",gcOnResolve,rtResolve,sampleInterval=2,scorchingSampleThreshold=10000,quickProfile
-Xmn512k -Xcheck:gc:vmthreads:all:quiet
aix 8: 19min, 11: 27min
osx 8: 17min, 11: 17min
plinux 8: 18min, 11: 22min
xlinux 8: 24min, 11: 33min
zlinux 8: 10min, 11: 14min
Windows 11: 27min

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
The OSRG modes run slow only on Windows, and contribute to Windows
special.system tests timing out. The Mode Mode107-OSRG (24) can timeout
the 60min limit and fail.

Issue eclipse-openj9/openj9#11904

variation: (24) Mode107-OSRG
JVM_OPTIONS:  -Xgcpolicy:optthruput -Xdebug
-Xrunjdwp:transport=dt_socket,address=8888,server=y,onthrow=no.pkg.foo,launch=echo
-Xjit:enableOSR,enableOSROnGuardFailure,count=1,disableAsyncCompilation
Windows 11: 67min
aix 8: 9min, 11: 4min
osx 8: 3min, 11: 5min
plinux 8: 2min, 11: 4min
xlinux 8: 4min, 11: 10min
zlinux 8: 4min, 11: 6min

variation: (25) Mode110-OSRG
JVM_OPTIONS:
-Xjit:enableOSR,enableOSROnGuardFailure,count=1,disableAsyncCompilation
-Xgcpolicy:gencon
Windows 11: 32min
aix 8: 4min, 11: 5min
osx 8: 3min, 11: 2min
plinux 8: 4min, 11: 5min
xlinux 8: 4min, 11: 6min
zlinux 8: 1min, 11: 2min

variation: (26) Mode610-OSRG
JVM_OPTIONS:  -Xcompressedrefs
-Xjit:enableOSR,enableOSROnGuardFailure,count=1,disableAsyncCompilation
-Xgcpolicy:gencon
Windows 11: 25min
aix 8: 4min, 11: 5min
osx 8: 3min, 11: 2min
plinux 8: 3min, 11: 4min
xlinux 8: 5min, 11: 6min
zlinux 8: 2min, 11: 2min

variation: (27) Mode612-OSRG
JVM_OPTIONS:  -Xcompressedrefs -Xgcpolicy:gencon
-Xjit:enableOSR,enableOSROnGuardFailure,count=1,disableAsyncCompilation
Windows 11: 33min
aix 8: 4min, 11: 5min
osx 8: 2min, 11: 4min
plinux 8: 3min, 11: 5min
xlinux 8: 4min, 11: 6min
zlinux 8: 1min, 11: 2min

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
- related: #2255 (comment)

Signed-off-by: GitHub <noreply@github.com>

Co-authored-by: smlambert <smlambert@users.noreply.github.com>
Fixes #2316 (comment)

The OSRG modes run slow only on Windows, and contribute to Windows
special.system tests timing out. The Mode Mode107-OSRG (24) can timeout
the 60min limit and fail.

Issue eclipse-openj9/openj9#11904

variation: (24) Mode107-OSRG
JVM_OPTIONS:  -Xgcpolicy:optthruput -Xdebug
-Xrunjdwp:transport=dt_socket,address=8888,server=y,onthrow=no.pkg.foo,launch=echo
-Xjit:enableOSR,enableOSROnGuardFailure,count=1,disableAsyncCompilation
Windows 11: 67min
aix 8: 9min, 11: 4min
osx 8: 3min, 11: 5min
plinux 8: 2min, 11: 4min
xlinux 8: 4min, 11: 10min
zlinux 8: 4min, 11: 6min

variation: (25) Mode110-OSRG
JVM_OPTIONS:
-Xjit:enableOSR,enableOSROnGuardFailure,count=1,disableAsyncCompilation
-Xgcpolicy:gencon
Windows 11: 32min
aix 8: 4min, 11: 5min
osx 8: 3min, 11: 2min
plinux 8: 4min, 11: 5min
xlinux 8: 4min, 11: 6min
zlinux 8: 1min, 11: 2min

variation: (26) Mode610-OSRG
JVM_OPTIONS:  -Xcompressedrefs
-Xjit:enableOSR,enableOSROnGuardFailure,count=1,disableAsyncCompilation
-Xgcpolicy:gencon
Windows 11: 25min
aix 8: 4min, 11: 5min
osx 8: 3min, 11: 2min
plinux 8: 3min, 11: 4min
xlinux 8: 5min, 11: 6min
zlinux 8: 2min, 11: 2min

variation: (27) Mode612-OSRG
JVM_OPTIONS:  -Xcompressedrefs -Xgcpolicy:gencon
-Xjit:enableOSR,enableOSROnGuardFailure,count=1,disableAsyncCompilation
Windows 11: 33min
aix 8: 4min, 11: 5min
osx 8: 2min, 11: 4min
plinux 8: 3min, 11: 5min
xlinux 8: 4min, 11: 6min
zlinux 8: 1min, 11: 2min

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
- using version in playlists

Signed-off-by: renfeiw <renfeiw@ca.ibm.com>
If we call git ls-remote on Window, when the repo doesn't exist,
the command asks for a username and hangs.

This change disables the request for a username and provides more
useful error messages. It also handles the problem appropriately,
so instead of failing the build, it correctly sets the repo location
to the non-'u' version.

Added some timeout-handling code as well, in case someone's running
this on a version of git older than 2.3, which may ignore the env
var we're using to suppress the username request.

Signed-off-by: Adam Farley <adfarley@redhat.com>
This test requires CLinker, and the upstream community has no
current intention to port it to new platforms. So we need to exclude
the test on platforms that aren't x86-64 or aarch64.

Signed-off-by: Adam Farley <adfarley@redhat.com>
Signed-off-by: Sophia Guo <sophia.gwf@gmail.com>
Also simplifies the mapping of comment/command arguments to outputs.
Signed-off-by: Stewart X Addison <sxa@redhat.com>
@patkarns patkarns merged commit fca88f9 into patkarns:master Mar 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.