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

🐛 CI should pass when on codecov upload fail. #396

Conversation

xuezhaojun
Copy link
Member

@xuezhaojun xuezhaojun commented Apr 7, 2024

Summary

Codecov becomes because github api outage sometimes. Change to pass CI job when the codecov upload is failed.

Related issue(s)

codecov/codecov-action#926

@openshift-ci openshift-ci bot requested review from jnpacker and qiujian16 April 7, 2024 13:00
@xuezhaojun xuezhaojun changed the title 🐛 Retry on codecov upload. 🐛 Skip when on codecov upload fail. Apr 8, 2024
Signed-off-by: GitHub <noreply@github.com>
@xuezhaojun xuezhaojun changed the title 🐛 Skip when on codecov upload fail. 🐛 CI success when on codecov upload fail. Apr 8, 2024
@xuezhaojun xuezhaojun changed the title 🐛 CI success when on codecov upload fail. 🐛 CI should pass when on codecov upload fail. Apr 8, 2024
@xuezhaojun
Copy link
Member Author

/assign @qiujian16

@@ -69,7 +69,7 @@ jobs:
flags: unit
name: unit
verbose: true
fail_ci_if_error: true
fail_ci_if_error: false
Copy link
Member

Choose a reason for hiding this comment

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

add a TODO like change this back when we find a way to stabilize it?

Copy link
Member Author

Choose a reason for hiding this comment

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

That's a good point to discuss on, I prefer leave it be to false. And if codecov continously being unstable we transfer to another tool.

@qiujian16
Copy link
Member

/approve
/lgtm

@openshift-ci openshift-ci bot added the lgtm label Apr 8, 2024
Copy link
Contributor

openshift-ci bot commented Apr 8, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: qiujian16, xuezhaojun

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved label Apr 8, 2024
@openshift-merge-bot openshift-merge-bot bot merged commit 603a92c into open-cluster-management-io:main Apr 8, 2024
15 checks passed
@xuezhaojun xuezhaojun deleted the retry-on-codecov branch August 16, 2024 08:32
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