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.

Sign up for GitHub

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

Jump to bottom

Execute all tests in report dir #2989

Merged
merged 1 commit into from
Oct 1, 2018
Merged

Execute all tests in report dir #2989

merged 1 commit into from
Oct 1, 2018

Conversation

renfeiw
Copy link
Contributor

@renfeiw renfeiw commented Sep 24, 2018

  • cd into report dir before each test target
  • cd out and remove test result if test passed

[ci skip]

Signed-off-by: Renfei Wang [email protected]

@renfeiw
Copy link
Contributor Author

renfeiw commented Sep 24, 2018

Reviewer: @llxia

@llxia
Copy link
Contributor

llxia commented Sep 24, 2018

Jenkins test all

@llxia
Copy link
Contributor

llxia commented Sep 24, 2018

Jenkins test sanity xlinux jdk11

1 similar comment
@llxia
Copy link
Contributor

llxia commented Sep 24, 2018

Jenkins test sanity xlinux jdk11

@llxia
Copy link
Contributor

llxia commented Sep 24, 2018

Jenkins test all

@renfeiw renfeiw force-pushed the tkg4 branch 2 times, most recently from 2209ad4 to 9d23957 Compare September 25, 2018 19:02
@llxia
Copy link
Contributor

llxia commented Sep 25, 2018

Jenkins test sanity xlinux jdk11

@llxia llxia mentioned this pull request Sep 26, 2018
Merged
@llxia
Copy link
Contributor

llxia commented Sep 26, 2018

Jenkins test sanity xlinux jdk11

1 similar comment
@llxia
Copy link
Contributor

llxia commented Sep 26, 2018

Jenkins test sanity xlinux jdk11

@llxia
Copy link
Contributor

llxia commented Sep 27, 2018

Jenkins test all

@renfeiw
Execute all tests in report dir
9cae94a 
- cd into report dir before each test target
- cd out and remove test result if test passed
- use absolute directory in MethodVisibilityTests and UnreflectTests
- remove all redundant mkdir and cd process in playlist.xml

[ci skip]

Signed-off-by: Renfei Wang <[email protected]>
@llxia
Copy link
Contributor

llxia commented Sep 27, 2018

Jenkins test all xlinux jdk11

@llxia
Copy link
Contributor

llxia commented Sep 27, 2018

Jenkins test sanity xlinux jdk11

@llxia
Copy link
Contributor

llxia commented Sep 27, 2018

Jenkins test extended xlinux jdk11

@llxia
Copy link
Contributor

llxia commented Sep 28, 2018

Jenkins test all

@smlambert
Copy link
Contributor

Note: this change accomplishes 2 things

  1. reducing the amount of space by not keeping test output for tests that pass
  2. executes tests inside output folder, which means that any tests that crash and produce a core file will now see the core file included in the test artifact that will be downloadable via Jenkins (Fixes Find and archive any core / dmp files in post test step  #969)

@smlambert smlambert merged commit 53783a3 into eclipse-openj9:master Oct 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@llxia llxia llxia approved these changes

@smlambert smlambert smlambert approved these changes

Assignees
No one assigned
Labels
comp:test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@renfeiw @llxia @smlambert @fjeremic