Make sure all variables are defined + handle exception in a better way (#142)

* Make sure all variables are defined + handle exception in a better way

Co-authored-by: Adrià Mercader <amercadero@gmail.com>

Author: zelima

ckan_cloud_operator/providers/solr/manager.py

@@ -98,12 +98,15 @@ def get_collection_status(collection_name):
                 'collection_name': collection_name,
                 'solr_http_endpoint': get_internal_http_endpoint()}
     else:
-
-        res = {'schema': {'version': '2.8', 'name': 'ckan'}}
+        def_ver, def_name = '2.8', 'ckan'
+        res = {'schema': {'version': def_ver, 'name': def_name}}
         try:
-            res = json.loads(new_output)
-        except:
-            pass
+            res = json.loads(output)
+        except json.decoder.JSONDecodeError as e:
+            logs.warning(
+                f'Not able to decode response from SOLR. Using default values for schema version/name - {def_ver}/{def_name}\n SOLR response: \n{output}'
+            )
+
         return {'ready': True,
                 'collection_name': collection_name,
                 'solr_http_endpoint': get_internal_http_endpoint(),