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

Jenkins continues regardless of the test compilation and test execution results #2068

Closed
llxia opened this issue Nov 23, 2020 · 5 comments
Closed
Labels
Milestone

Comments

@llxia
Copy link
Contributor

llxia commented Nov 23, 2020

The recent PR #2059 changes the exit code of maketest.sh. As a result, Jenkins continues regardless of the test compilation and test execution results.

In our recent zos test build, the test compilation failed, but the build continued to run all the tests.

Maybe PR #2059 code should be in Post stage https://github.com/AdoptOpenJDK/openjdk-tests/blob/13d113e63e13b21860f453372b97327f6fa440c0/buildenv/jenkins/JenkinsfileBase#L572

@llxia llxia added the bug label Nov 23, 2020
@sophia-guo
Copy link
Contributor

Or can just store the $MAKE command exit code.

make_exit_code=$?
'window clean up code'
exit $make_exit_code

@llxia
Copy link
Contributor Author

llxia commented Nov 23, 2020

yes, we can. But if the build crashes, the Post stage will execute regardless.

@smlambert
Copy link
Contributor

Note: the process clean up (#2059) added to maketest.sh is meant to be 'temporary', as in, waiting for PR for adoptium/TKG#45 to land, so that then this check can be moved to TKG setup phase.

@smlambert
Copy link
Contributor

Not sure what the ETA for PR for TKG/45 is, if its going to be a while, we can/should take one of the solutions proposed in this issue.

@smlambert
Copy link
Contributor

#2121 addresses this, closing this now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants