From a2bd65f8963a1d418ec49402cf46b41a402d6737 Mon Sep 17 00:00:00 2001 From: Benjamin Dauvergne Date: Fri, 8 Mar 2019 15:52:59 +0100 Subject: [PATCH 9/9] reset MUST_NOTIFY on request start (#29240) It breaks test tests_hobo/test_manager.py:test_unlogged_access which do not expect any DB access but the flag seems to still be True from a previous test. --- hobo/deploy/signals.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/hobo/deploy/signals.py b/hobo/deploy/signals.py index b8507f2..a2ca1eb 100644 --- a/hobo/deploy/signals.py +++ b/hobo/deploy/signals.py @@ -20,7 +20,7 @@ from celery import Celery from kombu.common import Broadcast from django.conf import settings -from django.core.signals import request_finished +from django.core.signals import request_finished, request_started from django.db.models.signals import post_save from django.dispatch import receiver @@ -43,6 +43,11 @@ def post_environment_save(sender, instance, **kwargs): tls.MUST_NOTIFY = True +@receiver(request_started) +def reset_must_notify(sender, **kwargs): + tls.MUST_NOTIFY = False + + @receiver(request_finished) def notify_agents(sender, **kwargs): if not tls.MUST_NOTIFY: -- 2.20.1