This is an automated email from the ASF dual-hosted git repository.
voonhous pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hudi.git
The following commit(s) were added to refs/heads/master by this push:
new 7f786d434fa8 chore: upload Flink integration-test coverage (#19343)
7f786d434fa8 is described below
commit 7f786d434fa8e1d60880fac9bc5a9323b99c0f0a
Author: Danny Chan <[email protected]>
AuthorDate: Wed Jul 22 15:02:39 2026 +0800
chore: upload Flink integration-test coverage (#19343)
* ci: upload Flink integration-test coverage
* chore: clarify Flink integration-test coverage
---
.github/workflows/bot.yml | 26 ++++++++++++++++++++++++--
pom.xml | 9 +++++----
2 files changed, 29 insertions(+), 6 deletions(-)
diff --git a/.github/workflows/bot.yml b/.github/workflows/bot.yml
index 52198c4cf3bb..ba93eb03db30 100644
--- a/.github/workflows/bot.yml
+++ b/.github/workflows/bot.yml
@@ -1087,7 +1087,18 @@ jobs:
if: ${{ endsWith(env.FLINK_PROFILE, '2.1') &&
needs.changes.outputs.relevant == 'true' }}
run: |
mvn clean install -T 2 -Pintegration-tests -D"$SCALA_PROFILE"
-D"$FLINK_PROFILE" -pl hudi-flink-datasource/hudi-flink -am
-Davro.version="$FLINK_AVRO_VERSION" -Dparquet.version="$FLINK_PARQUET_VERSION"
-DskipTests=true $MVN_ARGS
- mvn verify -Pintegration-tests -D"$SCALA_PROFILE" -D"$FLINK_PROFILE"
-Davro.version="$FLINK_AVRO_VERSION" -Dparquet.version="$FLINK_PARQUET_VERSION"
$FLINK_IT_FILTER1 -pl hudi-flink-datasource/hudi-flink $MVN_ARGS
+ mvn verify -Pintegration-tests -D"$SCALA_PROFILE" -D"$FLINK_PROFILE"
-Davro.version="$FLINK_AVRO_VERSION" -Dparquet.version="$FLINK_PARQUET_VERSION"
$FLINK_IT_FILTER1 -pl hudi-flink-datasource/hudi-flink $MVN_ARGS
-Djacoco.skip=false
+ - name: Generate merged coverage report
+ if: always() && endsWith(matrix.flinkProfile, '2.1') &&
needs.changes.outputs.relevant == 'true'
+ run: ./scripts/jacoco/generate_merged_coverage_report.sh
$GITHUB_WORKSPACE
+ - name: Upload coverage to Codecov
+ if: always() && endsWith(matrix.flinkProfile, '2.1') &&
needs.changes.outputs.relevant == 'true'
+ uses: codecov/codecov-action@75cd11691c0faa626561e295848008c8a7dddffe
# v5
+ with:
+ files: ./jacoco-report.xml
+ disable_search: true
+ flags: flink-integration-tests
+ token: ${{ secrets.CODECOV_TOKEN }}
test-flink-2:
runs-on: ubuntu-latest
@@ -1127,7 +1138,18 @@ jobs:
if: ${{ endsWith(env.FLINK_PROFILE, '2.1') &&
needs.changes.outputs.relevant == 'true' }}
run: |
mvn clean install -T 2 -Pintegration-tests -D"$SCALA_PROFILE"
-D"$FLINK_PROFILE" -pl hudi-flink-datasource/hudi-flink -am
-Davro.version="$FLINK_AVRO_VERSION" -Dparquet.version="$FLINK_PARQUET_VERSION"
-DskipTests=true $MVN_ARGS
- mvn verify -Pintegration-tests -D"$SCALA_PROFILE" -D"$FLINK_PROFILE"
-Davro.version="$FLINK_AVRO_VERSION" -Dparquet.version="$FLINK_PARQUET_VERSION"
$FLINK_IT_FILTER2 -pl hudi-flink-datasource/hudi-flink $MVN_ARGS
+ mvn verify -Pintegration-tests -D"$SCALA_PROFILE" -D"$FLINK_PROFILE"
-Davro.version="$FLINK_AVRO_VERSION" -Dparquet.version="$FLINK_PARQUET_VERSION"
$FLINK_IT_FILTER2 -pl hudi-flink-datasource/hudi-flink $MVN_ARGS
-Djacoco.skip=false
+ - name: Generate merged coverage report
+ if: always() && endsWith(matrix.flinkProfile, '2.1') &&
needs.changes.outputs.relevant == 'true'
+ run: ./scripts/jacoco/generate_merged_coverage_report.sh
$GITHUB_WORKSPACE
+ - name: Upload coverage to Codecov
+ if: always() && endsWith(matrix.flinkProfile, '2.1') &&
needs.changes.outputs.relevant == 'true'
+ uses: codecov/codecov-action@75cd11691c0faa626561e295848008c8a7dddffe
# v5
+ with:
+ files: ./jacoco-report.xml
+ disable_search: true
+ flags: flink-integration-tests
+ token: ${{ secrets.CODECOV_TOKEN }}
docker-java17-test:
runs-on: ubuntu-latest
diff --git a/pom.xml b/pom.xml
index 178c9f52c811..a5a4d0685cd5 100644
--- a/pom.xml
+++ b/pom.xml
@@ -2409,10 +2409,11 @@
</execution>
</executions>
</plugin>
- <!-- Attach the JaCoCo agent so the integration-test (failsafe) and
integ-test unit
- (surefire) runs emit coverage. The agent argument is appended
to the argLine
- property, which both plugins pick up. destFile matches the
jacoco-agent path
- that scripts/jacoco/generate_merged_coverage_report.sh globs
for. -->
+ <!-- Attach the JaCoCo agent so integration-tests profile runs,
including Flink
+ datasource jobs, emit coverage from both integration-test
(failsafe) and
+ integ-test unit (surefire) executions. The agent argument is
appended to the
+ argLine property, which both plugins pick up. destFile matches
the jacoco-agent
+ path that scripts/jacoco/generate_merged_coverage_report.sh
globs for. -->
<plugin>
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
