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

[Enhancement] Improve unit test retry mechanism for PR submissions #8742

Closed
1 task done
chi3316 opened this issue Sep 24, 2024 · 0 comments · Fixed by #8741
Closed
1 task done

[Enhancement] Improve unit test retry mechanism for PR submissions #8742

chi3316 opened this issue Sep 24, 2024 · 0 comments · Fixed by #8741

Comments

@chi3316
Copy link
Contributor

chi3316 commented Sep 24, 2024

Before Creating the Enhancement Request

  • I have confirmed that this should be classified as an enhancement rather than a bug/feature.

Summary

I propose enhancing the retry mechanism to allow it to trigger automatically when submitting pull requests.

Motivation

This enhancement is necessary to improve the testing process and ensure that developers can easily rerun workflows when facing transient issues during PR submissions. It benefits the project by increasing the reliability of CI/CD pipelines.

Describe the Solution You'd Like

Implement a workflow that automatically triggers retries based on the workflow_run event. This would replace the current manual invocation of rerun workflows and eliminate permission issues that hinder the retry mechanism.

Describe Alternatives You've Considered

I considered keeping the existing manual retry mechanism, but it does not address the permission issues and is less user-friendly. The proposed solution offers a more streamlined approach.

Additional Context

No response

RongtongJin pushed a commit that referenced this issue Sep 25, 2024
…ission (#8741)

* Enable retry mechanism when submitting PR

* Update

* Remove gh run watch

* Fix bug

* Revert "Remove gh run watch"

This reverts commit cd5da59.

* Add 'Build and Run Tests by Bazel'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant