Merge pull request #2467 from opencb/TASK-4970

TASK-4970 - New CI/CD should be triggered when a PR is approved for OpenCGA repository

.github/workflows/develop.yml

@@ -13,23 +13,24 @@ jobs:
     with:
       maven_opts: -Phdp3.1,RClient -Dopencga.war.name=opencga -Dcheckstyle.skip
 
-  test:
-    uses: ./.github/workflows/test-analysis.yml
-    needs: build
-    with:
-      test_profile: runShortTests,runMediumTests
-    secrets: inherit
+## TASK-4970: remove the test job from the develop.yml workflow because it is not needed anymore.
+#  test:
+#    uses: ./.github/workflows/test-analysis.yml
+#    needs: build
+#    with:
+#      test_profile: runShortTests,runMediumTests
+#    secrets: inherit
 
   deploy-maven:
     uses: opencb/java-common-libs/.github/workflows/deploy-maven-repository-workflow.yml@develop
-    needs: test
+    needs: build
     with:
       maven_opts: -P hdp3.1 -Dopencga.war.name=opencga
     secrets: inherit
 
   deploy-docker:
     uses: opencb/java-common-libs/.github/workflows/deploy-docker-hub-workflow.yml@develop
-    needs: test
+    needs: build
     with:
       cli: python3 ./build/cloud/docker/docker-build.py push --images base,init
     secrets: inherit

.github/workflows/long-test-analysis.yml

@@ -14,8 +14,8 @@ jobs:
       matrix:
         hadoop: [ "hdp3.1", "hdi5.1", "emr6.1", "emr6.13" ]
     uses: ./.github/workflows/test-analysis.yml
-    secrets: inherit
     with:
       test_profile: runShortTests,runMediumTests,runLongTests
       hadoop: ${{ matrix.hadoop }}
+    secrets: inherit
 

.github/workflows/manual-deploy-docker.yml

@@ -51,10 +51,9 @@ jobs:
           name: build-folder
           path: build
 
-
   deploy-docker:
     uses: opencb/java-common-libs/.github/workflows/deploy-docker-hub-workflow.yml@develop
     needs: build
     with:
       cli: python3 ./build/cloud/docker/docker-build.py push --images base,init --tag ${{ inputs.tag }}
-    secrets: inherit
+    secrets: inherit

.github/workflows/manual-deploy-ext-tools.yml

@@ -46,11 +46,9 @@ jobs:
           name: build-folder
           path: build
 
-
   deploy-docker-ext-tools:
     uses: opencb/java-common-libs/.github/workflows/deploy-docker-hub-workflow.yml@develop
     needs: build
     with:
       cli: python3 ./build/cloud/docker/docker-build.py push --images ext-tools --tag ${{ inputs.tag }}
     secrets: inherit
-

.github/workflows/pull-request-approved.yml

@@ -0,0 +1,19 @@
+name: Pull request approve workflow
+
+on:
+  pull_request_review:
+    types: [submitted]
+
+jobs:
+  build:
+    uses: opencb/java-common-libs/.github/workflows/build-java-app-workflow.yml@develop
+    with:
+      maven_opts: -Phdp3.1,RClient -Dopencga.war.name=opencga -Dcheckstyle.skip
+
+  test:
+    name: "Run all tests before merging, ie. short, medium and long tests."
+    uses: ./.github/workflows/test-analysis.yml
+    needs: build
+    with:
+      test_profile: runShortTests,runMediumTests,runLongTests
+    secrets: inherit

.github/workflows/task.yml

@@ -17,9 +17,9 @@ jobs:
   test:
     uses: ./.github/workflows/test-analysis.yml
     needs: build
-    secrets: inherit
     with:
       test_profile: runShortTests
+    secrets: inherit
 
   deploy-docker:
     uses: opencb/java-common-libs/.github/workflows/deploy-docker-hub-workflow.yml@develop

.github/workflows/test-analysis.yml

@@ -38,7 +38,7 @@ jobs:
     steps:
       - uses: actions/checkout@v4
         with:
-          fetch-depth: '0'
+          fetch-depth: '10'
       - name: Set up JDK 11
         uses: actions/setup-java@v4
         with:
@@ -65,7 +65,7 @@ jobs:
     steps:
       - uses: actions/checkout@v4
         with:
-          fetch-depth: '0'
+          fetch-depth: '10'
       - name: Set up JDK 8
         uses: actions/setup-java@v4
         with:
@@ -91,7 +91,7 @@ jobs:
         run: mvn -B clean install -DskipTests -P ${{ inputs.hadoop }} -Dcheckstyle.skip ${{ inputs.mvn_opts }}
       - name: Run Junit tests
         run: mvn -B verify surefire-report:report --fail-never -f ${{ (inputs.module == '' || inputs.module == 'all') && '.' || inputs.module }} -P ${{ inputs.hadoop }},${{ inputs.test_profile }} -Dcheckstyle.skip ${{ inputs.mvn_opts }}
-      - name: Publish Test Report
+      - name: Publish Test Report on GitHub
         uses: scacap/action-surefire-report@v1
         env:
           NODE_OPTIONS: '--max_old_space_size=4096'