[WIP] - Add periodic Codecov coverage job for kueue-operator#74836
[WIP] - Add periodic Codecov coverage job for kueue-operator#74836anahas-redhat wants to merge 1 commit intoopenshift:mainfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: anahas-redhat The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Configure a periodic coverage test for the kueue-operator main branch that runs every 96 hours (4 days) and uploads results to Codecov. Changes: - Add 'coverage' test to ci-operator config with minimum_interval of 96h - Use kueue-operator-codecov-token secret for Codecov authentication - Fix indentation of coverage test fields to match repo conventions - Remove duplicate coverage definition and unused postsubmit job - Generate corresponding Prow periodic job definition
e207489 to
3c5d2ec
Compare
|
[REHEARSALNOTIFIER]
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
@anahas-redhat: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
| minimum_interval: 96h | ||
| secret: | ||
| mount_path: /tmp/secret | ||
| name: kueue-operator-codecov-token |
There was a problem hiding this comment.
So if this is intended as a periodic job, it would probably make sense to replace the job "test" (on line 60) with this, which adds that extra step to upload the coverage to codecov in PR.
not required though, just a recommendation.
Following the initial Codecov onboarding #1446, this PR implements the second phase of the integration. The goal is to establish a consistent baseline of coverage data for the main branch.
Proposed Changes
Dependencies
Related PRs: