Remove coveralls integration and generate metadata for Sonarqube

Author: natemort

.buildkite/pipeline.yml

@@ -52,7 +52,8 @@ steps:
 
   - label: ":java: Unit test with test services"
     artifact_paths:
-      - "build/reports/jacoco/test/*.xml"
+      - "build/reports/jaccoco/test/jacocoTestReport.xml"
+      - "build/reports/metadata.txt"
     timeout_in_minutes: 30
     retry:
       automatic:
@@ -68,6 +69,7 @@ steps:
                 command:
                 - |-
                   ./gradlew --no-daemon test jacocoTestReport
+                  .buildkite/scripts/coverage_metadata.sh build/reports/metadata.txt
       - docker-compose#v3.0.0:
           run: unit-test-test-service
           config: docker/buildkite/docker-compose.yaml
@@ -111,24 +113,3 @@ steps:
       - docker-compose#v3.0.0:
           run: unit-test-docker-sticky-off
           config: docker/buildkite/docker-compose.yaml
-
-  - wait
-
-  - label: ":java: Report test coverage"
-    retry:
-      automatic:
-        - exit_status: "*"
-          limit: 3
-    plugins:
-      - kubernetes:
-          <<: *kubernetes
-          podSpec:
-            <<: *podSpec
-            containers:
-              - <<: *commandContainer
-                command:
-                - |-
-                  .buildkite/scripts/coverage.sh
-      - docker-compose#v3.0.0:
-          run: test-coverage-report
-          config: docker/buildkite/docker-compose.yaml

.buildkite/scripts/coverage.sh

@@ -0,0 +1,11 @@
+#!/bin/sh
+set -ex
+# This script generates coverage metadata for the coverage report.
+# Output is used by SonarQube integration in Uber and not used by OS repo coverage tool itself.
+# Example output:
+#   commit-sha: 6953daa563e8e44512bc349c9608484cfd4ec4ff
+#   timestamp: 2024-03-04T19:29:16Z
+output_path="$1"
+echo "commit-sha: $(git rev-parse HEAD)" > "$output_path"
+echo "timestamp: $(date -u +%Y-%m-%dT%H:%M:%SZ)" >> "$output_path"
+echo "Coverage metadata written to $output_path"

.buildkite/scripts/coverage_metadata.sh

@@ -7,7 +7,6 @@ buildscript {
 
 plugins {
     id "com.github.hierynomus.license" version"0.15.0"
-    id 'com.github.nbaztec.coveralls-jacoco' version '1.2.15'
     id 'com.github.sherter.google-java-format' version '0.9'
     id 'net.ltgt.errorprone' version  '1.1.1'
     id 'java-library'
@@ -362,7 +361,7 @@ jacoco {
 jacocoTestReport {
     dependsOn test
     reports {
-        xml.enabled = true // coveralls plugin depends on xml format report
+        xml.enabled = true
         html.enabled = true
     }