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

feat: increase uwsgi timeouts to 10 minutes (600 seconds) #357

Merged
merged 1 commit into from
Oct 15, 2021

Conversation

m0nhawk
Copy link
Contributor

@m0nhawk m0nhawk commented Oct 14, 2021

New Features

Breaking Changes

Bug Fixes

Improvements

  • Increased uWSGI timeout to 10 minutes (600 seconds)

Dependency updates

Deployment changes

@PlanXCyborg
Copy link

The style in this PR agrees with black. ✔️

This formatting comment was generated automatically by a script in uc-cdis/wool.

Copy link
Contributor

@paulineribeyre paulineribeyre left a comment

Choose a reason for hiding this comment

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

context:
attempting to fix timeout issue when exporting data from a node:

In [35]: sub.export_node(prog,proj,node_type='ct_instance',fileformat='tsv')
Out[35]: ''
{..., "response_secs": 45.005, "http_status_code": 200, "http_request": "/v0/submission/TCIA/COVID-19-NY-SBU/export?node_label=ct_instance&format=tsv", "http_verb": "GET", "http_referer": "https://data.midrc.org/TCIA-COVID-19-NY-SBU/search?node_type=imaging_study", ...}
Thu Oct 14 18:28:47 2021 - SIGPIPE: writing to a closed pipe/socket/fd (probably the client disconnected) on request /v0/submission/TCIA/COVID-19-NY-SBU/export?node_label=ct_instance&format=tsv (...) !!!
Thu Oct 14 18:28:47 2021 - uwsgi_response_write_body_do(): Broken pipe [core/writer.c line 429] during GET /v0/submission/TCIA/COVID-19-NY-SBU/export?node_label=ct_instance&format=tsv (...)
OSError: write error

@m0nhawk m0nhawk merged commit d0cfeb7 into master Oct 15, 2021
@m0nhawk m0nhawk deleted the feat/uwsgi-timeouts branch October 15, 2021 20:53
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.

3 participants