Infra/lemur
5
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix import metrics in notifications/messaging.py (#1254)

Browse Source 
`from lemur import metrics` is incorrect for notifications/messaging.py
because that is importing the `metrics` module rather than the
instanciated `lemur.extensions.metrics` object.  This will cause errors
if you import notifications/messaging.py elsewhere, since it can cause
circular dependencies.

Change-Id: Ice28c480373601420fc83bae2d27bb6467cdb752
This commit is contained in:
James Chuong 2018-05-29 18:54:16 -07:00 committed by kevgliss
parent c5e7e5ab68
commit 4911d713a5
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lemur/notifications/messaging.py
View File

@ -17,9 +17,9 @@ from flask import current_app
from sqlalchemy import and_
from lemur import database, metrics
from lemur import database
from lemur.constants import FAILURE_METRIC_STATUS, SUCCESS_METRIC_STATUS
from lemur.extensions import sentry
from lemur.extensions import metrics, sentry
from lemur.common.utils import windowed_query
from lemur.certificates.schemas import certificate_notification_output_schema