Reformat LCM to standardized format
[osm/LCM.git] / osm_lcm / vim_sdn.py
index a1623ba..1641e96 100644 (file)
@@ -36,8 +36,15 @@ __author__ = "Alfonso Tierno"
 
 class VimLcm(LcmBase):
     # values that are encrypted at vim config because they are passwords
-    vim_config_encrypted = {"1.1": ("admin_password", "nsx_password", "vcenter_password"),
-                            "default": ("admin_password", "nsx_password", "vcenter_password", "vrops_password")}
+    vim_config_encrypted = {
+        "1.1": ("admin_password", "nsx_password", "vcenter_password"),
+        "default": (
+            "admin_password",
+            "nsx_password",
+            "vcenter_password",
+            "vrops_password",
+        ),
+    }
 
     def __init__(self, msg, lcm_tasks, config, loop):
         """
@@ -46,7 +53,7 @@ class VimLcm(LcmBase):
         :return: None
         """
 
-        self.logger = logging.getLogger('lcm.vim')
+        self.logger = logging.getLogger("lcm.vim")
         self.loop = loop
         self.lcm_tasks = lcm_tasks
         self.ro_config = config["ro_config"]
@@ -59,8 +66,8 @@ class VimLcm(LcmBase):
         # If 'vim_content' does not include 'op_id', we a running a legacy NBI version.
         # In such a case, HA is not supported by NBI, 'op_id' is None, and lock_HA() will do nothing.
         # Register 'create' task here for related future HA operations
-        op_id = vim_content.pop('op_id', None)
-        if not self.lcm_tasks.lock_HA('vim', 'create', op_id):
+        op_id = vim_content.pop("op_id", None)
+        if not self.lcm_tasks.lock_HA("vim", "create", op_id):
             return
 
         vim_id = vim_content["_id"]
@@ -74,19 +81,32 @@ class VimLcm(LcmBase):
         try:
             step = "Getting vim-id='{}' from db".format(vim_id)
             db_vim = self.db.get_one("vim_accounts", {"_id": vim_id})
-            if vim_content.get("config") and vim_content["config"].get("sdn-controller"):
-                step = "Getting sdn-controller-id='{}' from db".format(vim_content["config"]["sdn-controller"])
-                db_sdn = self.db.get_one("sdns", {"_id": vim_content["config"]["sdn-controller"]})
+            if vim_content.get("config") and vim_content["config"].get(
+                "sdn-controller"
+            ):
+                step = "Getting sdn-controller-id='{}' from db".format(
+                    vim_content["config"]["sdn-controller"]
+                )
+                db_sdn = self.db.get_one(
+                    "sdns", {"_id": vim_content["config"]["sdn-controller"]}
+                )
 
                 # If the VIM account has an associated SDN account, also
                 # wait for any previous tasks in process for the SDN
-                await self.lcm_tasks.waitfor_related_HA('sdn', 'ANY', db_sdn["_id"])
+                await self.lcm_tasks.waitfor_related_HA("sdn", "ANY", db_sdn["_id"])
 
-                if db_sdn.get("_admin") and db_sdn["_admin"].get("deployed") and db_sdn["_admin"]["deployed"].get("RO"):
+                if (
+                    db_sdn.get("_admin")
+                    and db_sdn["_admin"].get("deployed")
+                    and db_sdn["_admin"]["deployed"].get("RO")
+                ):
                     RO_sdn_id = db_sdn["_admin"]["deployed"]["RO"]
                 else:
-                    raise LcmException("sdn-controller={} is not available. Not deployed at RO".format(
-                        vim_content["config"]["sdn-controller"]))
+                    raise LcmException(
+                        "sdn-controller={} is not available. Not deployed at RO".format(
+                            vim_content["config"]["sdn-controller"]
+                        )
+                    )
 
             step = "Creating vim at RO"
             db_vim_update["_admin.deployed.RO"] = None
@@ -107,66 +127,87 @@ class VimLcm(LcmBase):
             desc = await RO.create("vim", descriptor=vim_RO)
             RO_vim_id = desc["uuid"]
             db_vim_update["_admin.deployed.RO"] = RO_vim_id
-            self.logger.debug(logging_text + "VIM created at RO_vim_id={}".format(RO_vim_id))
+            self.logger.debug(
+                logging_text + "VIM created at RO_vim_id={}".format(RO_vim_id)
+            )
 
             step = "Creating vim_account at RO"
             db_vim_update["_admin.detailed-status"] = step
             self.update_db_2("vim_accounts", vim_id, db_vim_update)
 
             if vim_content.get("vim_password"):
-                vim_content["vim_password"] = self.db.decrypt(vim_content["vim_password"],
-                                                              schema_version=schema_version,
-                                                              salt=vim_id)
-            vim_account_RO = {"vim_tenant_name": vim_content["vim_tenant_name"],
-                              "vim_username": vim_content["vim_user"],
-                              "vim_password": vim_content["vim_password"]
-                              }
+                vim_content["vim_password"] = self.db.decrypt(
+                    vim_content["vim_password"],
+                    schema_version=schema_version,
+                    salt=vim_id,
+                )
+            vim_account_RO = {
+                "vim_tenant_name": vim_content["vim_tenant_name"],
+                "vim_username": vim_content["vim_user"],
+                "vim_password": vim_content["vim_password"],
+            }
             if vim_RO.get("config"):
                 vim_account_RO["config"] = vim_RO["config"]
                 if "sdn-controller" in vim_account_RO["config"]:
                     del vim_account_RO["config"]["sdn-controller"]
                 if "sdn-port-mapping" in vim_account_RO["config"]:
                     del vim_account_RO["config"]["sdn-port-mapping"]
-                vim_config_encrypted_keys = self.vim_config_encrypted.get(schema_version) or \
-                    self.vim_config_encrypted.get("default")
+                vim_config_encrypted_keys = self.vim_config_encrypted.get(
+                    schema_version
+                ) or self.vim_config_encrypted.get("default")
                 for p in vim_config_encrypted_keys:
                     if vim_account_RO["config"].get(p):
-                        vim_account_RO["config"][p] = self.db.decrypt(vim_account_RO["config"][p],
-                                                                      schema_version=schema_version,
-                                                                      salt=vim_id)
+                        vim_account_RO["config"][p] = self.db.decrypt(
+                            vim_account_RO["config"][p],
+                            schema_version=schema_version,
+                            salt=vim_id,
+                        )
 
             desc = await RO.attach("vim_account", RO_vim_id, descriptor=vim_account_RO)
             db_vim_update["_admin.deployed.RO-account"] = desc["uuid"]
             db_vim_update["_admin.operationalState"] = "ENABLED"
             db_vim_update["_admin.detailed-status"] = "Done"
             # Mark the VIM 'create' HA task as successful
-            operation_state = 'COMPLETED'
-            operation_details = 'Done'
+            operation_state = "COMPLETED"
+            operation_details = "Done"
 
-            self.logger.debug(logging_text + "Exit Ok VIM account created at RO_vim_account_id={}".format(desc["uuid"]))
+            self.logger.debug(
+                logging_text
+                + "Exit Ok VIM account created at RO_vim_account_id={}".format(
+                    desc["uuid"]
+                )
+            )
             return
 
         except (ROclient.ROClientException, DbException, asyncio.CancelledError) as e:
             self.logger.error(logging_text + "Exit Exception {}".format(e))
             exc = e
         except Exception as e:
-            self.logger.critical(logging_text + "Exit Exception {}".format(e), exc_info=True)
+            self.logger.critical(
+                logging_text + "Exit Exception {}".format(e), exc_info=True
+            )
             exc = e
         finally:
             if exc and db_vim:
                 db_vim_update["_admin.operationalState"] = "ERROR"
-                db_vim_update["_admin.detailed-status"] = "ERROR {}: {}".format(step, exc)
+                db_vim_update["_admin.detailed-status"] = "ERROR {}: {}".format(
+                    step, exc
+                )
                 # Mark the VIM 'create' HA task as erroneous
-                operation_state = 'FAILED'
+                operation_state = "FAILED"
                 operation_details = "ERROR {}: {}".format(step, exc)
             try:
                 if db_vim_update:
                     self.update_db_2("vim_accounts", vim_id, db_vim_update)
                 # Register the VIM 'create' HA task either
                 # succesful or erroneous, or do nothing (if legacy NBI)
-                self.lcm_tasks.unlock_HA('vim', 'create', op_id,
-                                         operationState=operation_state,
-                                         detailed_status=operation_details)
+                self.lcm_tasks.unlock_HA(
+                    "vim",
+                    "create",
+                    op_id,
+                    operationState=operation_state,
+                    detailed_status=operation_details,
+                )
             except DbException as e:
                 self.logger.error(logging_text + "Cannot update database: {}".format(e))
 
@@ -177,8 +218,8 @@ class VimLcm(LcmBase):
         # HA tasks and backward compatibility:
         # If 'vim_content' does not include 'op_id', we a running a legacy NBI version.
         # In such a case, HA is not supported by NBI, and the HA check always returns True
