Merge pull request #1132 from opensearch-project/ci_pull_request_target

GHA trigger CI with pull_request_target for secrets access
opensearch-project · Nov 13, 2024 · c434c64 · c434c64
2 parents d7884e9 + 75fafba
commit c434c64
Showing 1 changed file with 24 additions and 34 deletions.
diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml
@@ -2,7 +2,7 @@ name: CI
 
 on:
   push:
-  pull_request:
+  pull_request_target:
 
 env:
   python-version: '3.11'
@@ -63,17 +63,14 @@ jobs:
           pipenv install --deploy --dev
           pipenv run test
           pipenv run coverage xml
-      - name: Upload Coverage with retry
-        uses: Wandalen/wretry.action@v3.7.2
+      - name: Upload Coverage
+        uses: codecov/codecov-action@v4
         with:
-          attempt_limit: 5
-          attempt_delay: 2000
-          action: codecov/codecov-action@v4
-          with: |
-            token: ${{ secrets.CODECOV_TOKEN }}
-            files: ./coverage.xml
-            flags: python-test
-            fail_ci_if_error: true
+          fail_ci_if_error: true
+          files: ./coverage.xml
+          flags: python-test
+          token: ${{ secrets.CODECOV_TOKEN }}
+          verbose: true
 
   console-api-tests:
     runs-on: ubuntu-latest
@@ -90,17 +87,14 @@ jobs:
           pipenv install --deploy --dev
           pipenv run coverage run --source='.' manage.py test console_api
           pipenv run coverage xml
-      - name: Upload Coverage with retry
-        uses: Wandalen/wretry.action@v3.7.2
+      - name: Upload Coverage
+        uses: codecov/codecov-action@v4
         with:
-          attempt_limit: 5
-          attempt_delay: 2000
-          action: codecov/codecov-action@v4
-          with: |
-            token: ${{ secrets.CODECOV_TOKEN }}
-            files: ./coverage.xml
-            flags: python-test
-            fail_ci_if_error: true
+          fail_ci_if_error: true
+          files: ./coverage.xml
+          flags: python-test
+          token: ${{ secrets.CODECOV_TOKEN }}
+          verbose: true
 
 
   gradle-tests:
@@ -132,19 +126,15 @@ jobs:
           path: |
             **/build/reports/tests/
             **/reports/jacoco/mergedReport/
-      - name: Upload Coverage with retry
-        uses: Wandalen/wretry.action@v3.7.2
-        with:
-          attempt_limit: 5
-          attempt_delay: 2000
-          action: codecov/codecov-action@v4
-          with: |
-            token: ${{ secrets.CODECOV_TOKEN }}
-            files: ${{ github.workspace }}/build/reports/jacoco/mergedReport/jacocoMergedReport.xml
-            flags: gradle-test
-            fail_ci_if_error: true
-            disable_search: true
-
+      - name: Upload Coverage
+        uses: codecov/codecov-action@v4
+        with:
+          disable_search: true
+          fail_ci_if_error: true
+          files: ${{ github.workspace }}/build/reports/jacoco/mergedReport/jacocoMergedReport.xml
+          flags: gradle-test
+          token: ${{ secrets.CODECOV_TOKEN }}
+          verbose: true
 
 
   python-e2e-tests: