This is an automated email from the ASF dual-hosted git repository. jshao pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/main by this push: new 6180bc650 [#4032] Fix(CI): Remove some CI actions to make CI back to normal (#4030) 6180bc650 is described below commit 6180bc650aec6847d625a8551b9a54c0a913eb28 Author: Qi Yu <y...@datastrato.com> AuthorDate: Wed Jul 3 11:37:35 2024 +0800 [#4032] Fix(CI): Remove some CI actions to make CI back to normal (#4030) ### What changes were proposed in this pull request? Remove `csexton/debugger-action@master` action in the github CI, if someone wants this functionality, they can add it in his fork repo and do the debug work the personal account. ### Why are the changes needed? This action is not allowed in the Apache project. Fixed: #4032 ### Does this PR introduce _any_ user-facing change? N/A. ### How was this patch tested? Test in the CI --- .github/workflows/backend-integration-test.yml | 4 ---- .github/workflows/build.yml | 18 ------------------ .github/workflows/cron-integration-test.yml | 4 ---- .github/workflows/flink-integration-test.yml | 4 ---- .github/workflows/frontend-integration-test.yml | 4 ---- .github/workflows/spark-integration-test.yml | 4 ---- docs/how-to-test.md | 1 - .../integration/test/web/ui/MetalakePageTest.java | 5 ++--- 8 files changed, 2 insertions(+), 42 deletions(-) diff --git a/.github/workflows/backend-integration-test.yml b/.github/workflows/backend-integration-test.yml index aa564ac8d..da534f467 100644 --- a/.github/workflows/backend-integration-test.yml +++ b/.github/workflows/backend-integration-test.yml @@ -85,10 +85,6 @@ jobs: run: | ./gradlew compileDistribution -x test -PjdkVersion=${{ matrix.java-version }} - - name: Setup debug Github Action - if: ${{ contains(github.event.pull_request.labels.*.name, 'debug action') }} - uses: csexton/debugger-action@master - - name: Free up disk space run: | dev/ci/util_free_space.sh diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index e6a22cf35..cbb9eaffb 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -107,21 +107,3 @@ jobs: build/reports catalogs/**/*.log catalogs/**/*.tar - - - name: Jacoco Report to PR - id: jacoco - uses: madrapps/jacoco-report@v1.6.1 - with: - paths: ${{ github.workspace }}/**/build/reports/jacoco/test/jacocoTestReport.xml - token: ${{ secrets.GITHUB_TOKEN }} - min-coverage-overall: 40 - min-coverage-changed-files: 60 - title: 'Code Coverage Report' - debug-mode: false - update-comment: true - pass-emoji: ':green_circle:' - fail-emoji: ':red_circle:' - - name: Get the Coverage info - run: | - echo "Total coverage ${{ steps.jacoco.outputs.coverage-overall }}" - echo "Changed Files coverage ${{ steps.jacoco.outputs.coverage-changed-files }}" diff --git a/.github/workflows/cron-integration-test.yml b/.github/workflows/cron-integration-test.yml index 4a7c498ee..195e1b6e9 100644 --- a/.github/workflows/cron-integration-test.yml +++ b/.github/workflows/cron-integration-test.yml @@ -76,10 +76,6 @@ jobs: run: | ./gradlew compileDistribution -x test -PjdkVersion=${{ matrix.java-version }} - - name: Setup debug Github Action - if: ${{ contains(github.event.pull_request.labels.*.name, 'debug action') }} - uses: csexton/debugger-action@master - - name: Free up disk space run: | dev/ci/util_free_space.sh diff --git a/.github/workflows/flink-integration-test.yml b/.github/workflows/flink-integration-test.yml index 54141c842..ba7648c72 100644 --- a/.github/workflows/flink-integration-test.yml +++ b/.github/workflows/flink-integration-test.yml @@ -79,10 +79,6 @@ jobs: run: | ./gradlew compileDistribution -x test -PjdkVersion=${{ matrix.java-version }} - - name: Setup debug Github Action - if: ${{ contains(github.event.pull_request.labels.*.name, 'debug action') }} - uses: csexton/debugger-action@master - - name: Free up disk space run: | dev/ci/util_free_space.sh diff --git a/.github/workflows/frontend-integration-test.yml b/.github/workflows/frontend-integration-test.yml index e8925afb7..7b0315e46 100644 --- a/.github/workflows/frontend-integration-test.yml +++ b/.github/workflows/frontend-integration-test.yml @@ -82,10 +82,6 @@ jobs: run: | ./gradlew compileDistribution -x test -PjdkVersion=${{ matrix.java-version }} - - name: Setup debug Github Action - if: ${{ contains(github.event.pull_request.labels.*.name, 'debug action') }} - uses: csexton/debugger-action@master - - name: Free up disk space run: | dev/ci/util_free_space.sh diff --git a/.github/workflows/spark-integration-test.yml b/.github/workflows/spark-integration-test.yml index c730942e2..06dd385af 100644 --- a/.github/workflows/spark-integration-test.yml +++ b/.github/workflows/spark-integration-test.yml @@ -82,10 +82,6 @@ jobs: run: | ./gradlew compileDistribution -x test -PjdkVersion=${{ matrix.java-version }} - - name: Setup debug Github Action - if: ${{ contains(github.event.pull_request.labels.*.name, 'debug action') }} - uses: csexton/debugger-action@master - - name: Free up disk space run: | dev/ci/util_free_space.sh diff --git a/docs/how-to-test.md b/docs/how-to-test.md index 16a3e605c..460f9d155 100644 --- a/docs/how-to-test.md +++ b/docs/how-to-test.md @@ -154,7 +154,6 @@ This mode is closer to the actual environment, but more complex to debug. To deb * View the test results in the `Actions` tab of the pull request page. * Run the integration tests in several steps: * The Gravitino integration tests pull the CI Docker image from the Docker Hub repository. This step typically takes around 15 seconds. - * If you set the `debug action` label in the pull request, GitHub actions runs an SSH server with `csexton/debugger-action@master`, allowing you to log into the GitHub actions environment for remote debugging. * The Gravitino project compiles and packages in the `distribution` directory using the `./gradlew compileDistribution` command. * Run the `./gradlew test -PtestMode=[embedded|deploy]` command. diff --git a/integration-test/src/test/java/com/datastrato/gravitino/integration/test/web/ui/MetalakePageTest.java b/integration-test/src/test/java/com/datastrato/gravitino/integration/test/web/ui/MetalakePageTest.java index f99552839..7c92962a2 100644 --- a/integration-test/src/test/java/com/datastrato/gravitino/integration/test/web/ui/MetalakePageTest.java +++ b/integration-test/src/test/java/com/datastrato/gravitino/integration/test/web/ui/MetalakePageTest.java @@ -22,9 +22,8 @@ public class MetalakePageTest extends AbstractWebIT { private static final String FOOTER_LINK_DATASTRATO = "https://datastrato.ai/"; private static final String FOOTER_LINK_DOCS = "https://datastrato.ai/docs/"; private static final String FOOTER_LINK_LICENSE = - "https://github.com/datastrato/gravitino/blob/main/LICENSE"; - private static final String FOOTER_LINK_SUPPORT = - "https://github.com/datastrato/gravitino/issues"; + "https://github.com/apache/gravitino/blob/main/LICENSE"; + private static final String FOOTER_LINK_SUPPORT = "https://github.com/apache/gravitino/issues"; MetalakePage metalakePage = new MetalakePage(); // Create a metalake by name, set the default comment and properties.