Skip to content

Remove timeout limit from gunicorn - #74

Merged
jkalayan merged 2 commits into
mainfrom
69-bug-extraction-stops-after-15-minutes
Oct 16, 2026
Merged

jkalayan merged 2 commits into
mainfrom
69-bug-extraction-stops-after-15-minutes

Conversation

@jkalayan

Copy link
Copy Markdown
Collaborator

Summary

This pull request includes two main changes: one to the backend deployment configuration and another to the client-side form submission logic.

Deployment configuration changes:

  • Simplified the CMD in the docker/Dockerfile to remove the gthread worker class and threading options, and set the Gunicorn timeout to unlimited (--timeout 0). This may affect how the application handles long-running requests and concurrency.

Form submission logic changes:

  • Updated the validateAndSubmit function in metadata.js to avoid re-uploading files that were already stored server-side during extraction by deleting the topology and trajectory[] fields from the form data before submission.

Impact

  • Allows for testing infrastructure without timeouts imposed by gunicorn.

@jkalayan jkalayan linked an issue Oct 16, 2026 that may be closed by this pull request
@jkalayan
jkalayan merged commit 65a322a into main Oct 16, 2026
11 checks passed
@jkalayan jkalayan changed the title 69 bug extraction stops after 15 minutes Remove timeout limit from gunicorn Oct 16, 2026
@jkalayan jkalayan self-assigned this Oct 16, 2026
@jkalayan jkalayan added the bug Something isn't working label Oct 16, 2026
@jkalayan
jkalayan deleted the 69-bug-extraction-stops-after-15-minutes branch October 16, 2026 08:47
@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report for CI Build 38875742150

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage remained the same at 91.609%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 727
Covered Lines: 666
Line Coverage: 91.61%
Coverage Strength: 8.24 hits per line

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Extraction stops after 30 seconds

1 participant