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

Codecov handling #11

Closed
erikbosch opened this issue Apr 8, 2024 · 4 comments
Closed

Codecov handling #11

erikbosch opened this issue Apr 8, 2024 · 4 comments

Comments

@erikbosch
Copy link
Contributor

We use codecov, see https://github.com/eclipse-kuksa/kuksa-databroker/blob/main/.github/workflows/kuksa_databroker_build.yml

Some problems observed:

      - name: Upload coverage to Codecov
        # To use v4 a token must be specifed by "token: ${{ secrets.CODECOV_TOKEN }}""
        # Uploaded result available at https://app.codecov.io/gh/eclipse-kuksa/kuksa-databroker
        uses: codecov/codecov-action@v3
        with:
          files: lcov.info

      uses: codecov/codecov-action@v3
      with:
        files: ${{github.workspace}}/out/all_targets.info
        flags: unittests # optional
        name: codecov-umbrella # optional
        fail_ci_if_error: true # optional (default = false)
        verbose: true # optional (default = false)
@SebastianSchildt
Copy link
Contributor

I am not sure how Eclipse stance on Codecov is, but we are likely not the first project asking for this.

Someone taking this should make a ticket asking for a orga token here https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues

(after creating a ticket ping me or @bs-jokri so we could comment with a "virtual" project-lead thumbs up)

@erikbosch
Copy link
Contributor Author

erikbosch commented May 13, 2024

There have been projects before

https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues/4546

I actually do not know what Otterdog supports here, if you just can request/initiate it by adding it to https://github.com/eclipse-kuksa/.eclipsefdn/blob/main/otterdog/eclipse-kuksa.jsonnet

(Or if someone must add/generate they key manually anyway)

@erikbosch
Copy link
Contributor Author

Upgrade to new version fixed by #31

Upload worked for latest main, see https://app.codecov.io/github/eclipse-kuksa/kuksa-databroker/commits

But still we could discuss bit more how we want to track code coverage. With the current setup we cannot expect codecov to succeed for all pull requests (as it depends on that the source fork has a token or has luck). But even if it succeeds uploading - what do we want to happen if coverage is shrinking. Shall that be acceptable?

Leaving this issue open for now

@erikbosch
Copy link
Contributor Author

Think this one can be closed for now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants