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

Codecove has released beta version of Test Analytics feature #3996

Merged
merged 4 commits into from
Oct 21, 2024

Conversation

ggivo
Copy link
Contributor

@ggivo ggivo commented Oct 15, 2024

Codecove has released beta version of Test Analytics feature
more details can be found here

Since we are already integrated with Codecov (storing code coverage data) the idea is to try out if Test Analytics feature will provide visibility over flaky tests and stability of the tests/build. Hopefully, we will have a base to compare and identify regressions.

Changes (only in build actions)
- Add step to publish junit report files to codecov
- Add basic codecov.yml config (filter out files that are not junit reports)

ggivo and others added 3 commits October 15, 2024 11:59
Codecove has released beta version of Test Analytics feature
more details can be found [here|https://docs.codecov.com/docs/test-result-ingestion-beta]

Since we are already integrated with Codecov (storing code coverage data) the idea is to try out if Test Analytics feature will provide visibility over flaky tests and stability of the tests/build. Hopefully, we will have a base to compare and identify regressions.

Changes (only in build actions)
    - Add step to publish junit report files to codecove
    - Add basic codecov.yml config (filter out files that are not junit reports)

Conditionally upload junit reports only on nightly build & push to branch
@ggivo ggivo requested a review from uglide October 16, 2024 06:50
@sazzad16 sazzad16 merged commit f1750fa into redis:master Oct 21, 2024
5 checks passed
uses: codecov/test-results-action@v1
with:
fail_ci_if_error: false
files: ./target/surefire-reports/TEST*
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Folks, sorry for the late moment but aren't we going to miss the failsafe-reports this way?

./target/surefire-reports/TEST*
./target/failsafe-reports/TEST*

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tishun Do we use the failsafe plugin in Jedis? I might have missed it.

@ggivo ggivo deleted the codecov-test-result branch October 24, 2024 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants