[HDDS-10101] Set sonar.coverage.jacoco.xmlReportPaths

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Implemented
• Fix Version/s: 2.0.0
• Component/s: CI
• Labels: pull-request-available
• Assignee: Attila Doroszlai
• Reporter: Attila Doroszlai
• Created: 09/Jan/24 20:19
• Updated: 27/Jan/24 07:10
• Resolved: 27/Jan/24 07:10

Description

sonar.sh copies the combined coverage report to all submodules:

https://github.com/apache/ozone/blob/2ae531b0f6a069db5a46bd486bb50225a168485d/hadoop-ozone/dev-support/checks/sonar.sh#L26-L31

#Workaround: Sonar expects per-project Sonar XML report, but we have one, combined. Sonar seems to handle it well.
# Only the classes from the current project will be used. We can copy the same, combined report to all the subprojects.
if [ -f "$PROJECT_DIR/target/coverage/all.xml" ]; then
   find "$PROJECT_DIR" -name pom.xml | grep -v target | xargs dirname | xargs -n1 -IDIR mkdir -p DIR/target/coverage/
   find "$PROJECT_DIR" -name pom.xml | grep -v target | xargs dirname | xargs -n1 -IDIR cp "$PROJECT_DIR/target/coverage/all.xml" DIR/target/coverage/
fi

We can avoid the need for this hack by setting sonar.coverage.jacoco.xmlReportPaths.

Issue Links

• links to: GitHub Pull Request #5972