Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update codecov to v5 with new token #5280

Merged
4 changes: 0 additions & 4 deletions .github/workflows/groovy-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,3 @@ jobs:
- name: Run Tests
run: |
./gradlew test --info
- name: Upload Coverage Report
uses: codecov/codecov-action@v2
with:
files: ./build/reports/jacoco/test/jacocoTestReport.xml
peterzhuamazon marked this conversation as resolved.
Show resolved Hide resolved
23 changes: 21 additions & 2 deletions .github/workflows/python-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ on: [push, pull_request]

jobs:
Get-CI-Image-Tag:
uses: ./.github/workflows/get-ci-image-tag.yml
uses: opensearch-project/opensearch-build/.github/workflows/get-ci-image-tag.yml@main
with:
product: opensearch

Expand Down Expand Up @@ -76,7 +76,26 @@ jobs:
- name: Run Tests with Coverage
run: |
pipenv run coverage run -m pytest --cov=./src --cov-report=xml
- uses: actions/upload-artifact@v4
with:
name: coverage-report-${{ matrix.os }}
path: ./coverage.xml

python-tests-codecov:
needs: python-tests-linux
strategy:
matrix:
os:
- ubuntu-24.04-arm # arm64-preview
- ubuntu-24.04 # x64
runs-on: ${{ matrix.os }}
steps:
- uses: actions/download-artifact@v4
peterzhuamazon marked this conversation as resolved.
Show resolved Hide resolved
with:
name: coverage-report-${{ matrix.os }}
path: ./coverage.xml
- name: Upload Coverage Report
uses: codecov/codecov-action@v2
uses: codecov/codecov-action@v5
with:
token: ${{ secrets.CODECOV_TOKEN }}
files: ./coverage.xml