-        op_id = vim_content.pop('op_id', None)
-        if not self.lcm_tasks.lock_HA('vim', 'edit', op_id):
+        op_id = vim_content.pop("op_id", None)
+        if not self.lcm_tasks.lock_HA("vim", "edit", op_id):
             return
 
         vim_id = vim_content["_id"]
@@ -193,25 +234,41 @@ class VimLcm(LcmBase):
         step = "Getting vim-id='{}' from db".format(vim_id)
         try:
             # wait for any previous tasks in process
-            await self.lcm_tasks.waitfor_related_HA('vim', 'edit', op_id)
+            await self.lcm_tasks.waitfor_related_HA("vim", "edit", op_id)
 
             db_vim = self.db.get_one("vim_accounts", {"_id": vim_id})
 
-            if db_vim.get("_admin") and db_vim["_admin"].get("deployed") and db_vim["_admin"]["deployed"].get("RO"):
-                if vim_content.get("config") and vim_content["config"].get("sdn-controller"):
-                    step = "Getting sdn-controller-id='{}' from db".format(vim_content["config"]["sdn-controller"])
-                    db_sdn = self.db.get_one("sdns", {"_id": vim_content["config"]["sdn-controller"]})
+            if (
+                db_vim.get("_admin")
+                and db_vim["_admin"].get("deployed")
+                and db_vim["_admin"]["deployed"].get("RO")
+            ):
+                if vim_content.get("config") and vim_content["config"].get(
+                    "sdn-controller"
+                ):
+                    step = "Getting sdn-controller-id='{}' from db".format(
+                        vim_content["config"]["sdn-controller"]
+                    )
+                    db_sdn = self.db.get_one(
+                        "sdns", {"_id": vim_content["config"]["sdn-controller"]}
+                    )
 
                     # If the VIM account has an associated SDN account, also
                     # wait for any previous tasks in process for the SDN
-                    await self.lcm_tasks.waitfor_related_HA('sdn', 'ANY', db_sdn["_id"])
+                    await self.lcm_tasks.waitfor_related_HA("sdn", "ANY", db_sdn["_id"])
 
-                    if db_sdn.get("_admin") and db_sdn["_admin"].get("deployed") and db_sdn["_admin"]["deployed"].get(
-                            "RO"):
+                    if (
+                        db_sdn.get("_admin")
+                        and db_sdn["_admin"].get("deployed")
+                        and db_sdn["_admin"]["deployed"].get("RO")
+                    ):
                         RO_sdn_id = db_sdn["_admin"]["deployed"]["RO"]
                     else:
-                        raise LcmException("sdn-controller={} is not available. Not deployed at RO".format(
-                            vim_content["config"]["sdn-controller"]))
+                        raise LcmException(
+                            "sdn-controller={} is not available. Not deployed at RO".format(
+                                vim_content["config"]["sdn-controller"]
+                            )
+                        )
 
                 RO_vim_id = db_vim["_admin"]["deployed"]["RO"]
                 step = "Editing vim at RO"
@@ -246,19 +303,24 @@ class VimLcm(LcmBase):
                 if "vim_password" in vim_content:
                     vim_account_RO["vim_password"] = vim_content["vim_password"]
                 if vim_content.get("vim_password"):
-                    vim_account_RO["vim_password"] = self.db.decrypt(vim_content["vim_password"],
-                                                                     schema_version=schema_version,
-                                                                     salt=vim_id)
+                    vim_account_RO["vim_password"] = self.db.decrypt(
+                        vim_content["vim_password"],
+                        schema_version=schema_version,
+                        salt=vim_id,
+                    )
                 if "config" in vim_content:
                     vim_account_RO["config"] = vim_content["config"]
                 if vim_content.get("config"):
-                    vim_config_encrypted_keys = self.vim_config_encrypted.get(schema_version) or \
-                        self.vim_config_encrypted.get("default")
+                    vim_config_encrypted_keys = self.vim_config_encrypted.get(
+                        schema_version
+                    ) or self.vim_config_encrypted.get("default")
                     for p in vim_config_encrypted_keys:
                         if vim_content["config"].get(p):
-                            vim_account_RO["config"][p] = self.db.decrypt(vim_content["config"][p],
-                                                                          schema_version=schema_version,
-                                                                          salt=vim_id)
+                            vim_account_RO["config"][p] = self.db.decrypt(
+                                vim_content["config"][p],
+                                schema_version=schema_version,
+                                salt=vim_id,
+                            )
 
                 if "vim_user" in vim_content:
                     vim_content["vim_username"] = vim_content["vim_user"]
@@ -267,8 +329,8 @@ class VimLcm(LcmBase):
                 await RO.edit("vim_account", RO_vim_id, descriptor=vim_account_RO)
                 db_vim_update["_admin.operationalState"] = "ENABLED"
                 # Mark the VIM 'edit' HA task as successful
-            operation_state = 'COMPLETED'
-            operation_details = 'Done'
+            operation_state = "COMPLETED"
+            operation_details = "Done"
 
             self.logger.debug(logging_text + "Exit Ok RO_vim_id={}".format(RO_vim_id))
             return
@@ -277,23 +339,31 @@ class VimLcm(LcmBase):
             self.logger.error(logging_text + "Exit Exception {}".format(e))
             exc = e
         except Exception as e:
-            self.logger.critical(logging_text + "Exit Exception {}".format(e), exc_info=True)
+            self.logger.critical(
+                logging_text + "Exit Exception {}".format(e), exc_info=True
+            )
             exc = e
         finally:
             if exc and db_vim:
                 db_vim_update["_admin.operationalState"] = "ERROR"
-                db_vim_update["_admin.detailed-status"] = "ERROR {}: {}".format(step, exc)
+                db_vim_update["_admin.detailed-status"] = "ERROR {}: {}".format(
+                    step, exc
+                )
                 # Mark the VIM 'edit' HA task as erroneous
-                operation_state = 'FAILED'
+                operation_state = "FAILED"
                 operation_details = "ERROR {}: {}".format(step, exc)
             try:
                 if db_vim_update:
                     self.update_db_2("vim_accounts", vim_id, db_vim_update)
                 # Register the VIM 'edit' HA task either
                 # succesful or erroneous, or do nothing (if legacy NBI)
-                self.lcm_tasks.unlock_HA('vim', 'edit', op_id,
-                                         operationState=operation_state,
-                                         detailed_status=operation_details)
+                self.lcm_tasks.unlock_HA(
+                    "vim",
+                    "edit",
+                    op_id,
+                    operationState=operation_state,
+                    detailed_status=operation_details,
+                )
             except DbException as e:
                 self.logger.error(logging_text + "Cannot update database: {}".format(e))
 
@@ -304,8 +374,8 @@ class VimLcm(LcmBase):
         # HA tasks and backward compatibility:
         # If 'vim_content' does not include 'op_id', we a running a legacy NBI version.
         # In such a case, HA is not supported by NBI, and the HA check always returns True
-        op_id = vim_content.pop('op_id', None)
-        if not self.lcm_tasks.lock_HA('vim', 'delete', op_id):
+        op_id = vim_content.pop("op_id", None)
+        if not self.lcm_tasks.lock_HA("vim", "delete", op_id):
             return
 
         vim_id = vim_content["_id"]
@@ -318,10 +388,14 @@ class VimLcm(LcmBase):
         step = "Getting vim from db"
         try:
             # wait for any previous tasks in process
-            await self.lcm_tasks.waitfor_related_HA('vim', 'delete', op_id)
+            await self.lcm_tasks.waitfor_related_HA("vim", "delete", op_id)
             if not self.ro_config.get("ng"):
                 db_vim = self.db.get_one("vim_accounts", {"_id": vim_id})
-                if db_vim.get("_admin") and db_vim["_admin"].get("deployed") and db_vim["_admin"]["deployed"].get("RO"):
+                if (
+                    db_vim.get("_admin")
+                    and db_vim["_admin"].get("deployed")
+                    and db_vim["_admin"]["deployed"].get("RO")
+                ):
                     RO_vim_id = db_vim["_admin"]["deployed"]["RO"]
                     RO = ROclient.ROClient(self.loop, **self.ro_config)
                     step = "Detaching vim from RO tenant"
@@ -329,7 +403,10 @@ class VimLcm(LcmBase):
                         await RO.detach("vim_account", RO_vim_id)
                     except ROclient.ROClientException as e:
                         if e.http_code == 404:  # not found
-                            self.logger.debug(logging_text + "RO_vim_id={} already detached".format(RO_vim_id))
+                            self.logger.debug(
+                                logging_text
+                                + "RO_vim_id={} already detached".format(RO_vim_id)
+                            )
                         else:
                             raise
 
@@ -338,7 +415,10 @@ class VimLcm(LcmBase):
                         await RO.delete("vim", RO_vim_id)
                     except ROclient.ROClientException as e:
                         if e.http_code == 404:  # not found
-                            self.logger.debug(logging_text + "RO_vim_id={} already deleted".format(RO_vim_id))
+                            self.logger.debug(
+                                logging_text
+                                + "RO_vim_id={} already deleted".format(RO_vim_id)
+                            )
                         else:
                             raise
                 else:
@@ -353,19 +433,27 @@ class VimLcm(LcmBase):
             self.logger.error(logging_text + "Exit Exception {}".format(e))
             exc = e
         except Exception as e:
-            self.logger.critical(logging_text + "Exit Exception {}".format(e), exc_info=True)
+            self.logger.critical(
+                logging_text + "Exit Exception {}".format(e), exc_info=True
+            )
             exc = e
         finally:
             self.lcm_tasks.remove("vim_account", vim_id, order_id)
             if exc and db_vim:
                 db_vim_update["_admin.operationalState"] = "ERROR"
-                db_vim_update["_admin.detailed-status"] = "ERROR {}: {}".format(step, exc)
+                db_vim_update["_admin.detailed-status"] = "ERROR {}: {}".format(
+                    step, exc
+                )
                 # Mark the VIM 'delete' HA task as erroneous
-                operation_state = 'FAILED'
+                operation_state = "FAILED"
                 operation_details = "ERROR {}: {}".format(step, exc)
-                self.lcm_tasks.unlock_HA('vim', 'delete', op_id,
-                                         operationState=operation_state,
-                                         detailed_status=operation_details)
+                self.lcm_tasks.unlock_HA(
+                    "vim",
+                    "delete",
+                    op_id,
+                    operationState=operation_state,
+                    detailed_status=operation_details,
+                )
             try:
                 if db_vim and db_vim_update:
                     self.update_db_2("vim_accounts", vim_id, db_vim_update)
@@ -387,7 +475,7 @@ class WimLcm(LcmBase):
         :return: None
         """
 
-        self.logger = logging.getLogger('lcm.vim')
+        self.logger = logging.getLogger("lcm.vim")
         self.loop = loop
         self.lcm_tasks = lcm_tasks
         self.ro_config = config["ro_config"]
@@ -400,8 +488,8 @@ class WimLcm(LcmBase):
         # If 'wim_content' does not include 'op_id', we a running a legacy NBI version.
         # In such a case, HA is not supported by NBI, 'op_id' is None, and lock_HA() will do nothing.
         # Register 'create' task here for related future HA operations
-        op_id = wim_content.pop('op_id', None)
-        self.lcm_tasks.lock_HA('wim', 'create', op_id)
+        op_id = wim_content.pop("op_id", None)
+        self.lcm_tasks.lock_HA("wim", "create", op_id)
 
         wim_id = wim_content["_id"]
         logging_text = "Task wim_create={} ".format(wim_id)
@@ -431,62 +519,82 @@ class WimLcm(LcmBase):
             desc = await RO.create("wim", descriptor=wim_RO)
             RO_wim_id = desc["uuid"]
             db_wim_update["_admin.deployed.RO"] = RO_wim_id
-            self.logger.debug(logging_text + "WIM created at RO_wim_id={}".format(RO_wim_id))
+            self.logger.debug(
+                logging_text + "WIM created at RO_wim_id={}".format(RO_wim_id)
+            )
 
             step = "Creating wim_account at RO"
             db_wim_update["_admin.detailed-status"] = step
             self.update_db_2("wim_accounts", wim_id, db_wim_update)
 
             if wim_content.get("wim_password"):
-                wim_content["wim_password"] = self.db.decrypt(wim_content["wim_password"],
-                                                              schema_version=schema_version,
-                                                              salt=wim_id)
-            wim_account_RO = {"name": wim_content["name"],
-                              "user": wim_content["user"],
-                              "password": wim_content["password"]
-                              }
+                wim_content["wim_password"] = self.db.decrypt(
+                    wim_content["wim_password"],
+                    schema_version=schema_version,
+                    salt=wim_id,
+                )
+            wim_account_RO = {
+                "name": wim_content["name"],
+                "user": wim_content["user"],
+                "password": wim_content["password"],
+            }
             if wim_RO.get("config"):
                 wim_account_RO["config"] = wim_RO["config"]
                 if "wim_port_mapping" in wim_account_RO["config"]:
                     del wim_account_RO["config"]["wim_port_mapping"]
                 for p in self.wim_config_encrypted:
                     if wim_account_RO["config"].get(p):
-                        wim_account_RO["config"][p] = self.db.decrypt(wim_account_RO["config"][p],
-                                                                      schema_version=schema_version,
-                                                                      salt=wim_id)
+                        wim_account_RO["config"][p] = self.db.decrypt(
+                            wim_account_RO["config"][p],
+                            schema_version=schema_version,
+                            salt=wim_id,
+                        )
 
             desc = await RO.attach("wim_account", RO_wim_id, descriptor=wim_account_RO)
             db_wim_update["_admin.deployed.RO-account"] = desc["uuid"]
             db_wim_update["_admin.operationalState"] = "ENABLED"
             db_wim_update["_admin.detailed-status"] = "Done"
             # Mark the WIM 'create' HA task as successful
-            operation_state = 'COMPLETED'
-            operation_details = 'Done'
+            operation_state = "COMPLETED"
+            operation_details = "Done"
 
-            self.logger.debug(logging_text + "Exit Ok WIM account created at RO_wim_account_id={}".format(desc["uuid"]))
+            self.logger.debug(
+                logging_text
+                + "Exit Ok WIM account created at RO_wim_account_id={}".format(
+                    desc["uuid"]
+                )
+            )
             return
 
         except (ROclient.ROClientException, DbException, asyncio.CancelledError) as e:
             self.logger.error(logging_text + "Exit Exception {}".format(e))
             exc = e
         except Exception as e:
-            self.logger.critical(logging_text + "Exit Exception {}".format(e), exc_info=True)
+            self.logger.critical(
+                logging_text + "Exit Exception {}".format(e), exc_info=True
+            )
             exc = e
         finally:
             if exc and db_wim:
                 db_wim_update["_admin.operationalState"] = "ERROR"
-                db_wim_update["_admin.detailed-status"] = "ERROR {}: {}".format(step, exc)
+                db_wim_update["_admin.detailed-status"] = "ERROR {}: {}".format(
+                    step, exc
+                )
                 # Mark the WIM 'create' HA task as erroneous
-                operation_state = 'FAILED'
+                operation_state = "FAILED"
                 operation_details = "ERROR {}: {}".format(step, exc)
             try:
                 if db_wim_update:
                     self.update_db_2("wim_accounts", wim_id, db_wim_update)
                 # Register the WIM 'create' HA task either
                 # succesful or erroneous, or do nothing (if legacy NBI)
-                self.lcm_tasks.unlock_HA('wim', 'create', op_id,
-                                         operationState=operation_state,
-                                         detailed_status=operation_details)
+                self.lcm_tasks.unlock_HA(
+                    "wim",
+                    "create",
+                    op_id,
+                    operationState=operation_state,
+                    detailed_status=operation_details,
+                )
             except DbException as e:
                 self.logger.error(logging_text + "Cannot update database: {}".format(e))
             self.lcm_tasks.remove("wim_account", wim_id, order_id)
@@ -496,8 +604,8 @@ class WimLcm(LcmBase):
         # HA tasks and backward compatibility:
         # If 'wim_content' does not include 'op_id', we a running a legacy NBI version.
         # In such a case, HA is not supported by NBI, and the HA check always returns True
-        op_id = wim_content.pop('op_id', None)
-        if not self.lcm_tasks.lock_HA('wim', 'edit', op_id):
+        op_id = wim_content.pop("op_id", None)
+        if not self.lcm_tasks.lock_HA("wim", "edit", op_id):
             return
 
         wim_id = wim_content["_id"]
@@ -511,11 +619,15 @@ class WimLcm(LcmBase):
         step = "Getting wim-id='{}' from db".format(wim_id)
         try:
             # wait for any previous tasks in process
-            await self.lcm_tasks.waitfor_related_HA('wim', 'edit', op_id)
+            await self.lcm_tasks.waitfor_related_HA("wim", "edit", op_id)
 
             db_wim = self.db.get_one("wim_accounts", {"_id": wim_id})
 
-            if db_wim.get("_admin") and db_wim["_admin"].get("deployed") and db_wim["_admin"]["deployed"].get("RO"):
+            if (
+                db_wim.get("_admin")
+                and db_wim["_admin"].get("deployed")
+                and db_wim["_admin"]["deployed"].get("RO")
+            ):
 
                 RO_wim_id = db_wim["_admin"]["deployed"]["RO"]
                 step = "Editing wim at RO"
@@ -546,17 +658,21 @@ class WimLcm(LcmBase):
                 if "wim_password" in wim_content:
                     wim_account_RO["wim_password"] = wim_content["wim_password"]
                 if wim_content.get("wim_password"):
-                    wim_account_RO["wim_password"] = self.db.decrypt(wim_content["wim_password"],
-                                                                     schema_version=schema_version,
-                                                                     salt=wim_id)
+                    wim_account_RO["wim_password"] = self.db.decrypt(
+                        wim_content["wim_password"],
+                        schema_version=schema_version,
+                        salt=wim_id,
+                    )
                 if "config" in wim_content:
                     wim_account_RO["config"] = wim_content["config"]
                 if wim_content.get("config"):
                     for p in self.wim_config_encrypted:
                         if wim_content["config"].get(p):
-                            wim_account_RO["config"][p] = self.db.decrypt(wim_content["config"][p],
-                                                                          schema_version=schema_version,
-                                                                          salt=wim_id)
+                            wim_account_RO["config"][p] = self.db.decrypt(
+                                wim_content["config"][p],
+                                schema_version=schema_version,
+                                salt=wim_id,
+                            )
 
                 if "wim_user" in wim_content:
                     wim_content["wim_username"] = wim_content["wim_user"]
@@ -565,8 +681,8 @@ class WimLcm(LcmBase):
                 await RO.edit("wim_account", RO_wim_id, descriptor=wim_account_RO)
                 db_wim_update["_admin.operationalState"] = "ENABLED"
                 # Mark the WIM 'edit' HA task as successful
-            operation_state = 'COMPLETED'
-            operation_details = 'Done'
+            operation_state = "COMPLETED"
+            operation_details = "Done"
 
             self.logger.debug(logging_text + "Exit Ok RO_wim_id={}".format(RO_wim_id))
             return
@@ -575,23 +691,31 @@ class WimLcm(LcmBase):
             self.logger.error(logging_text + "Exit Exception {}".format(e))
             exc = e
         except Exception as e:
-            self.logger.critical(logging_text + "Exit Exception {}".format(e), exc_info=True)
+            self.logger.critical(
+                logging_text + "Exit Exception {}".format(e), exc_info=True
+            )
             exc = e
         finally:
             if exc and db_wim:
                 db_wim_update["_admin.operationalState"] = "ERROR"
-                db_wim_update["_admin.detailed-status"] = "ERROR {}: {}".format(step, exc)
+                db_wim_update["_admin.detailed-status"] = "ERROR {}: {}".format(
+                    step, exc
+                )
                 # Mark the WIM 'edit' HA task as erroneous
-                operation_state = 'FAILED'
+                operation_state = "FAILED"
                 operation_details = "ERROR {}: {}".format(step, exc)
             try:
                 if db_wim_update:
                     self.update_db_2("wim_accounts", wim_id, db_wim_update)
                 # Register the WIM 'edit' HA task either
                 # succesful or erroneous, or do nothing (if legacy NBI)
-                self.lcm_tasks.unlock_HA('wim', 'edit', op_id,
-                                         operationState=operation_state,
-                                         detailed_status=operation_details)
+                self.lcm_tasks.unlock_HA(
+                    "wim",
+                    "edit",
+                    op_id,
+                    operationState=operation_state,
+                    detailed_status=operation_details,
+                )
             except DbException as e:
                 self.logger.error(logging_text + "Cannot update database: {}".format(e))
             self.lcm_tasks.remove("wim_account", wim_id, order_id)
@@ -601,8 +725,8 @@ class WimLcm(LcmBase):
         # HA tasks and backward compatibility:
         # If 'vim_content' does not include 'op_id', we a running a legacy NBI version.
         # In such a case, HA is not supported by NBI, and the HA check always returns True
-        op_id = wim_content.pop('op_id', None)
-        if not self.lcm_tasks.lock_HA('wim', 'delete', op_id):
+        op_id = wim_content.pop("op_id", None)
+        if not self.lcm_tasks.lock_HA("wim", "delete", op_id):
             return
 
         wim_id = wim_content["_id"]
@@ -615,10 +739,14 @@ class WimLcm(LcmBase):
         step = "Getting wim from db"
         try:
             # wait for any previous tasks in process
-            await self.lcm_tasks.waitfor_related_HA('wim', 'delete', op_id)
+            await self.lcm_tasks.waitfor_related_HA("wim", "delete", op_id)
 
             db_wim = self.db.get_one("wim_accounts", {"_id": wim_id})
-            if db_wim.get("_admin") and db_wim["_admin"].get("deployed") and db_wim["_admin"]["deployed"].get("RO"):
+            if (
+                db_wim.get("_admin")
+                and db_wim["_admin"].get("deployed")
+                and db_wim["_admin"]["deployed"].get("RO")
+            ):
                 RO_wim_id = db_wim["_admin"]["deployed"]["RO"]
                 RO = ROclient.ROClient(self.loop, **self.ro_config)
                 step = "Detaching wim from RO tenant"
@@ -626,7 +754,10 @@ class WimLcm(LcmBase):
                     await RO.detach("wim_account", RO_wim_id)
                 except ROclient.ROClientException as e:
                     if e.http_code == 404:  # not found
-                        self.logger.debug(logging_text + "RO_wim_id={} already detached".format(RO_wim_id))
+                        self.logger.debug(
+                            logging_text
+                            + "RO_wim_id={} already detached".format(RO_wim_id)
+                        )
                     else:
                         raise
 
@@ -635,7 +766,10 @@ class WimLcm(LcmBase):
                     await RO.delete("wim", RO_wim_id)
                 except ROclient.ROClientException as e:
                     if e.http_code == 404:  # not found
-                        self.logger.debug(logging_text + "RO_wim_id={} already deleted".format(RO_wim_id))
+                        self.logger.debug(
+                            logging_text
+                            + "RO_wim_id={} already deleted".format(RO_wim_id)
+                        )
                     else:
                         raise
             else:
@@ -650,19 +784,27 @@ class WimLcm(LcmBase):
             self.logger.error(logging_text + "Exit Exception {}".format(e))
             exc = e
         except Exception as e:
-            self.logger.critical(logging_text + "Exit Exception {}".format(e), exc_info=True)
+            self.logger.critical(
+                logging_text + "Exit Exception {}".format(e), exc_info=True
+            )
             exc = e
         finally:
             self.lcm_tasks.remove("wim_account", wim_id, order_id)
             if exc and db_wim:
                 db_wim_update["_admin.operationalState"] = "ERROR"
-                db_wim_update["_admin.detailed-status"] = "ERROR {}: {}".format(step, exc)
+                db_wim_update["_admin.detailed-status"] = "ERROR {}: {}".format(
+                    step, exc
+                )
                 # Mark the WIM 'delete' HA task as erroneous
-                operation_state = 'FAILED'
+                operation_state = "FAILED"
                 operation_details = "ERROR {}: {}".format(step, exc)
-                self.lcm_tasks.unlock_HA('wim', 'delete', op_id,
-                                         operationState=operation_state,
-                                         detailed_status=operation_details)
+                self.lcm_tasks.unlock_HA(
+                    "wim",
+                    "delete",
+                    op_id,
+                    operationState=operation_state,
+                    detailed_status=operation_details,
+                )
             try:
                 if db_wim and db_wim_update:
                     self.update_db_2("wim_accounts", wim_id, db_wim_update)
@@ -674,7 +816,6 @@ class WimLcm(LcmBase):
 
 
 class SdnLcm(LcmBase):
-
     def __init__(self, msg, lcm_tasks, config, loop):
         """
         Init, Connect to database, filesystem storage, and messaging
