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

add timeouts to tests github action. #911

Merged
merged 1 commit into from
Aug 10, 2023

Conversation

nihohit
Copy link
Contributor

@nihohit nihohit commented Jul 31, 2023

This adds ~10 minutes to previous test run time, in order to leave a comfortable margin, while still significantly reducing the run time of the task in case it gets stuck. Without defining this, the task runs for 6 hours.

@nihohit nihohit changed the title add timeouts to tests task. add timeouts to tests github action. Jul 31, 2023
This adds ~10 minutes to previous test run time, in order to leave a
comfortable margin, while still significantly reducing the run time of
the task in case it gets stuck. Without defining this, the task runs
for 6 hours.
@nihohit nihohit merged commit bd3cef5 into redis-rs:main Aug 10, 2023
10 checks passed
@nihohit nihohit deleted the timeout branch August 10, 2023 16:11
altanozlu pushed a commit to altanozlu/redis-rs that referenced this pull request Aug 16, 2023
This adds ~10 minutes to previous test run time, in order to leave a
comfortable margin, while still significantly reducing the run time of
the task in case it gets stuck. Without defining this, the task runs
for 6 hours.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants