Merge lp:~suligap/canonical-identity-provider/charm-talisker-soft-request-timeout into lp:~ubuntuone-pqm-team/canonical-identity-provider/charm

[Przemysław Suliga]

Enable soft request timeout warnings from talisker

Set to 2500 milliseconds (same as SSO's settings.HANDLER_TIMEOUT_MILLIS)
This is part of move to talisker and SSO's raven setup standarisation:
https://code.launchpad.net/~suligap/canonical-identity-provider/+git/canonical-identity-provider/+merge/384254

[Daniel Manrique (roadmr)]

Dumb question, why isn't this set in SSO's settings_base.py ? it's not a setting we change often, and I fear putting it in the charm config will make it difficult to locate if we do need to change it.

review: Needs Information

[Przemysław Suliga (suligap)]

I agree that this might be harder to find now. But this is a talisker setting. It's not Django specific and talisker doesn't support plucking it out of Django settings module for example. It's WSGI server level functionality (and config) with talisker.

[Daniel Manrique (roadmr)]

The explanation makes sense, thanks for bearing with me. Let's merge!

review: Approve

[Otto Co-Pilot (otto-copilot)]

Running landing tests failed
https://jenkins.ols.canonical.com/online-services/job/canonical-identity-provider_charm/5/

=== modified file 'playbook.yaml'
--- playbook.yaml	2019-04-19 10:59:33 +0000
+++ playbook.yaml	2020-05-25 09:00:38 +0000
@@ -77,7 +77,7 @@
       wsgi_user: "{{ user }}"
       wsgi_group: "{{ user }}"
       wsgi_application: django_project.wsgi
-      env_extra: "DJANGO_SETTINGS_MODULE={{ DJANGO_SETTINGS_MODULE }}"
+      env_extra: "DJANGO_SETTINGS_MODULE={{ DJANGO_SETTINGS_MODULE }} TALISKER_SOFT_REQUEST_TIMEOUT=2500"
       # clean up any old config in the relation
       wsgi_extra: ""
       wsgi_extra_config: |