Skip to content
This repository has been archived by the owner on Aug 22, 2022. It is now read-only.

Shorter timeout for celery workers #65

Merged
merged 1 commit into from
May 2, 2016
Merged

Conversation

e-kolpakov
Copy link
Contributor

@e-kolpakov e-kolpakov force-pushed the ekolpakov/full-retries branch from 4c5ebe7 to 530a23d Compare April 29, 2016 12:44
@@ -75,3 +75,5 @@ COMMON_USER_INFO:{% for github_username in github_admin_username_list %}
- name: {{ github_username }}
github: true
type: admin{% endfor %}{% endif %}

edxapp_worker_default_stopwaitsecs: 60
Copy link
Member

Choose a reason for hiding this comment

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

@e-kolpakov Nit: It would be nice to put this in a dedicated section (i.e., add a comment on top, such as # Workers).

@e-kolpakov e-kolpakov force-pushed the ekolpakov/full-retries branch from 530a23d to 550a21c Compare April 29, 2016 13:07
@itsjeyd
Copy link
Member

itsjeyd commented Apr 29, 2016

@e-kolpakov Looks good, 👍 once build is green :)

@e-kolpakov e-kolpakov force-pushed the ekolpakov/full-retries branch from 550a21c to 1642d8b Compare May 2, 2016 09:41
@e-kolpakov e-kolpakov force-pushed the ekolpakov/full-retries branch from 1642d8b to e197fda Compare May 2, 2016 11:09
@e-kolpakov e-kolpakov merged commit a911176 into master May 2, 2016
@smarnach smarnach deleted the ekolpakov/full-retries branch August 22, 2017 17:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants