From 45245f594dc99ef01d1a27d4cf65b36d1d9860b2 Mon Sep 17 00:00:00 2001
From: Erik Reid <erik.reid@geant.org>
Date: Thu, 3 Jan 2019 12:48:48 +0100
Subject: [PATCH] bugfix: remove extra json-ification of data before saving

---
 inventory_provider/tasks/worker.py | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/inventory_provider/tasks/worker.py b/inventory_provider/tasks/worker.py
index 67290892..643adc48 100644
--- a/inventory_provider/tasks/worker.py
+++ b/inventory_provider/tasks/worker.py
@@ -27,31 +27,31 @@ class InventoryTask(Task):
         pass
 
     @staticmethod
-    def save_key(hostname, key, data):
+    def save_key(hostname, key, value):
         r = redis.StrictRedis(
             host=InventoryTask.config["redis"]["hostname"],
             port=InventoryTask.config["redis"]["port"])
         r.hset(
             name=hostname,
             key=key,
-            value=json.dumps(data))
+            value=value)
         InventoryTask.logger.debug(
             "saved %s, key %s" % (hostname, key))
         return "OK"
 
     @staticmethod
-    def save_key_json(hostname, key, data):
+    def save_key_json(hostname, key, data_obj):
         InventoryTask.save_key(
             hostname,
             key,
-            json.dumps(data))
+            json.dumps(data_obj))
 
     @staticmethod
-    def save_key_etree(hostname, key, data):
+    def save_key_etree(hostname, key, xml_doc):
         InventoryTask.save_key(
             hostname,
             key,
-            etree.tostring(data, encoding='unicode'))
+            etree.tostring(xml_doc, encoding='unicode'))
 
 
 class WorkerArgs(bootsteps.Step):
-- 
GitLab