This is an automated email from the ASF dual-hosted git repository.
SbloodyS pushed a commit to branch dev
in repository https://gitbox.apache.org/repos/asf/dolphinscheduler.git
The following commit(s) were added to refs/heads/dev by this push:
new 2ed0975b79 [Chore] Remove sonar check (#18374)
2ed0975b79 is described below
commit 2ed0975b793356ebbc3ccc15bfd8c16a8e719b73
Author: xiangzihao <[email protected]>
AuthorDate: Tue Jun 23 11:28:34 2026 +0800
[Chore] Remove sonar check (#18374)
---
.github/workflows/api-test.yml | 1 +
.github/workflows/backend.yml | 1 +
.github/workflows/codeql.yaml | 1 +
.github/workflows/docs.yml | 1 +
.github/workflows/e2e-k8s.yml | 1 +
.github/workflows/e2e.yml | 1 +
.github/workflows/frontend.yml | 1 +
.github/workflows/issue-robot.yml | 1 +
.github/workflows/mergeable.yml | 1 +
.github/workflows/owasp-dependency-check.yaml | 1 +
.github/workflows/publish-docker.yaml | 1 +
.github/workflows/publish-helm-chart.yaml | 1 +
.github/workflows/pull-request-target-robot.yml | 3 +-
.github/workflows/stale.yml | 1 +
.github/workflows/unit-test.yml | 137 +-----------------------
15 files changed, 19 insertions(+), 134 deletions(-)
diff --git a/.github/workflows/api-test.yml b/.github/workflows/api-test.yml
index 20e9eb95ab..2de3985523 100644
--- a/.github/workflows/api-test.yml
+++ b/.github/workflows/api-test.yml
@@ -16,6 +16,7 @@
#
on:
+ workflow_dispatch:
pull_request:
push:
branches:
diff --git a/.github/workflows/backend.yml b/.github/workflows/backend.yml
index 08f8769b64..969e3165e0 100644
--- a/.github/workflows/backend.yml
+++ b/.github/workflows/backend.yml
@@ -18,6 +18,7 @@
name: Backend
on:
+ workflow_dispatch:
push:
branches:
- dev
diff --git a/.github/workflows/codeql.yaml b/.github/workflows/codeql.yaml
index 926b43a1eb..8125a0108c 100644
--- a/.github/workflows/codeql.yaml
+++ b/.github/workflows/codeql.yaml
@@ -17,6 +17,7 @@
name: "CodeQL"
on:
+ workflow_dispatch:
push:
branches:
- 'dev'
diff --git a/.github/workflows/docs.yml b/.github/workflows/docs.yml
index f9f2ca2d7d..028ec735da 100644
--- a/.github/workflows/docs.yml
+++ b/.github/workflows/docs.yml
@@ -17,6 +17,7 @@
name: Docs
on:
+ workflow_dispatch:
pull_request:
schedule:
- cron: '0 18 * * *' # TimeZone: UTC 0
diff --git a/.github/workflows/e2e-k8s.yml b/.github/workflows/e2e-k8s.yml
index f3d3835c47..d8ce37399f 100644
--- a/.github/workflows/e2e-k8s.yml
+++ b/.github/workflows/e2e-k8s.yml
@@ -16,6 +16,7 @@
#
on:
+ workflow_dispatch:
pull_request:
push:
branches:
diff --git a/.github/workflows/e2e.yml b/.github/workflows/e2e.yml
index 448a4346f9..9ce4eb9f3c 100644
--- a/.github/workflows/e2e.yml
+++ b/.github/workflows/e2e.yml
@@ -16,6 +16,7 @@
#
on:
+ workflow_dispatch:
pull_request:
push:
branches:
diff --git a/.github/workflows/frontend.yml b/.github/workflows/frontend.yml
index 50b1ff4386..e7e75a9209 100644
--- a/.github/workflows/frontend.yml
+++ b/.github/workflows/frontend.yml
@@ -18,6 +18,7 @@
name: Frontend
on:
+ workflow_dispatch:
push:
branches:
- '[0-9]+.[0-9]+.[0-9]+-prepare'
diff --git a/.github/workflows/issue-robot.yml
b/.github/workflows/issue-robot.yml
index 01b2fb92bd..3c28c9f9bf 100644
--- a/.github/workflows/issue-robot.yml
+++ b/.github/workflows/issue-robot.yml
@@ -18,6 +18,7 @@
name: issue-robot
on:
+ workflow_dispatch:
issues:
types: [opened]
diff --git a/.github/workflows/mergeable.yml b/.github/workflows/mergeable.yml
index 835b34bf57..9c0094df00 100644
--- a/.github/workflows/mergeable.yml
+++ b/.github/workflows/mergeable.yml
@@ -16,6 +16,7 @@
# under the License.
on:
+ workflow_dispatch:
pull_request:
types:
- opened
diff --git a/.github/workflows/owasp-dependency-check.yaml
b/.github/workflows/owasp-dependency-check.yaml
index 4069628e69..4d9fa11321 100644
--- a/.github/workflows/owasp-dependency-check.yaml
+++ b/.github/workflows/owasp-dependency-check.yaml
@@ -18,6 +18,7 @@
name: OWASP Dependency Check
on:
+ workflow_dispatch:
push:
branches:
- '[0-9]+.[0-9]+.[0-9]+-prepare'
diff --git a/.github/workflows/publish-docker.yaml
b/.github/workflows/publish-docker.yaml
index 13df72c421..4b08bedc5c 100644
--- a/.github/workflows/publish-docker.yaml
+++ b/.github/workflows/publish-docker.yaml
@@ -17,6 +17,7 @@
name: publish-docker
on:
+ workflow_dispatch:
push:
branches:
- dev
diff --git a/.github/workflows/publish-helm-chart.yaml
b/.github/workflows/publish-helm-chart.yaml
index d4e39fd2c0..55522ef45d 100644
--- a/.github/workflows/publish-helm-chart.yaml
+++ b/.github/workflows/publish-helm-chart.yaml
@@ -17,6 +17,7 @@
name: publish-helm-chart
on:
+ workflow_dispatch:
push:
branches:
- dev
diff --git a/.github/workflows/pull-request-target-robot.yml
b/.github/workflows/pull-request-target-robot.yml
index c934e0ba1d..7697be990b 100644
--- a/.github/workflows/pull-request-target-robot.yml
+++ b/.github/workflows/pull-request-target-robot.yml
@@ -18,6 +18,7 @@
name: "pull-request-target-robot"
on:
+ workflow_dispatch:
pull_request_target:
jobs:
@@ -35,7 +36,7 @@ jobs:
submodules: true
- name: "Label in pull request"
- uses: actions/labeler@v4
+ uses: actions/labeler@v6
with:
repo-token: "${{ secrets.GITHUB_TOKEN }}"
configuration-path: .github/actions/labeler/labeler.yml
diff --git a/.github/workflows/stale.yml b/.github/workflows/stale.yml
index fc14d98b3b..5ba58381c3 100644
--- a/.github/workflows/stale.yml
+++ b/.github/workflows/stale.yml
@@ -17,6 +17,7 @@
# https://github.com/actions/stale
name: 'Close stale issues and PRs'
on:
+ workflow_dispatch:
schedule:
- cron: '0 0 * * *'
permissions:
diff --git a/.github/workflows/unit-test.yml b/.github/workflows/unit-test.yml
index ab78cfe56e..2117250026 100644
--- a/.github/workflows/unit-test.yml
+++ b/.github/workflows/unit-test.yml
@@ -18,15 +18,8 @@
name: Test
on:
+ workflow_dispatch:
pull_request:
- push:
- paths-ignore:
- - '**/*.md'
- - 'dolphinscheduler-ui'
- branches:
- - '[0-9]+.[0-9]+.[0-9]+-prepare'
- - '[0-9]+.[0-9]+.[0-9]+-release'
- - 'dev'
concurrency:
group: unit-test-${{ github.event.pull_request.number || github.ref }}
@@ -53,7 +46,7 @@ jobs:
name: Sanity Check
runs-on: ubuntu-latest
needs: paths-filter
- if: ${{ needs.paths-filter.outputs.not-ignore == 'true' ||
github.event_name == 'push' }}
+ if: ${{ needs.paths-filter.outputs.not-ignore == 'true' }}
steps:
- uses: actions/checkout@v6
with:
@@ -67,7 +60,7 @@ jobs:
name: Generate Module Matrix
runs-on: ubuntu-latest
needs: paths-filter
- if: ${{ needs.paths-filter.outputs.not-ignore == 'true' ||
github.event_name == 'push' }}
+ if: ${{ needs.paths-filter.outputs.not-ignore == 'true' }}
outputs:
matrix: ${{ steps.set-matrix.outputs.matrix }}
steps:
@@ -101,7 +94,7 @@ jobs:
unit-test:
name: Unit-Test (${{ matrix.module }} | Java ${{ matrix.java }})
needs: [ paths-filter, generate-matrix, sanity-check ]
- if: ${{ (needs.paths-filter.outputs.not-ignore == 'true') ||
(github.event_name == 'push') }}
+ if: ${{ needs.paths-filter.outputs.not-ignore == 'true' }}
runs-on: ubuntu-latest
strategy:
matrix:
@@ -160,25 +153,6 @@ jobs:
-Dsurefire.reportFormat=plain \
-Dsurefire.redirectTestOutputToFile=false
- - name: Collect jacoco exec files
- if: always()
- run: |
- mkdir -p /tmp/jacoco-exec
- find . -path "*/target/*.exec" -type f | while IFS= read -r
exec_file; do
- target_file="/tmp/jacoco-exec/${exec_file#./}"
- mkdir -p "$(dirname "$target_file")"
- cp "$exec_file" "$target_file"
- done
- find /tmp/jacoco-exec -name "*.exec" -print
-
- - name: Upload jacoco exec
- uses: actions/upload-artifact@v6
- if: always()
- with:
- name: jacoco-exec-java${{ matrix.java }}-${{ matrix.module }}
- path: /tmp/jacoco-exec/
- retention-days: 1
-
- name: Upload surefire reports
uses: actions/upload-artifact@v6
if: always()
@@ -187,109 +161,6 @@ jobs:
path: '**/target/surefire-reports/'
retention-days: 7
- - name: Upload coverage to Codecov
- if: matrix.java == '11'
- run: CODECOV_TOKEN="09c2663f-b091-4258-8a47-c981827eb29a" bash <(curl
-s https://codecov.io/bash)
-
- sonar:
- name: SonarCloud Analysis
- runs-on: ubuntu-latest
- needs: [ paths-filter, unit-test ]
- if: ${{ needs.unit-test.result == 'success' }}
- timeout-minutes: 30
- steps:
- - uses: actions/checkout@v6
- with:
- submodules: true
- fetch-depth: 0
-
- - name: Download all jacoco exec files (Java 11)
- uses: actions/download-artifact@v8
- with:
- pattern: jacoco-exec-java11-*
- path: all-jacoco-exec/
- merge-multiple: true
-
- - name: Set up JDK 17 for SonarCloud
- uses: actions/setup-java@v5
- with:
- java-version: 17
- distribution: 'adopt'
-
- - uses: actions/cache@v5
- with:
- path: ~/.m2/repository
- key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}-backend
- restore-keys: ${{ runner.os }}-maven-
-
- # Use jacococli.jar instead of jacoco:merge Maven goal.
- # The 'fileSets' parameter of jacoco:merge cannot be passed via -D on
- # the command line; it must be declared in pom.xml <configuration>.
- # jacococli.jar merge accepts file paths directly, no XML config needed.
- - name: Download jacococli.jar
- run: |
- JACOCO_VERSION=$(./mvnw help:evaluate -Dexpression=jacoco.version -q
-DforceStdout 2>/dev/null || echo "0.8.14")
- echo "Using JaCoCo version: $JACOCO_VERSION"
- mvn dependency:get \
- -Dartifact=org.jacoco:org.jacoco.cli:${JACOCO_VERSION}:jar:nodeps \
- -Ddest=/tmp/jacococli.jar \
- -q
- # Fallback: copy from local .m2 cache if dependency:get did not
place it at -Ddest
- find ~/.m2 -name "org.jacoco.cli-*-nodeps.jar" | head -1 | xargs
-I{} cp {} /tmp/jacococli.jar || true
-
- - name: Merge JaCoCo exec files via jacococli
- run: |
- # Collect all .exec files under the downloaded artifact directory
- EXEC_FILES=$(find all-jacoco-exec/ -name "*.exec" | tr '\n' ' ')
- echo "Merging exec files: $EXEC_FILES"
- mkdir -p target
- java -jar /tmp/jacococli.jar merge $EXEC_FILES \
- --destfile target/merged.exec
-
- - name: Generate merged JaCoCo XML report
- run: |
- ./mvnw jacoco:report \
- -Djacoco.dataFile=target/merged.exec \
- -Dspotless.skip=true \
- -Dmaven.test.skip=true
-
- - name: Run SonarCloud Analysis
- run: >
- ./mvnw --batch-mode verify sonar:sonar
- -Dsonar.coverage.jacoco.xmlReportPaths=target/site/jacoco/jacoco.xml
- -Dmaven.test.skip=true
- -Dspotless.skip=true
- -Dsonar.host.url=https://sonarcloud.io
- -Dsonar.organization=apache
- -Dsonar.core.codeCoveragePlugin=jacoco
- -Dsonar.projectKey=apache-dolphinscheduler
- -Dsonar.token=${{ secrets.SONARCLOUD_TOKEN }}
-
-Dsonar.exclusions=dolphinscheduler-ui/src/**/i18n/locale/*.js,dolphinscheduler-microbench/src/**/*
- -Dhttp.keepAlive=false
- -Dmaven.wagon.http.pool=false
- -Dmaven.wagon.httpconnectionManager.ttlSeconds=120
- -DskipUT=true
- -Danalyze.skip=true
-
- publish-test-results:
- name: Publish Test Results
- runs-on: ubuntu-latest
- needs: [ paths-filter, unit-test ]
- if: ${{ needs.unit-test.result == 'success' }}
- steps:
- - name: Download all surefire reports
- uses: actions/download-artifact@v8
- with:
- pattern: surefire-java11-*
- path: all-surefire/
- merge-multiple: true
-
- - name: Publish Unit Test Results
- uses: EnricoMi/publish-unit-test-result-action@v2
- with:
- files: all-surefire/**/*.xml
- comment_mode: off
-
result:
name: Unit Test
runs-on: ubuntu-latest
