adding kvstore_owner as new property

commenting the need to set kvstore owner to default value of  "nobody" as new property is created for the kvstore owner with default to "nobody"

Author: ashah-splunk

splunklib/client.py

@@ -678,25 +678,28 @@ def splunk_version(self):
 
     @property
     def kvstore_owner(self):
+        """Returns the KVStore owner for this instance of Splunk.
+
+        By default is the kvstore owner is not set, it will return "nobody"
+        :return: A string with the KVStore owner.
+        """
         if self._kvstore_owner is None:
             self._kvstore_owner = "nobody"
-            #self.namespace['owner'] = "nobody"
         return self._kvstore_owner
 
     @kvstore_owner.setter
     def kvstore_owner(self, value):
         self._kvstore_owner = value
-        #self.namespace['owner'] = value
 
     @property
     def kvstore(self):
         """Returns the collection of KV Store collections.
 
+        sets the owner for the namespace, before retrieving the KVStore Collection
+
         :return: A :class:`KVStoreCollections` collection of :class:`KVStoreCollection` entities.
         """
         self.namespace['owner'] = self.kvstore_owner
-        # if self.namespace['owner'] is None:
-        #     self.namespace['owner'] = "nobody"
         return KVStoreCollections(self)
 
     @property

tests/test_kvstore_batch.py

@@ -26,7 +26,7 @@
 class KVStoreBatchTestCase(testlib.SDKTestCase):
     def setUp(self):
         super(KVStoreBatchTestCase, self).setUp()
-        self.service.namespace['owner'] = 'nobody'
+        #self.service.namespace['owner'] = 'nobody'
         self.service.namespace['app'] = 'search'
         confs = self.service.kvstore
         if ('test' in confs):

tests/test_kvstore_conf.py

@@ -25,16 +25,16 @@
 class KVStoreConfTestCase(testlib.SDKTestCase):
     def setUp(self):
         super(KVStoreConfTestCase, self).setUp()
-        self.service.namespace['owner'] = 'nobody'
+        #self.service.namespace['owner'] = 'nobody'
         self.service.namespace['app'] = 'search'
         self.confs = self.service.kvstore
         if ('test' in self.confs):
             self.confs['test'].delete()
 
     def test_owner_restriction(self):
-        self.service.namespace['owner'] = 'admin'
+        self.service.kvstore_owner = 'admin'
         self.assertRaises(client.HTTPError, lambda: self.confs.list())
-        self.service.namespace['owner'] = 'nobody'
+        self.service.kvstore_owner = 'nobody'
 
     def test_create_delete_collection(self):
         self.confs.create('test')

tests/test_kvstore_data.py

@@ -27,7 +27,7 @@
 class KVStoreDataTestCase(testlib.SDKTestCase):
     def setUp(self):
         super(KVStoreDataTestCase, self).setUp()
-        self.service.namespace['owner'] = 'nobody'
+        #self.service.namespace['owner'] = 'nobody'
         self.service.namespace['app'] = 'search'
         self.confs = self.service.kvstore
         if ('test' in self.confs):