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

Switch to running the Github Action test workflow every 3 hours in single thread mode to observe if Sementation Faults occur #1022

Merged
merged 3 commits into from
Feb 25, 2021

Conversation

valeriupredoi
Copy link
Contributor

I am trying to gather enough statistic to prove that if we run the test in single thread mode we will not see the Segmentation Faults we've been seeing for nearly a year now. So to do that I want to have the GA run the test only workflow every three hours; unfortunately it's not possible to have this done on a branch, it has to be the main branch, but we can let this go for a few days, gather enough statistic then revert to the nightly run as we had it so far. If this is true then we can do what @stefsmeets suggests in this comment and hopefully rid ourselves of those pesty SegFaults for good. Please see logic behind this PR in #1020

Copy link
Contributor

@stefsmeets stefsmeets left a comment

Choose a reason for hiding this comment

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

Looks fine to me! 🚀

Comment on lines +31 to +32
# - cron: '0 0 * * *' # nightly
- cron: '0 */3 * * *' # every 3 hours
Copy link
Contributor

Choose a reason for hiding this comment

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

Are you going to change this back? Otherwise I'd suggest to just replace the line:

Suggested change
# - cron: '0 0 * * *' # nightly
- cron: '0 */3 * * *' # every 3 hours
- cron: '0 */3 * * *' # every 3 hours

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah no, gonna change it back, we really don't want to run that test every 3h for the rest of our time 😁

@valeriupredoi
Copy link
Contributor Author

cheers @stefsmeets 🍺

@valeriupredoi valeriupredoi merged commit fee8d93 into master Feb 25, 2021
@valeriupredoi valeriupredoi deleted the run_test_hourly branch February 25, 2021 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants