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

workflows: timeouts refextract #2445

Merged
merged 1 commit into from
Jun 15, 2017
Merged

workflows: timeouts refextract #2445

merged 1 commit into from
Jun 15, 2017

Conversation

kaplun
Copy link
Contributor

@kaplun kaplun commented Jun 15, 2017

  • I have all the information that I need (if not, move to RFC and look for it).
  • I linked the related issue(s) in the corresponding commit logs.
  • I wrote good commit log messages.
  • My code follows the code style of this project.
  • I've added any new docs if API/utils methods were added.
  • I have updated the existing documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Worksaround inspirehep/refextract#26 by interrupting the running away
refextract process.

Signed-off-by: Samuele Kaplun [email protected]

@kaplun kaplun force-pushed the timeout branch 2 times, most recently from 1e4a1af to eb6013b Compare June 15, 2017 14:09
Times out the `refextract` task after 300 seconds to work around
inspirehep/refextract#26, which would otherwise block a Celery
worker indefinitely.

Signed-off-by: Samuele Kaplun <[email protected]>
@jacquerie
Copy link
Contributor

jacquerie commented Jun 15, 2017

If you're wondering about the close/reopen, there was some trouble in hook delivery between GitHub and Travis for a little while.

@jacquerie jacquerie merged commit 61798b0 into inspirehep:master Jun 15, 2017
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.

2 participants