Hello Zoltan Chovan, Alexey Serbin, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24128
to look at the new patch set (#2).
Change subject: Fix JaCoCo aggregation skipped on test failure
......................................................................
Fix JaCoCo aggregation skipped on test failure
jacocoAggregatedReport was skipped whenever any test failed because
Gradle's hard dependency chain (jacocoAggregatedReport ->
jacocoTestReport -> test) caused it to be blocked by the failed test
task.
Wire test.finalizedBy jacocoTestReport (coverage builds only, via
-PgenerateCoverage) so per-subproject reports are always generated from
.exec files even on partial test failure. Remove dependsOn test to break
the blocking chain. Split the Gradle invocation into two steps in
build-and-test.sh: run tests first, then aggregate; the second step
finds jacocoTestReport UP-TO-DATE and never re-runs tests.
Tested with overriding the Jenkins coverage job:
https://jenkins.kudu.apache.org/job/coverage/61/console
Change-Id: I618069d3d3dc9548d97fcdcc5bed3fca9db67ae6
---
M build-support/jenkins/build-and-test.sh
M java/gradle/quality.gradle
2 files changed, 33 insertions(+), 8 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/28/24128/2
--
To view, visit http://gerrit.cloudera.org:8080/24128
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I618069d3d3dc9548d97fcdcc5bed3fca9db67ae6
Gerrit-Change-Number: 24128
Gerrit-PatchSet: 2
Gerrit-Owner: Marton Greber <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