@@ -682,7 +823,7 @@ class SdnLcm(LcmBase):
         :return: None
         """
 
-        self.logger = logging.getLogger('lcm.sdn')
+        self.logger = logging.getLogger("lcm.sdn")
         self.loop = loop
         self.lcm_tasks = lcm_tasks
         self.ro_config = config["ro_config"]
@@ -695,8 +836,8 @@ class SdnLcm(LcmBase):
         # If 'sdn_content' does not include 'op_id', we a running a legacy NBI version.
         # In such a case, HA is not supported by NBI, 'op_id' is None, and lock_HA() will do nothing.
         # Register 'create' task here for related future HA operations
-        op_id = sdn_content.pop('op_id', None)
-        self.lcm_tasks.lock_HA('sdn', 'create', op_id)
+        op_id = sdn_content.pop("op_id", None)
+        self.lcm_tasks.lock_HA("sdn", "create", op_id)
 
         sdn_id = sdn_content["_id"]
         logging_text = "Task sdn_create={} ".format(sdn_id)
@@ -723,7 +864,9 @@ class SdnLcm(LcmBase):
             sdn_RO.pop("schema_type", None)
             sdn_RO.pop("description", None)
             if sdn_RO.get("password"):
-                sdn_RO["password"] = self.db.decrypt(sdn_RO["password"], schema_version=schema_version, salt=sdn_id)
+                sdn_RO["password"] = self.db.decrypt(
+                    sdn_RO["password"], schema_version=schema_version, salt=sdn_id
+                )
 
             desc = await RO.create("sdn", descriptor=sdn_RO)
             RO_sdn_id = desc["uuid"]
@@ -731,31 +874,39 @@ class SdnLcm(LcmBase):
             db_sdn_update["_admin.operationalState"] = "ENABLED"
             self.logger.debug(logging_text + "Exit Ok RO_sdn_id={}".format(RO_sdn_id))
             # Mark the SDN 'create' HA task as successful
-            operation_state = 'COMPLETED'
-            operation_details = 'Done'
+            operation_state = "COMPLETED"
+            operation_details = "Done"
             return
 
         except (ROclient.ROClientException, DbException, asyncio.CancelledError) as e:
             self.logger.error(logging_text + "Exit Exception {}".format(e))
             exc = e
         except Exception as e:
-            self.logger.critical(logging_text + "Exit Exception {}".format(e), exc_info=True)
+            self.logger.critical(
+                logging_text + "Exit Exception {}".format(e), exc_info=True
+            )
             exc = e
         finally:
             if exc and db_sdn:
                 db_sdn_update["_admin.operationalState"] = "ERROR"
-                db_sdn_update["_admin.detailed-status"] = "ERROR {}: {}".format(step, exc)
+                db_sdn_update["_admin.detailed-status"] = "ERROR {}: {}".format(
+                    step, exc
+                )
                 # Mark the SDN 'create' HA task as erroneous
-                operation_state = 'FAILED'
+                operation_state = "FAILED"
                 operation_details = "ERROR {}: {}".format(step, exc)
             try:
                 if db_sdn and db_sdn_update:
                     self.update_db_2("sdns", sdn_id, db_sdn_update)
                 # Register the SDN 'create' HA task either
                 # succesful or erroneous, or do nothing (if legacy NBI)
-                self.lcm_tasks.unlock_HA('sdn', 'create', op_id,
-                                         operationState=operation_state,
-                                         detailed_status=operation_details)
+                self.lcm_tasks.unlock_HA(
+                    "sdn",
+                    "create",
+                    op_id,
+                    operationState=operation_state,
+                    detailed_status=operation_details,
+                )
             except DbException as e:
                 self.logger.error(logging_text + "Cannot update database: {}".format(e))
             self.lcm_tasks.remove("sdn", sdn_id, order_id)
@@ -765,8 +916,8 @@ class SdnLcm(LcmBase):
         # HA tasks and backward compatibility:
         # If 'sdn_content' does not include 'op_id', we a running a legacy NBI version.
         # In such a case, HA is not supported by NBI, and the HA check always returns True
-        op_id = sdn_content.pop('op_id', None)
-        if not self.lcm_tasks.lock_HA('sdn', 'edit', op_id):
+        op_id = sdn_content.pop("op_id", None)
+        if not self.lcm_tasks.lock_HA("sdn", "edit", op_id):
             return
 
         sdn_id = sdn_content["_id"]
@@ -779,11 +930,15 @@ class SdnLcm(LcmBase):
         step = "Getting sdn from db"
         try:
             # wait for any previous tasks in process
-            await self.lcm_tasks.waitfor_related_HA('sdn', 'edit', op_id)
+            await self.lcm_tasks.waitfor_related_HA("sdn", "edit", op_id)
 
             db_sdn = self.db.get_one("sdns", {"_id": sdn_id})
             RO_sdn_id = None
-            if db_sdn.get("_admin") and db_sdn["_admin"].get("deployed") and db_sdn["_admin"]["deployed"].get("RO"):
+            if (
+                db_sdn.get("_admin")
+                and db_sdn["_admin"].get("deployed")
+                and db_sdn["_admin"]["deployed"].get("RO")
+            ):
                 RO_sdn_id = db_sdn["_admin"]["deployed"]["RO"]
                 RO = ROclient.ROClient(self.loop, **self.ro_config)
                 step = "Editing sdn at RO"
@@ -794,13 +949,15 @@ class SdnLcm(LcmBase):
                 sdn_RO.pop("schema_type", None)
                 sdn_RO.pop("description", None)
                 if sdn_RO.get("password"):
-                    sdn_RO["password"] = self.db.decrypt(sdn_RO["password"], schema_version=schema_version, salt=sdn_id)
+                    sdn_RO["password"] = self.db.decrypt(
+                        sdn_RO["password"], schema_version=schema_version, salt=sdn_id
+                    )
                 if sdn_RO:
                     await RO.edit("sdn", RO_sdn_id, descriptor=sdn_RO)
                 db_sdn_update["_admin.operationalState"] = "ENABLED"
                 # Mark the SDN 'edit' HA task as successful
-            operation_state = 'COMPLETED'
-            operation_details = 'Done'
+            operation_state = "COMPLETED"
+            operation_details = "Done"
 
             self.logger.debug(logging_text + "Exit Ok RO_sdn_id={}".format(RO_sdn_id))
             return
@@ -809,23 +966,29 @@ class SdnLcm(LcmBase):
             self.logger.error(logging_text + "Exit Exception {}".format(e))
             exc = e
         except Exception as e:
-            self.logger.critical(logging_text + "Exit Exception {}".format(e), exc_info=True)
+            self.logger.critical(
+                logging_text + "Exit Exception {}".format(e), exc_info=True
+            )
             exc = e
         finally:
             if exc and db_sdn:
                 db_sdn["_admin.operationalState"] = "ERROR"
                 db_sdn["_admin.detailed-status"] = "ERROR {}: {}".format(step, exc)
                 # Mark the SDN 'edit' HA task as erroneous
-                operation_state = 'FAILED'
+                operation_state = "FAILED"
                 operation_details = "ERROR {}: {}".format(step, exc)
             try:
                 if db_sdn_update:
                     self.update_db_2("sdns", sdn_id, db_sdn_update)
                 # Register the SDN 'edit' HA task either
                 # succesful or erroneous, or do nothing (if legacy NBI)
-                self.lcm_tasks.unlock_HA('sdn', 'edit', op_id,
-                                         operationState=operation_state,
-                                         detailed_status=operation_details)
+                self.lcm_tasks.unlock_HA(
+                    "sdn",
+                    "edit",
+                    op_id,
+                    operationState=operation_state,
+                    detailed_status=operation_details,
+                )
             except DbException as e:
                 self.logger.error(logging_text + "Cannot update database: {}".format(e))
             self.lcm_tasks.remove("sdn", sdn_id, order_id)
@@ -835,8 +998,8 @@ class SdnLcm(LcmBase):
         # HA tasks and backward compatibility:
         # If 'vim_content' does not include 'op_id', we a running a legacy NBI version.
         # In such a case, HA is not supported by NBI, and the HA check always returns True
-        op_id = sdn_content.pop('op_id', None)
-        if not self.lcm_tasks.lock_HA('sdn', 'delete', op_id):
+        op_id = sdn_content.pop("op_id", None)
+        if not self.lcm_tasks.lock_HA("sdn", "delete", op_id):
             return
 
         sdn_id = sdn_content["_id"]
@@ -849,10 +1012,14 @@ class SdnLcm(LcmBase):
         step = "Getting sdn from db"
         try:
             # wait for any previous tasks in process
-            await self.lcm_tasks.waitfor_related_HA('sdn', 'delete', op_id)
+            await self.lcm_tasks.waitfor_related_HA("sdn", "delete", op_id)
 
             db_sdn = self.db.get_one("sdns", {"_id": sdn_id})
-            if db_sdn.get("_admin") and db_sdn["_admin"].get("deployed") and db_sdn["_admin"]["deployed"].get("RO"):
+            if (
+                db_sdn.get("_admin")
+                and db_sdn["_admin"].get("deployed")
+                and db_sdn["_admin"]["deployed"].get("RO")
+            ):
                 RO_sdn_id = db_sdn["_admin"]["deployed"]["RO"]
                 RO = ROclient.ROClient(self.loop, **self.ro_config)
                 step = "Deleting sdn from RO"
@@ -860,12 +1027,17 @@ class SdnLcm(LcmBase):
                     await RO.delete("sdn", RO_sdn_id)
                 except ROclient.ROClientException as e:
                     if e.http_code == 404:  # not found
-                        self.logger.debug(logging_text + "RO_sdn_id={} already deleted".format(RO_sdn_id))
+                        self.logger.debug(
+                            logging_text
+                            + "RO_sdn_id={} already deleted".format(RO_sdn_id)
+                        )
                     else:
                         raise
             else:
                 # nothing to delete
-                self.logger.error(logging_text + "Skipping. There is not RO information at database")
+                self.logger.error(
+                    logging_text + "Skipping. There is not RO information at database"
+                )
             self.db.del_one("sdns", {"_id": sdn_id})
             db_sdn = None
             self.logger.debug("sdn_delete task sdn_id={} Exit Ok".format(sdn_id))
@@ -875,18 +1047,24 @@ class SdnLcm(LcmBase):
             self.logger.error(logging_text + "Exit Exception {}".format(e))
             exc = e
         except Exception as e:
-            self.logger.critical(logging_text + "Exit Exception {}".format(e), exc_info=True)
+            self.logger.critical(
+                logging_text + "Exit Exception {}".format(e), exc_info=True
+            )
             exc = e
         finally:
             if exc and db_sdn:
                 db_sdn["_admin.operationalState"] = "ERROR"
                 db_sdn["_admin.detailed-status"] = "ERROR {}: {}".format(step, exc)
                 # Mark the SDN 'delete' HA task as erroneous
-                operation_state = 'FAILED'
+                operation_state = "FAILED"
                 operation_details = "ERROR {}: {}".format(step, exc)
-                self.lcm_tasks.unlock_HA('sdn', 'delete', op_id,
-                                         operationState=operation_state,
-                                         detailed_status=operation_details)
+                self.lcm_tasks.unlock_HA(
+                    "sdn",
+                    "delete",
+                    op_id,
+                    operationState=operation_state,
+                    detailed_status=operation_details,
+                )
             try:
                 if db_sdn and db_sdn_update:
                     self.update_db_2("sdns", sdn_id, db_sdn_update)
@@ -907,7 +1085,7 @@ class K8sClusterLcm(LcmBase):
         :return: None
         """
 
