Merge pull request #13692 from edx/common_cleanup/monitoring

Move monitoring from common to openedx/core

cms/envs/common.py

@@ -866,7 +866,7 @@ INSTALLED_APPS = (
     'embargo',
 
     # Monitoring signals
-    'monitoring',
+    'openedx.core.djangoapps.monitoring',
 
     # Course action state
     'course_action_state',

common/djangoapps/monitoring/startup.py

-# Register signal handlers
-# pylint: disable=unused-import
-import signals
-import exceptions

lms/envs/common.py

@@ -2008,7 +2008,7 @@ INSTALLED_APPS = (
     'embargo',
 
     # Monitoring functionality
-    'monitoring',
+    'openedx.core.djangoapps.monitoring',
 
     # Course action state
     'course_action_state',

common/djangoapps/monitoring/exceptions.py → openedx/core/djangoapps/monitoring/exceptions.py

+"""
+Signal handler for exceptions.
+"""
 from django.core.signals import got_request_exception
 from django.dispatch import receiver
 import logging
 
 
 @receiver(got_request_exception)
-def record_request_exception(sender, **kwargs):
+def record_request_exception(sender, **kwargs):  # pylint: disable=unused-argument
+    """
+    Logs the stack trace whenever an exception
+    occurs in processing a request.
+    """
     logging.exception("Uncaught exception from {sender}".format(
         sender=sender
     ))

common/djangoapps/monitoring/signals.py → openedx/core/djangoapps/monitoring/signals.py

@@ -13,7 +13,7 @@ from django.dispatch import receiver
 import dogstats_wrapper as dog_stats_api
 
 
-def _database_tags(action, sender, kwargs):
+def _database_tags(action, sender, kwargs):  # pylint: disable=unused-argument
     """
     Return a tags for the sender and database used in django.db.models signals.
 

openedx/core/djangoapps/monitoring/startup.py

+"""
+Registers signal handlers at startup.
+"""
+# pylint: disable=unused-import
+import openedx.core.djangoapps.monitoring.signals
+import openedx.core.djangoapps.monitoring.exceptions