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

Don't run robustness tests on user forks #17912

Closed
ivanvc opened this issue Apr 30, 2024 · 0 comments · Fixed by etcd-io/bbolt#742
Closed

Don't run robustness tests on user forks #17912

ivanvc opened this issue Apr 30, 2024 · 0 comments · Fixed by etcd-io/bbolt#742

Comments

@ivanvc
Copy link
Member

ivanvc commented Apr 30, 2024

What would you like to be added?

This is something I have spotted for a long time (and can potentially flood user notifications). The configuration for the robustness tests workflow asks for a runner that regular users don't have access to (and it's also trying to run the tests on actuated ARM64 runners, which regular users don't have access to).

We should run these tests only for the etcd-io/etcd (and etcd-io/bbolt) repository, as we have configured other runners.

Note I know that the plan is to eventually relocate all Robustness tests to the Prow infrastructure, but in the meantime, it floods with notifications, which decreases the contributor experience.

Why is this needed?

To save users from receiving notifications that jobs are failing due to timeout trying to secure a runner to do the job. Improve ContribEx.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

1 participant