-        self.logger = logging.getLogger('lcm.k8scluster')
+        self.logger = logging.getLogger("lcm.k8scluster")
         self.loop = loop
         self.lcm_tasks = lcm_tasks
         self.vca_config = config["VCA"]
@@ -920,7 +1098,7 @@ class K8sClusterLcm(LcmBase):
             log=self.logger,
             on_update_db=None,
             db=self.db,
-            fs=self.fs
+            fs=self.fs,
         )
 
         self.helm3_k8scluster = K8sHelm3Connector(
@@ -929,7 +1107,7 @@ class K8sClusterLcm(LcmBase):
             fs=self.fs,
             log=self.logger,
             db=self.db,
-            on_update_db=None
+            on_update_db=None,
         )
 
         self.juju_k8scluster = K8sJujuConnector(
@@ -939,7 +1117,7 @@ class K8sClusterLcm(LcmBase):
             loop=self.loop,
             on_update_db=None,
             db=self.db,
-            fs=self.fs
+            fs=self.fs,
         )
 
         self.k8s_map = {
@@ -950,8 +1128,8 @@ class K8sClusterLcm(LcmBase):
 
     async def create(self, k8scluster_content, order_id):
 
-        op_id = k8scluster_content.pop('op_id', None)
-        if not self.lcm_tasks.lock_HA('k8scluster', 'create', op_id):
+        op_id = k8scluster_content.pop("op_id", None)
+        if not self.lcm_tasks.lock_HA("k8scluster", "create", op_id):
             return
 
         k8scluster_id = k8scluster_content["_id"]
@@ -965,8 +1143,13 @@ class K8sClusterLcm(LcmBase):
             step = "Getting k8scluster-id='{}' from db".format(k8scluster_id)
             self.logger.debug(logging_text + step)
             db_k8scluster = self.db.get_one("k8sclusters", {"_id": k8scluster_id})
-            self.db.encrypt_decrypt_fields(db_k8scluster.get("credentials"), 'decrypt', ['password', 'secret'],
-                                           schema_version=db_k8scluster["schema_version"], salt=db_k8scluster["_id"])
+            self.db.encrypt_decrypt_fields(
+                db_k8scluster.get("credentials"),
+                "decrypt",
+                ["password", "secret"],
+                schema_version=db_k8scluster["schema_version"],
+                salt=db_k8scluster["_id"],
+            )
             k8s_credentials = yaml.safe_dump(db_k8scluster.get("credentials"))
             pending_tasks = []
             task2name = {}
@@ -991,10 +1174,13 @@ class K8sClusterLcm(LcmBase):
             now = time()
 
             while pending_tasks:
-                _timeout = max(1, self.timeout_create - (time() - now))  # ensure not negative with max
+                _timeout = max(
+                    1, self.timeout_create - (time() - now)
+                )  # ensure not negative with max
                 step = "Waiting for k8scluster init tasks"
-                done, pending_tasks = await asyncio.wait(pending_tasks, timeout=_timeout,
-                                                         return_when=asyncio.FIRST_COMPLETED)
+                done, pending_tasks = await asyncio.wait(
+                    pending_tasks, timeout=_timeout, return_when=asyncio.FIRST_COMPLETED
+                )
                 if not done:
                     # timeout. Set timeout is reached and process pending as if they hase been finished
                     done = pending_tasks
@@ -1010,28 +1196,48 @@ class K8sClusterLcm(LcmBase):
                         exc = task.exception()
 
                     if exc:
-                        error_text_list.append("Failing init {}: {}".format(task_name, exc))
-                        db_k8scluster_update["_admin.{}.error_msg".format(task_name)] = str(exc)
+                        error_text_list.append(
+                            "Failing init {}: {}".format(task_name, exc)
+                        )
+                        db_k8scluster_update[
+                            "_admin.{}.error_msg".format(task_name)
+                        ] = str(exc)
                         db_k8scluster_update["_admin.{}.id".format(task_name)] = None
-                        db_k8scluster_update["_admin.{}.operationalState".format(task_name)] = "ERROR"
-                        self.logger.error(logging_text + "{} init fail: {}".format(task_name, exc),
-                                          exc_info=not isinstance(exc, (N2VCException, str)))
+                        db_k8scluster_update[
+                            "_admin.{}.operationalState".format(task_name)
+                        ] = "ERROR"
+                        self.logger.error(
+                            logging_text + "{} init fail: {}".format(task_name, exc),
+                            exc_info=not isinstance(exc, (N2VCException, str)),
+                        )
                     else:
                         k8s_id, uninstall_sw = task.result()
                         tasks_name_ok.append(task_name)
-                        self.logger.debug(logging_text + "{} init success. id={} created={}".format(
-                            task_name, k8s_id, uninstall_sw))
-                        db_k8scluster_update["_admin.{}.error_msg".format(task_name)] = None
+                        self.logger.debug(
+                            logging_text
+                            + "{} init success. id={} created={}".format(
+                                task_name, k8s_id, uninstall_sw
+                            )
+                        )
+                        db_k8scluster_update[
+                            "_admin.{}.error_msg".format(task_name)
+                        ] = None
                         db_k8scluster_update["_admin.{}.id".format(task_name)] = k8s_id
-                        db_k8scluster_update["_admin.{}.created".format(task_name)] = uninstall_sw
-                        db_k8scluster_update["_admin.{}.operationalState".format(task_name)] = "ENABLED"
+                        db_k8scluster_update[
+                            "_admin.{}.created".format(task_name)
+                        ] = uninstall_sw
+                        db_k8scluster_update[
+                            "_admin.{}.operationalState".format(task_name)
+                        ] = "ENABLED"
                 # update database
                 step = "Updating database for " + task_name
                 self.update_db_2("k8sclusters", k8scluster_id, db_k8scluster_update)
             if tasks_name_ok:
                 operation_details = "ready for " + ", ".join(tasks_name_ok)
                 operation_state = "COMPLETED"
-                db_k8scluster_update["_admin.operationalState"] = "ENABLED" if not error_text_list else "DEGRADED"
+                db_k8scluster_update["_admin.operationalState"] = (
+                    "ENABLED" if not error_text_list else "DEGRADED"
+                )
                 operation_details += "; " + ";".join(error_text_list)
             else:
                 db_k8scluster_update["_admin.operationalState"] = "ERROR"
@@ -1042,25 +1248,42 @@ class K8sClusterLcm(LcmBase):
             exc = None
 
         except Exception as e:
-            if isinstance(e, (LcmException, DbException, K8sException, N2VCException, asyncio.CancelledError)):
+            if isinstance(
+                e,
+                (
+                    LcmException,
+                    DbException,
+                    K8sException,
+                    N2VCException,
+                    asyncio.CancelledError,
+                ),
+            ):
                 self.logger.error(logging_text + "Exit Exception {}".format(e))
             else:
-                self.logger.critical(logging_text + "Exit Exception {}".format(e), exc_info=True)
+                self.logger.critical(
+                    logging_text + "Exit Exception {}".format(e), exc_info=True
+                )
             exc = e
         finally:
             if exc and db_k8scluster:
                 db_k8scluster_update["_admin.operationalState"] = "ERROR"
-                db_k8scluster_update["_admin.detailed-status"] = "ERROR {}: {}".format(step, exc)
-                operation_state = 'FAILED'
+                db_k8scluster_update["_admin.detailed-status"] = "ERROR {}: {}".format(
+                    step, exc
+                )
+                operation_state = "FAILED"
                 operation_details = "ERROR {}: {}".format(step, exc)
             try:
                 if db_k8scluster and db_k8scluster_update:
                     self.update_db_2("k8sclusters", k8scluster_id, db_k8scluster_update)
 
                 # Register the operation and unlock
-                self.lcm_tasks.unlock_HA('k8scluster', 'create', op_id,
-                                         operationState=operation_state,
-                                         detailed_status=operation_details)
+                self.lcm_tasks.unlock_HA(
+                    "k8scluster",
+                    "create",
+                    op_id,
+                    operationState=operation_state,
+                    detailed_status=operation_details,
+                )
             except DbException as e:
                 self.logger.error(logging_text + "Cannot update database: {}".format(e))
             self.lcm_tasks.remove("k8scluster", k8scluster_id, order_id)
@@ -1071,8 +1294,8 @@ class K8sClusterLcm(LcmBase):
         # If 'vim_content' does not include 'op_id', we a running a legacy NBI version.
         # In such a case, HA is not supported by NBI, 'op_id' is None, and lock_HA() will do nothing.
         # Register 'delete' task here for related future HA operations
-        op_id = k8scluster_content.pop('op_id', None)
-        if not self.lcm_tasks.lock_HA('k8scluster', 'delete', op_id):
+        op_id = k8scluster_content.pop("op_id", None)
+        if not self.lcm_tasks.lock_HA("k8scluster", "delete", op_id):
             return
 
         k8scluster_id = k8scluster_content["_id"]
@@ -1093,7 +1316,10 @@ class K8sClusterLcm(LcmBase):
             cluster_removed = True
             if k8s_jb_id:  # delete in reverse order of creation
                 step = "Removing juju-bundle '{}'".format(k8s_jb_id)
-                uninstall_sw = deep_get(db_k8scluster, ("_admin", "juju-bundle", "created")) or False
+                uninstall_sw = (
+                    deep_get(db_k8scluster, ("_admin", "juju-bundle", "created"))
+                    or False
+                )
                 cluster_removed = await self.juju_k8scluster.reset(
                     cluster_uuid=k8s_jb_id,
                     uninstall_sw=uninstall_sw,
@@ -1104,28 +1330,46 @@ class K8sClusterLcm(LcmBase):
 
             if k8s_hc_id:
                 step = "Removing helm-chart '{}'".format(k8s_hc_id)
-                uninstall_sw = deep_get(db_k8scluster, ("_admin", "helm-chart", "created")) or False
-                cluster_removed = await self.helm2_k8scluster.reset(cluster_uuid=k8s_hc_id, uninstall_sw=uninstall_sw)
+                uninstall_sw = (
+                    deep_get(db_k8scluster, ("_admin", "helm-chart", "created"))
+                    or False
+                )
+                cluster_removed = await self.helm2_k8scluster.reset(
+                    cluster_uuid=k8s_hc_id, uninstall_sw=uninstall_sw
+                )
                 db_k8scluster_update["_admin.helm-chart.id"] = None
                 db_k8scluster_update["_admin.helm-chart.operationalState"] = "DISABLED"
 
             if k8s_h3c_id:
                 step = "Removing helm-chart-v3 '{}'".format(k8s_hc_id)
-                uninstall_sw = deep_get(db_k8scluster, ("_admin", "helm-chart-v3", "created")) or False
-                cluster_removed = await self.helm3_k8scluster.reset(cluster_uuid=k8s_h3c_id, uninstall_sw=uninstall_sw)
+                uninstall_sw = (
+                    deep_get(db_k8scluster, ("_admin", "helm-chart-v3", "created"))
+                    or False
+                )
+                cluster_removed = await self.helm3_k8scluster.reset(
+                    cluster_uuid=k8s_h3c_id, uninstall_sw=uninstall_sw
+                )
                 db_k8scluster_update["_admin.helm-chart-v3.id"] = None
-                db_k8scluster_update["_admin.helm-chart-v3.operationalState"] = "DISABLED"
+                db_k8scluster_update[
+                    "_admin.helm-chart-v3.operationalState"
+                ] = "DISABLED"
 
             # Try to remove from cluster_inserted to clean old versions
             if k8s_hc_id and cluster_removed:
                 step = "Removing k8scluster='{}' from k8srepos".format(k8scluster_id)
                 self.logger.debug(logging_text + step)
-                db_k8srepo_list = self.db.get_list("k8srepos", {"_admin.cluster-inserted": k8s_hc_id})
+                db_k8srepo_list = self.db.get_list(
+                    "k8srepos", {"_admin.cluster-inserted": k8s_hc_id}
+                )
                 for k8srepo in db_k8srepo_list:
                     try:
                         cluster_list = k8srepo["_admin"]["cluster-inserted"]
                         cluster_list.remove(k8s_hc_id)
-                        self.update_db_2("k8srepos", k8srepo["_id"], {"_admin.cluster-inserted": cluster_list})
+                        self.update_db_2(
+                            "k8srepos",
+                            k8srepo["_id"],
+                            {"_admin.cluster-inserted": cluster_list},
+                        )
                     except Exception as e:
                         self.logger.error("{}: {}".format(step, e))
             self.db.del_one("k8sclusters", {"_id": k8scluster_id})
@@ -1133,20 +1377,33 @@ class K8sClusterLcm(LcmBase):
             self.logger.debug(logging_text + "Done")
 
         except Exception as e:
-            if isinstance(e, (LcmException, DbException, K8sException, N2VCException, asyncio.CancelledError)):
+            if isinstance(
+                e,
+                (
+                    LcmException,
+                    DbException,
+                    K8sException,
+                    N2VCException,
+                    asyncio.CancelledError,
+                ),
+            ):
                 self.logger.error(logging_text + "Exit Exception {}".format(e))
             else:
-                self.logger.critical(logging_text + "Exit Exception {}".format(e), exc_info=True)
+                self.logger.critical(
+                    logging_text + "Exit Exception {}".format(e), exc_info=True
+                )
             exc = e
         finally:
             if exc and db_k8scluster:
                 db_k8scluster_update["_admin.operationalState"] = "ERROR"
-                db_k8scluster_update["_admin.detailed-status"] = "ERROR {}: {}".format(step, exc)
+                db_k8scluster_update["_admin.detailed-status"] = "ERROR {}: {}".format(
+                    step, exc
+                )
                 # Mark the WIM 'create' HA task as erroneous
-                operation_state = 'FAILED'
+                operation_state = "FAILED"
                 operation_details = "ERROR {}: {}".format(step, exc)
             else:
-                operation_state = 'COMPLETED'
+                operation_state = "COMPLETED"
                 operation_details = "deleted"
 
             try:
@@ -1154,9 +1411,13 @@ class K8sClusterLcm(LcmBase):
                     self.update_db_2("k8sclusters", k8scluster_id, db_k8scluster_update)
                 # Register the K8scluster 'delete' HA task either
                 # succesful or erroneous, or do nothing (if legacy NBI)
-                self.lcm_tasks.unlock_HA('k8scluster', 'delete', op_id,
-                                         operationState=operation_state,
-                                         detailed_status=operation_details)
+                self.lcm_tasks.unlock_HA(
+                    "k8scluster",
+                    "delete",
+                    op_id,
+                    operationState=operation_state,
+                    detailed_status=operation_details,
+                )
             except DbException as e:
                 self.logger.error(logging_text + "Cannot update database: {}".format(e))
             self.lcm_tasks.remove("k8scluster", k8scluster_id, order_id)
@@ -1180,10 +1441,7 @@ class VcaLcm(LcmBase):
 
         # create N2VC connector
         self.n2vc = N2VCJujuConnector(
-            log=self.logger,
-            loop=self.loop,
-            fs=self.fs,
-            db=self.db
+            log=self.logger, loop=self.loop, fs=self.fs, db=self.db
         )
 
     def _get_vca_by_id(self, vca_id: str) -> dict:
@@ -1192,7 +1450,8 @@ class VcaLcm(LcmBase):
             db_vca,
             "decrypt",
             ["secret", "cacert"],
-            schema_version=db_vca["schema_version"], salt=db_vca["_id"]
+            schema_version=db_vca["schema_version"],
+            salt=db_vca["_id"],
         )
         return db_vca
 
@@ -1208,7 +1467,9 @@ class VcaLcm(LcmBase):
         db_vca_update = {}
 
         try:
-            self.logger.debug("Task vca_create={} {}".format(vca_id, "Getting vca from db"))
+            self.logger.debug(
+                "Task vca_create={} {}".format(vca_id, "Getting vca from db")
+            )
             db_vca = self._get_vca_by_id(vca_id)
 
             task = asyncio.ensure_future(
@@ -1221,13 +1482,21 @@ class VcaLcm(LcmBase):
             await asyncio.wait([task], return_when=asyncio.FIRST_COMPLETED)
             if task.exception():
                 raise task.exception()
-            self.logger.debug("Task vca_create={} {}".format(vca_id, "vca registered and validated successfully"))
+            self.logger.debug(
+                "Task vca_create={} {}".format(
+                    vca_id, "vca registered and validated successfully"
+                )
+            )
             db_vca_update["_admin.operationalState"] = "ENABLED"
             db_vca_update["_admin.detailed-status"] = "Connectivity: ok"
             operation_details = "VCA validated"
             operation_state = "COMPLETED"
 
-            self.logger.debug("Task vca_create={} {}".format(vca_id, "Done. Result: {}".format(operation_state)))
+            self.logger.debug(
+                "Task vca_create={} {}".format(
+                    vca_id, "Done. Result: {}".format(operation_state)
+                )
+            )
 
         except Exception as e:
             error_msg = "Failed with exception: {}".format(e)
@@ -1246,10 +1515,14 @@ class VcaLcm(LcmBase):
                     "create",
                     op_id,
                     operationState=operation_state,
-                    detailed_status=operation_details
+                    detailed_status=operation_details,
                 )
             except DbException as e:
-                self.logger.error("Task vca_create={} {}".format(vca_id, "Cannot update database: {}".format(e)))
+                self.logger.error(
+                    "Task vca_create={} {}".format(
+                        vca_id, "Cannot update database: {}".format(e)
+                    )
+                )
             self.lcm_tasks.remove("vca", vca_id, order_id)
 
     async def delete(self, vca_content, order_id):
@@ -1266,13 +1539,19 @@ class VcaLcm(LcmBase):
         vca_id = vca_content["_id"]
 
         try:
-            self.logger.debug("Task vca_delete={} {}".format(vca_id, "Deleting vca from db"))
+            self.logger.debug(
+                "Task vca_delete={} {}".format(vca_id, "Deleting vca from db")
+            )
             self.db.del_one("vca", {"_id": vca_id})
             db_vca_update = None
             operation_details = "deleted"
             operation_state = "COMPLETED"
 
-            self.logger.debug("Task vca_delete={} {}".format(vca_id, "Done. Result: {}".format(operation_state)))
+            self.logger.debug(
+                "Task vca_delete={} {}".format(
+                    vca_id, "Done. Result: {}".format(operation_state)
+                )
+            )
         except Exception as e:
             error_msg = "Failed with exception: {}".format(e)
             self.logger.error("Task vca_delete={} {}".format(vca_id, error_msg))
@@ -1291,12 +1570,15 @@ class VcaLcm(LcmBase):
                     detailed_status=operation_details,
                 )
             except DbException as e:
-                self.logger.error("Task vca_delete={} {}".format(vca_id, "Cannot update database: {}".format(e)))
+                self.logger.error(
+                    "Task vca_delete={} {}".format(
+                        vca_id, "Cannot update database: {}".format(e)
+                    )
+                )
             self.lcm_tasks.remove("vca", vca_id, order_id)
 
 
 class K8sRepoLcm(LcmBase):
-
     def __init__(self, msg, lcm_tasks, config, loop):
         """
         Init, Connect to database, filesystem storage, and messaging
@@ -1304,7 +1586,7 @@ class K8sRepoLcm(LcmBase):
         :return: None
         """
 
-        self.logger = logging.getLogger('lcm.k8srepo')
+        self.logger = logging.getLogger("lcm.k8srepo")
         self.loop = loop
         self.lcm_tasks = lcm_tasks
         self.vca_config = config["VCA"]
@@ -1317,7 +1599,7 @@ class K8sRepoLcm(LcmBase):
             fs=self.fs,
             log=self.logger,
             db=self.db,
-            on_update_db=None
+            on_update_db=None,
         )
 
     async def create(self, k8srepo_content, order_id):
@@ -1327,8 +1609,8 @@ class K8sRepoLcm(LcmBase):
         # In such a case, HA is not supported by NBI, 'op_id' is None, and lock_HA() will do nothing.
         # Register 'create' task here for related future HA operations
 
-        op_id = k8srepo_content.pop('op_id', None)
-        if not self.lcm_tasks.lock_HA('k8srepo', 'create', op_id):
+        op_id = k8srepo_content.pop("op_id", None)
+        if not self.lcm_tasks.lock_HA("k8srepo", "create", op_id):
             return
 
         k8srepo_id = k8srepo_content.get("_id")
@@ -1338,33 +1620,49 @@ class K8sRepoLcm(LcmBase):
         db_k8srepo = None
         db_k8srepo_update = {}
         exc = None
-        operation_state = 'COMPLETED'
-        operation_details = ''
+        operation_state = "COMPLETED"
+        operation_details = ""
         try:
             step = "Getting k8srepo-id='{}' from db".format(k8srepo_id)
             self.logger.debug(logging_text + step)
             db_k8srepo = self.db.get_one("k8srepos", {"_id": k8srepo_id})
             db_k8srepo_update["_admin.operationalState"] = "ENABLED"
         except Exception as e:
-            self.logger.error(logging_text + "Exit Exception {}".format(e),
-                              exc_info=not isinstance(e, (LcmException, DbException, K8sException, N2VCException,
-                                                          asyncio.CancelledError)))
+            self.logger.error(
+                logging_text + "Exit Exception {}".format(e),
+                exc_info=not isinstance(
+                    e,
+                    (
+                        LcmException,
+                        DbException,
+                        K8sException,
+                        N2VCException,
+                        asyncio.CancelledError,
+                    ),
+                ),
+            )
             exc = e
         finally:
             if exc and db_k8srepo:
                 db_k8srepo_update["_admin.operationalState"] = "ERROR"
-                db_k8srepo_update["_admin.detailed-status"] = "ERROR {}: {}".format(step, exc)
+                db_k8srepo_update["_admin.detailed-status"] = "ERROR {}: {}".format(
+                    step, exc
+                )
                 # Mark the WIM 'create' HA task as erroneous
-                operation_state = 'FAILED'
+                operation_state = "FAILED"
                 operation_details = "ERROR {}: {}".format(step, exc)
             try:
                 if db_k8srepo_update:
                     self.update_db_2("k8srepos", k8srepo_id, db_k8srepo_update)
                 # Register the K8srepo 'create' HA task either
                 # succesful or erroneous, or do nothing (if legacy NBI)
-                self.lcm_tasks.unlock_HA('k8srepo', 'create', op_id,
-                                         operationState=operation_state,
-                                         detailed_status=operation_details)
+                self.lcm_tasks.unlock_HA(
+                    "k8srepo",
+                    "create",
+                    op_id,
+                    operationState=operation_state,
+                    detailed_status=operation_details,
+                )
             except DbException as e:
                 self.logger.error(logging_text + "Cannot update database: {}".format(e))
             self.lcm_tasks.remove("k8srepo", k8srepo_id, order_id)
@@ -1375,8 +1673,8 @@ class K8sRepoLcm(LcmBase):
         # If 'vim_content' does not include 'op_id', we a running a legacy NBI version.
         # In such a case, HA is not supported by NBI, 'op_id' is None, and lock_HA() will do nothing.
         # Register 'delete' task here for related future HA operations
-        op_id = k8srepo_content.pop('op_id', None)
-        if not self.lcm_tasks.lock_HA('k8srepo', 'delete', op_id):
+        op_id = k8srepo_content.pop("op_id", None)
+        if not self.lcm_tasks.lock_HA("k8srepo", "delete", op_id):
             return
 
         k8srepo_id = k8srepo_content.get("_id")
@@ -1387,33 +1685,49 @@ class K8sRepoLcm(LcmBase):
         db_k8srepo_update = {}
 
         exc = None
-        operation_state = 'COMPLETED'
-        operation_details = ''
+        operation_state = "COMPLETED"
+        operation_details = ""
         try:
             step = "Getting k8srepo-id='{}' from db".format(k8srepo_id)
             self.logger.debug(logging_text + step)
             db_k8srepo = self.db.get_one("k8srepos", {"_id": k8srepo_id})
 
         except Exception as e:
-            self.logger.error(logging_text + "Exit Exception {}".format(e),
-                              exc_info=not isinstance(e, (LcmException, DbException, K8sException, N2VCException,
-                                                          asyncio.CancelledError)))
+            self.logger.error(
+                logging_text + "Exit Exception {}".format(e),
+                exc_info=not isinstance(
+                    e,
+                    (
+                        LcmException,
+                        DbException,
+                        K8sException,
+                        N2VCException,
+                        asyncio.CancelledError,
+                    ),
+                ),
+            )
             exc = e
         finally:
             if exc and db_k8srepo:
                 db_k8srepo_update["_admin.operationalState"] = "ERROR"
-                db_k8srepo_update["_admin.detailed-status"] = "ERROR {}: {}".format(step, exc)
+                db_k8srepo_update["_admin.detailed-status"] = "ERROR {}: {}".format(
+                    step, exc
+                )
                 # Mark the WIM 'create' HA task as erroneous
-                operation_state = 'FAILED'
+                operation_state = "FAILED"
                 operation_details = "ERROR {}: {}".format(step, exc)
             try:
                 if db_k8srepo_update:
                     self.update_db_2("k8srepos", k8srepo_id, db_k8srepo_update)
                 # Register the K8srepo 'delete' HA task either
                 # succesful or erroneous, or do nothing (if legacy NBI)
-                self.lcm_tasks.unlock_HA('k8srepo', 'delete', op_id,
-                                         operationState=operation_state,
-                                         detailed_status=operation_details)
+                self.lcm_tasks.unlock_HA(
+                    "k8srepo",
+                    "delete",
+                    op_id,
+                    operationState=operation_state,
+                    detailed_status=operation_details,
+                )
                 self.db.del_one("k8srepos", {"_id": k8srepo_id})
             except DbException as e:
                 self.logger.error(logging_text + "Cannot update database: {}".format(e))