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

Exclude test_startupShutdownRobustness #2173

Merged
merged 1 commit into from
Jun 16, 2018

Conversation

pdbain-ibm
Copy link
Contributor

This fails intermittently and can cause spurious build failures:
#2047.
Exclude this test while investigating.

Signed-off-by: Peter Bain peter_bain@ca.ibm.com

@pdbain-ibm
Copy link
Contributor Author

@llxia would you kindly review?
Thank you.

@llxia
Copy link
Contributor

llxia commented Jun 13, 2018

Jenkins test extended aix jdk10

@llxia
Copy link
Contributor

llxia commented Jun 13, 2018

fyi, for future reference, please add [ci skip] in the commit msg for the PR that does not need SDK compilation validation.

@llxia
Copy link
Contributor

llxia commented Jun 14, 2018

latest_exclude_SE80.txt is modified, but the issue is reported on JDK10. Should we exclude for SE100? Did it fail on SE80 as well?

@pdbain-ibm
Copy link
Contributor Author

We should exclude it on all versions. Do I need to update all exclusion files?

@llxia
Copy link
Contributor

llxia commented Jun 14, 2018

yes, we need to update exclusion files for all versions.

@pdbain-ibm pdbain-ibm force-pushed the attach branch 2 times, most recently from 91c2385 to df74a15 Compare June 14, 2018 15:28
@pdbain-ibm
Copy link
Contributor Author

Updated the other files.

@llxia
Copy link
Contributor

llxia commented Jun 14, 2018

Jenkins test extended aix jdk8

@llxia
Copy link
Contributor

llxia commented Jun 14, 2018

Jenkins test extended plinux jdk10

# Exclude this while investigating https://github.com/eclipse/openj9/issues/2047
# Intermittent NullPointerException in TestAttachErrorHandling.deleteIpcDirectories
org.openj9.test.attachAPI.TestAttachErrorHandling:test_startupShutdownRobustness
Copy link
Contributor

Choose a reason for hiding this comment

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

missing issue number and platform. We can remove line Exclude this while investigating https://github.com/eclipse/openj9/issues/2047

@@ -27,3 +27,6 @@ org.openj9.test.java.lang.Test_ClassLoader 134754 generic-all
org.openj9.test.java.lang.Test_Thread:test_getContextClassLoader 125908 generic-all
org.openj9.test.vmArguments.VmArgumentTests:testCrNocr 244 generic-all
j9vm.test.softmx.SoftmxRemoteTest 480 generic-all
# Exclude this while investigating https://github.com/eclipse/openj9/issues/2047
# Intermittent NullPointerException in TestAttachErrorHandling.deleteIpcDirectories
org.openj9.test.attachAPI.TestAttachErrorHandling:test_startupShutdownRobustness
Copy link
Contributor

Choose a reason for hiding this comment

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

missing issue number and platform. We can remove line Exclude this while investigating https://github.com/eclipse/openj9/issues/2047


# Exclude this while investigating https://github.com/eclipse/openj9/issues/2047
# Intermittent NullPointerException in TestAttachErrorHandling.deleteIpcDirectories
org.openj9.test.attachAPI.TestAttachErrorHandling:test_startupShutdownRobustness 480 generic-all
Copy link
Contributor

@llxia llxia Jun 14, 2018

Choose a reason for hiding this comment

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

  • it should be 2047 not 480. We can remove line Exclude this while investigating https://github.com/eclipse/openj9/issues/2047
  • class/method and platform have to be on the same line

[ci skip]

This fails intermittently and can cause spurious build failures:
eclipse-openj9#2047.
Exclude this test while investigating.

Signed-off-by: Peter Bain <peter_bain@ca.ibm.com>
@pdbain-ibm
Copy link
Contributor Author

Updated. Thanks.

@llxia
Copy link
Contributor

llxia commented Jun 15, 2018

Jenkins test extended plinux jdk8,jdk10

@llxia llxia merged commit da56ab5 into eclipse-openj9:master Jun 16, 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.

2 participants