[GitHub] [skywalking] wu-sheng commented on a change in pull request #4510: [WIP] Rewrite the E2E test framework and tests

[GitBox <gi...@apache.org>]

wu-sheng commented on a change in pull request #4510: [WIP] Rewrite the E2E test framework and tests
URL: https://github.com/apache/skywalking/pull/4510#discussion_r392561244
 
 

 ##########
 File path: .github/workflows/e2e.yaml
 ##########
 @@ -24,125 +24,97 @@ on:
     tags:
       - 'v*'
 
+env:
+  SKIP_TEST: true
+  SW_TEST_ID: ${GITHUB_RUN_ID}
+  SW_LOG_DIR: ${HOME}/logs/${GITHUB_RUN_ID}/
+
 jobs:
-  Single:
+  Checkstyle:
+    name: Check Codes Style
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      # In the checkout@v2, it doesn't support git submodule. Execute the commands manually.
-      - name: checkout submodules
+      - name: Checkout Submodules
         shell: bash
         run: |
           git submodule sync --recursive
           git -c protocol.version=2 submodule update --init --force --recursive --depth=1
-      - name: Set environment
-        run: export MAVEN_OPTS='-Dmaven.repo.local=~/.m2/repository -XX:+TieredCompilation -XX:TieredStopAtLevel=1 -XX:+CMSClassUnloadingEnabled -XX:+UseConcMarkSweepGC -XX:-UseGCOverheadLimit -Xmx3g'
-      - name: Compile & Install Test Codes
-        run: |
-          ./mvnw --batch-mode checkstyle:check apache-rat:check
-          ./mvnw --batch-mode -Dcheckstyle.skip -Drat.skip -T2 -Dmaven.compile.fork -Dmaven.compiler.maxmem=3072 -DskipTests clean install
-          ./mvnw --batch-mode -f test/e2e/pom.xml -pl e2e-base clean install
-      - name: Single Node Tests(JDK8)
-        run: export E2E_VERSION=jdk8-1.5 && bash -x test/e2e/run.sh e2e-single-service
-      - name: Single Node Tests(InfluxDB/JDK8)
-        run: export E2E_VERSION=jdk8-1.5 && bash -x test/e2e/run.sh e2e-influxdb
-      - name: Single Node Tests(MySQL/JDK8)
-        run: export E2E_VERSION=jdk8-1.5 && bash -x test/e2e/run.sh e2e-mysql
-      - name: Single Node Tests(JDK9)
-        run: export E2E_VERSION=jdk9-1.5 && bash -x test/e2e/run.sh e2e-single-service
-      - name: Single Node Tests(JDK11)
-        run: export E2E_VERSION=jdk11-1.5 && bash -x test/e2e/run.sh e2e-single-service
-      - name: Single Node Tests(JDK12)
-        run: export E2E_VERSION=jdk12-1.5 && bash -x test/e2e/run.sh e2e-single-service
-      - name: Agent Reboot Tests(JDK8)
-        run: export E2E_VERSION=jdk8-1.5 && bash -x test/e2e/run.sh e2e-agent-reboot
-      - name: Agent Auth Tests(JDK8)
-        run: export E2E_VERSION=jdk8-1.5 && bash -x test/e2e/run.sh e2e-agent-auth
+      - name: Checkstyle
+        run: ./mvnw --batch-mode -f test/e2e/pom.xml -DskipTests checkstyle:check
 
-  Cluster:
+  FeaturesGroup1:
 
 Review comment:
   Once this name changed, we are facing the awkward situation. The PR can't be merged as we set that by the INFRA team. Could we consider to change it back?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services