Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24128 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/24128 Tested-by: Kudu Jenkins Reviewed-by: Zoltan Chovan <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M build-support/jenkins/build-and-test.sh M java/gradle/quality.gradle 2 files changed, 33 insertions(+), 8 deletions(-) Approvals: Kudu Jenkins: Verified Zoltan Chovan: Looks good to me, approved Alexey Serbin: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I618069d3d3dc9548d97fcdcc5bed3fca9db67ae6 Gerrit-Change-Number: 24128 Gerrit-PatchSet: 3 Gerrit-Owner: Marton Greber <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]>
