8000 Add timeout to scripted statements by eatkins · Pull Request #5151 · sbt/sbt · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add timeout to scripted statements #5151

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 4, 2019
Merged

Add timeout to scripted statements #5151

merged 1 commit into from
Oct 4, 2019

Conversation

eatkins
Copy link
Contributor
@eatkins eatkins commented Oct 3, 2019

Sometimes scripted tasks hang in ci and they are effectively impossible
to debug. To workaround this, I add a five minute timeout to any
scripted test and print the log if the test fails.

@eatkins eatkins mentioned this pull request Oct 3, 2019
Merged
@eed3si9n
Copy link
Member
eed3si9n commented Oct 3, 2019 
[error] /home/travis/build/sbt/sbt/scripted-sbt-redux/src/main/scala/sbt/scriptedtest/BatchScriptRunner.scala:72:15: not found: type TimeoutException
[error]       case e: TimeoutException => throw new TestException(statement, "Command timed out", e)
[error]               ^
[error] one error found

@eatkins
Add timeout to scripted statements
155526f 
Sometimes scripted tasks hang in ci and they are effectively impossible
to debug. To workaround this, I add a five minute timeout to any
scripted test and print the log if the test fails.
@eatkins
Copy link
Contributor Author
eatkins commented Oct 3, 2019

Whoops. Squashing error.

@eed3si9n eed3si9n merged commit ee471a2 into sbt:develop Oct 4, 2019
@eatkins eatkins deleted the scripted-timeout branch December 18, 2019 16:26
@eed3si9n eed3si9n mentioned this pull request Jul 11, 2023
Closed
@eed3si9n eed3si9n mentioned this pull request Jul 23, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eed3si9n eed3si9n eed3si9n approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@eatkins @eed3si9n
0