Correct case of Ansible setting for Celery worker timeout. #101
Conversation
|
@smarnach Code LGTM. We already have a ticket for investigating the browser tests that failed in the latest build for this PR (OC-1651). They should go away when you restart the build. I haven't seen the error from the integration test before, but it also looks like an intermittent issue and doesn't seem to be related to the changes in this PR at all. |
|
@smarnach I have cherry-picked this on top of my new relic branch on stage. I am deploying a new appserver with it, let's see if it works. |
|
@smarnach it failed before it got to the edxapp role. Trying again... |
|
@smarnach it worked! 👍 |
|
@omarkhan Thanks, I will merge and deploy to production now. Is the failure you got in the first attempt something I should look into as well? |
smarnach
force-pushed
the
smarnach/celery-setting-case
branch
from
abe0178 to
fff0bf4
Compare
|
This is deployed in production now. |
The setting was uppercased in edx/configuration#2993 after #65 was merged, and never corrected here.