-
Notifications
You must be signed in to change notification settings - Fork 44
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
Modified e2e regression tests to reproduce only one bug #275
Conversation
The change looks good, just a minor comment on naming the tests, as the first step to make the per-PR bug reproduction CI work again. I think we should explicitly name the tests as bug reproduction tests, because there are other regression tests in the repo that are not doing bug reproduction. We will add a weekly action to reproduce all bugs after this. |
I think the |
@ThrivikramanV Do you know what is causing the e2e test to be canceled? |
I had manually canceled it since I assumed that it would be running the original regressions (reproducing all the bugs) until we merge this PR with main. Is this not the case? |
I think it would run against the PR branch, which contains the updated github actions. |
set 'fork' as the default process start method
The e2e regression tests on every PR were reproducing all the bugs and hence consuming a lot of time. Have modified it to reproduce only one bug (chosen at random).