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

github: bump timeout duration for acceptance tests #393

Merged
merged 1 commit into from
May 30, 2023

Conversation

lbajolet-hashicorp
Copy link
Contributor

Last run failed after ~3h runtime, which is dangerously close to the timeout previously set. While the logs don't show stack traces, it's possible they did timeout, so to highten their chances to finish the acutal tests, we bump the timeout to 5h, and raise the session duration for the token to 6h.

Last run failed after ~3h runtime, which is dangerously close to the
timeout previously set. While the logs don't show stack traces, it's
possible they did timeout, so to highten their chances to finish the
acutal tests, we bump the timeout to 5h, and raise the session duration
for the token to 6h.
@lbajolet-hashicorp lbajolet-hashicorp requested a review from a team as a code owner May 29, 2023 13:24
Copy link
Contributor

@nywilken nywilken left a comment

Choose a reason for hiding this comment

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

LGTM

@nywilken nywilken merged commit 5904169 into main May 30, 2023
@nywilken nywilken deleted the bump_acc_test_timeout branch May 30, 2023 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants