chore(helmbroker): fixup extract_tgz path

Author: jianxiaoguo

rootfs/helmbroker/loader.py

@@ -105,7 +105,7 @@ def load_addons(repository):
             addon_tgz_url = f'{url}/{tgz_name}.tgz'
             download_file(addon_tgz_url, ADDONS_PATH)
             extract_tgz(f'{ADDONS_PATH}/{tgz_name}.tgz',
-                        f'{ADDONS_PATH}/{tgz_name}')
+                        f'{ADDONS_PATH}')
         addons_meta_file()
 
 

rootfs/helmbroker/tasks.py

@@ -2,6 +2,7 @@
 import time
 import shutil
 import yaml
+import logging
 
 from openbrokerapi.service_broker import ProvisionDetails, OperationState, \
     UpdateDetails, BindDetails
@@ -11,6 +12,8 @@
     InstanceLock, dump_instance_meta, dump_binding_meta, load_instance_meta, \
     get_instance_file, helm
 
+logger = logging.getLogger(__name__)
+
 
 @app.task(serializer='pickle')
 def provision(instance_id: str, details: ProvisionDetails):
@@ -42,7 +45,10 @@ def provision(instance_id: str, details: ProvisionDetails):
             "--set",
             f"fullnameOverride=helmbroker-{details.context['instance_name']}"
         ]
-
+        logger.info(f"helm install parameters :{details.parameters}")
+        logger.info(f"helm install parameters type:{type(details.parameters)}")
+        # for k, v in details.parameters:
+        #     args.append("--set", k, v)
         status, output = helm(instance_id, *args)
         data = load_instance_meta(instance_id)
         if status != 0:
@@ -90,7 +96,8 @@ def update(instance_id: str, details: UpdateDetails):
         "--set",
         f"fullnameOverride=helmbroker-{details.context['instance_name']}"
     ]
-
+    logger.info(f"helm upgrade parameters: {details.parameters}")
+    logger.info(f"helm upgrade parameters type: {type(details.parameters)}")
     status, output = helm(instance_id, *args)
     if status != 0:
         data["last_operation"]["state"] = OperationState.FAILED.value