diff --git a/cms/envs/common.py b/cms/envs/common.py index 0bb357666e..9fa2332f10 100644 --- a/cms/envs/common.py +++ b/cms/envs/common.py @@ -966,7 +966,7 @@ INSTALLED_APPS = [ 'eventtracking.django.apps.EventTrackingConfig', # Monitoring - 'openedx.core.djangoapps.datadog', + 'openedx.core.djangoapps.datadog.apps.DatadogConfig', # For asset pipelining 'edxmako.apps.EdxMakoConfig', @@ -1009,9 +1009,6 @@ INSTALLED_APPS = [ # Signals 'openedx.core.djangoapps.signals.apps.SignalConfig', - # Monitoring signals - 'openedx.core.djangoapps.monitoring', - # Course action state 'course_action_state', diff --git a/lms/envs/common.py b/lms/envs/common.py index 2806b35b95..403c2d2fb0 100644 --- a/lms/envs/common.py +++ b/lms/envs/common.py @@ -2161,7 +2161,7 @@ INSTALLED_APPS = [ 'splash', # Monitoring - 'openedx.core.djangoapps.datadog', + 'openedx.core.djangoapps.datadog.apps.DatadogConfig', # User API 'rest_framework', @@ -2202,9 +2202,6 @@ INSTALLED_APPS = [ # Country embargo support 'openedx.core.djangoapps.embargo', - # Monitoring functionality - 'openedx.core.djangoapps.monitoring', - # Course action state 'course_action_state', diff --git a/openedx/core/djangoapps/datadog/apps.py b/openedx/core/djangoapps/datadog/apps.py new file mode 100644 index 0000000000..b14d1ebb1a --- /dev/null +++ b/openedx/core/djangoapps/datadog/apps.py @@ -0,0 +1,32 @@ +""" +Configuration for datadog Django app +""" +from django.apps import AppConfig +from django.conf import settings +from dogapi import dog_http_api, dog_stats_api + + +class DatadogConfig(AppConfig): + """ + Configuration class for datadog Django app + """ + name = 'openedx.core.djangoapps.datadog' + verbose_name = "Datadog" + + def ready(self): + """ + Initialize connection to datadog during django startup. + + Configure using DATADOG dictionary in the django project settings. + """ + # By default use the statsd agent + options = {'statsd': True} + + if hasattr(settings, 'DATADOG'): + options.update(settings.DATADOG) + + # Not all arguments are documented. + # Look at the source code for details. + dog_stats_api.start(**options) + + dog_http_api.api_key = options.get('api_key') diff --git a/openedx/core/djangoapps/datadog/startup.py b/openedx/core/djangoapps/datadog/startup.py deleted file mode 100644 index 8c02244742..0000000000 --- a/openedx/core/djangoapps/datadog/startup.py +++ /dev/null @@ -1,27 +0,0 @@ -""" -Start up initialization of datadog. -""" -from django.conf import settings -from dogapi import dog_http_api, dog_stats_api - - -def run(): - """ - Initialize connection to datadog during django startup. - - Can be configured using a dictionary named DATADOG in the django - project settings. - - """ - - # By default use the statsd agent - options = {'statsd': True} - - if hasattr(settings, 'DATADOG'): - options.update(settings.DATADOG) - - # Not all arguments are documented. - # Look at the source code for details. - dog_stats_api.start(**options) - - dog_http_api.api_key = options.get('api_key') diff --git a/openedx/core/djangoapps/monitoring/README.rst b/openedx/core/djangoapps/monitoring/README.rst deleted file mode 100644 index 4b7617d88b..0000000000 --- a/openedx/core/djangoapps/monitoring/README.rst +++ /dev/null @@ -1,6 +0,0 @@ -This djangoapp is incorrectly named 'monitoring'. - -The name is related to old functionality that used to be a part of this app. - -TODO: The current contents of this app should be joined with other generic -platform utilities and renamed appropriately. diff --git a/openedx/core/djangoapps/monitoring/__init__.py b/openedx/core/djangoapps/monitoring/__init__.py deleted file mode 100644 index e69de29bb2..0000000000 diff --git a/openedx/core/djangoapps/monitoring/startup.py b/openedx/core/djangoapps/monitoring/startup.py deleted file mode 100644 index 108f290772..0000000000 --- a/openedx/core/djangoapps/monitoring/startup.py +++ /dev/null @@ -1,5 +0,0 @@ -""" -Registers signal handlers at startup. -""" -# pylint: disable=unused-import -import openedx.core.djangoapps.monitoring.exceptions diff --git a/openedx/core/djangoapps/util/apps.py b/openedx/core/djangoapps/util/apps.py index 2fc4224a0b..852f4dbd09 100644 --- a/openedx/core/djangoapps/util/apps.py +++ b/openedx/core/djangoapps/util/apps.py @@ -7,8 +7,15 @@ from django.apps import AppConfig class UtilConfig(AppConfig): """ - Let Django know that this is an app with management commands. + Configuration class for the openedx.core.djangoapps.util Django application """ label = 'open_edx_util' name = 'openedx.core.djangoapps.util' verbose_name = 'Open edX Utilities' + + def ready(self): + """ + Registers signal handlers at startup. + """ + # pylint: disable=unused-import + import openedx.core.djangoapps.util.signals diff --git a/openedx/core/djangoapps/monitoring/exceptions.py b/openedx/core/djangoapps/util/signals.py similarity index 100% rename from openedx/core/djangoapps/monitoring/exceptions.py rename to openedx/core/djangoapps/util/signals.py