Skip to content

Conversation

@v1v
Copy link
Member

@v1v v1v commented Apr 1, 2022

Trying to enrich the CI with the source integration as stated in #585 (comment)

Test

If you go to here

Then you will see the code coverage.

There are some files that are not found, so no idea about the intrinsic behaviour for the codecoverage either for this project or in the CI.

Logs can be found here

Or see the below extract from the entire log output while using the new coverage step in the CI.

[2022-04-01T14:00:41.334Z] Copied cmd/promote.go. 
[2022-04-01T14:00:41.334Z] Starting copy source file internal/testrunner/testrunner.go. 
[2022-04-01T14:00:41.354Z] Copied internal/testrunner/testrunner.go. 
[2022-04-01T14:00:41.354Z] Starting copy source file internal/profile/profile.go. 
[2022-04-01T14:00:41.369Z] Copied internal/profile/profile.go. 
[2022-04-01T14:00:41.369Z] Starting copy source file multiinput/test. 
[2022-04-01T14:00:41.381Z] Unable to find source file multiinput/test in workspace /var/lib/jenkins/workspace/t-manager_elastic-package_PR-775/src/github.com/elastic/elastic-package
[2022-04-01T14:00:41.381Z] Starting copy source file apache/status. 
[2022-04-01T14:00:41.392Z] Unable to find source file apache/status in workspace /var/lib/jenkins/workspace/t-manager_elastic-package_PR-775/src/github.com/elastic/elastic-package

image

image

@elasticmachine
Copy link
Collaborator

elasticmachine commented Apr 1, 2022

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-04-05T08:44:10.951+0000

  • Duration: 32 min 50 sec

Test stats 🧪

Test Results
Failed 0
Passed 628
Skipped 0
Total 628

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@v1v v1v requested review from a team and adriansr April 1, 2022 14:05
@v1v v1v self-assigned this Apr 1, 2022
@v1v v1v added the automation label Apr 1, 2022
@v1v v1v marked this pull request as ready for review April 1, 2022 14:05
@mtojek
Copy link
Contributor

mtojek commented Apr 5, 2022

/test

@v1v
Copy link
Member Author

v1v commented Apr 5, 2022

I'll close this PR since https://github.com/elastic/apm-pipeline-library/releases/tag/v1.1.313 is now supporting this feature implicitly. The coverageReport step is the one now driving this

@v1v v1v closed this Apr 5, 2022
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.

4 participants