Skip to content

Commit

Permalink
use private action
Browse files Browse the repository at this point in the history
Signed-off-by: Shreyas Kirtane <[email protected]>
  • Loading branch information
skirtan1 committed Jan 24, 2025
1 parent b26cfca commit fa84792
Show file tree
Hide file tree
Showing 2 changed files with 37 additions and 12 deletions.
34 changes: 34 additions & 0 deletions .github/actions/upload-codecov/action.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
# Copyright (c) 2023 The Jaeger Authors.
# SPDX-License-Identifier: Apache-2.0

# Codecov upload often fails on rate limits if used without a token.
# See https://github.com/codecov/codecov-action/issues/837
# This action embeds a token directly.
# We cannot define it as "secret" as we need it accessible from forks.

name: 'Upload coverage to codecov'
description: 'Uploads coverage to codecov with retries'
inputs:
files:
description: 'Coverage files to upload'
required: true
flags:
description: 'Flags for codecov'
required: true
runs:
using: 'composite'
steps:
- name: Retry upload
uses: Wandalen/[email protected]
with:
attempt_limit: 6
# sleep 10 seconds between retries
attempt_delay: 10000
action: codecov/codecov-action@v5
with: |
slug: jaegertracing/jaeger-idl
files: ${{ inputs.files }}
flags: ${{ inputs.flags }}
verbose: true
fail_ci_if_error: true
token: 95fab3d9-8038-40ba-ac56-0d8bd91390a4
15 changes: 3 additions & 12 deletions .github/workflows/ci-unit-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,19 +36,10 @@ jobs:
run: make test-ci

- name: upload code coverage
uses: Wandalen/[email protected]
uses: ./.github/actions/upload-codecov
with:
attempt_limit: 6
# sleep 10 seconds between retries
attempt_delay: 10000
action: codecov/codecov-action@v5
with: |
slug: jaegertracing/jaeger-idl
files: coverage.txt
flags: unittests
verbose: true
fail_ci_if_error: true
token: 95fab3d9-8038-40ba-ac56-0d8bd91390a4
files: coverage.txt
flags: unittests

test-code-gen:
runs-on: ubuntu-latest
Expand Down

0 comments on commit fa84792

Please sign in to comment.