Resolved issue payload dict config with prism.Config.*_categories and prism.Config.*_projects

Author: grdavies-ntnx

docs/changelog.rst

@@ -7,6 +7,7 @@ A log of changes by version and date.
     :header: "Version", "Date", "Notes"
     :widths: 10, 10, 60
 
+    "1.1.18", "3/26/2021", "Resolved issue payload dict config with prism.Config.*_categories and prism.Config.*_projects"
     "1.1.17", "3/26/2021", "Resolved issue with the returned value from prism.Cluster.get"
     "1.1.16", "3/26/2021", "Changed version import in __init.py__ to be absolute. Modified Vm.get to allow for conditional return of both VM disks and VM nics."
     "1.1.15", "3/25/2021", "Fixed inconsistency between master& develop branches"

ntnx_api/_version.py

@@ -3,5 +3,5 @@
 # 2) we can import it in setup.py for the same reason
 # 3) we can import it into your module module
 
-__version_info__ = ('1', '1', '17')
+__version_info__ = ('1', '1', '18')
 __version__ = '.'.join(__version_info__)

ntnx_api/prism.py

@@ -164,7 +164,11 @@ def get_categories(self):
 
         if self.api_client.connection_type == "pc":
             uri = '/categories/list'
-            payload = '{ "kind":"category", "offset": 0, "length": 2147483647  }'
+            payload = {
+                "kind": "category",
+                "offset": 0,
+                "length": 2147483647
+            }
             self.categories = self.api_client.request(uri=uri, payload=payload, params=params).get(
                 'entities')
 
@@ -187,7 +191,11 @@ def get_category_keys(self, category):
 
         if self.api_client.connection_type == "pc":
             uri = '/categories/{0}/list'.format(category)
-            payload = '{ "kind":"category", "offset": 0, "length": 2147483647  }'
+            payload = {
+                "kind": "category",
+                "offset": 0,
+                "length": 2147483647
+            }
             self.category_keys = self.api_client.request(uri=uri, payload=payload, params=params).get(
                 'entities')
 
@@ -254,7 +262,11 @@ def get_projects(self):
 
         if self.api_client == "pc":
             uri = '/projects/list'
-            payload = '{ "kind":"project", "offset": 0, "length": 2147483647  }'
+            payload = {
+                "kind": "project",
+                "offset": 0,
+                "length": 2147483647
+            }
             self.projects = self.api_client.request(uri=uri, payload=payload, params=params).get(
                 'entities')
 
@@ -278,7 +290,11 @@ def get_project_usage(self, project_name):
 
         if self.api_client == "pc":
             uri = '/vms/list'
-            payload = '{"kind": "vm", "offset": 0, "length": 2147483647 }'
+            payload = {
+                "kind": "vm",
+                "offset": 0,
+                "length": 2147483647
+            }
             vms = self.api_client.request(uri=uri, payload=payload, params=params)
 
             for vm in vms:
@@ -2316,17 +2332,20 @@ def get(self, clusteruuid=None, refresh=False):
 
         # Remove existing data for this cluster if it exists
         if refresh:
+            logger.info('refresh selected')
             if self.cluster.get(clusteruuid):
                 self.cluster.pop(clusteruuid)
                 logger.info('removing existing data from class dict cluster for cluster {0}'.format(clusteruuid))
             result = self.api_client.request(uri=uri, api_version='v2.0', payload=payload, params=params)
             self.cluster[clusteruuid] = result
 
         elif not self.cluster.get(clusteruuid):
+            logger.info('no existing data. getting cluster data from api.')
             result = self.api_client.request(uri=uri, api_version='v2.0', payload=payload, params=params)
             self.cluster[clusteruuid] = result
 
         else:
+            logger.info('using existing data')
             result = self.cluster.get(clusteruuid)
 
         return result