diff options
| author | Aravinda VK <avishwan@redhat.com> | 2016-09-09 12:40:38 +0530 |
|---|---|---|
| committer | Aravinda VK <avishwan@redhat.com> | 2016-09-09 05:59:50 -0700 |
| commit | aa742fd88e08eeb807dba1cb98402e8813ec6797 (patch) | |
| tree | 737b3c90163ee6337d11ebc39af8f1ddf1632660 /events | |
| parent | 36af81ac7cb2d459f9bfc0c436f0038a68f85235 (diff) | |
eventsapi: Add conditional import for requests library
requests lib is used only during publishing event. gf_event python
imports utils.py, and indirectly imports requests lib even though
it is not required while sending event to eventsd.
Moved "import requests" inside the "plugin_webhook" function.
BUG: 1374587
Change-Id: Ie3c8088b43d4d7952d01352731999bf8519c73c4
Signed-off-by: Aravinda VK <avishwan@redhat.com>
Reviewed-on: http://review.gluster.org/15439
Reviewed-by: Prashanth Pai <ppai@redhat.com>
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Kotresh HR <khiremat@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Diffstat (limited to 'events')
| -rw-r--r-- | events/src/utils.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/events/src/utils.py b/events/src/utils.py index db8ebfe29a9..dadd9ae3332 100644 --- a/events/src/utils.py +++ b/events/src/utils.py @@ -15,7 +15,6 @@ import logging import fcntl from errno import ESRCH, EBADF -import requests from eventsapiconf import (LOG_FILE, WEBHOOKS_FILE, DEFAULT_CONFIG_FILE, @@ -145,6 +144,9 @@ def publish(ts, event_key, data): def plugin_webhook(message): + # Import requests here since not used in any other place + import requests + message_json = json.dumps(message, sort_keys=True) logger.debug("EVENT: {0}".format(message_json)) for url, token in _webhooks.items(): |