sbcli-dev 17.4.44__tar.gz → 17.4.45__tar.gz

{sbcli_dev-17.4.44 → sbcli_dev-17.4.45}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sbcli-dev
-Version: 17.4.44
+Version: 17.4.45
 Summary: CLI for managing SimplyBlock cluster
 Home-page: https://www.simplyblock.io/
 Author: Hamdy

{sbcli_dev-17.4.44 → sbcli_dev-17.4.45}/sbcli_dev.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sbcli-dev
-Version: 17.4.44
+Version: 17.4.45
 Summary: CLI for managing SimplyBlock cluster
 Home-page: https://www.simplyblock.io/
 Author: Hamdy

{sbcli_dev-17.4.44 → sbcli_dev-17.4.45}/simplyblock_core/controllers/health_controller.py

@@ -130,7 +130,10 @@ def _check_node_hublvol(node: StorageNode, node_bdev_names=None, node_lvols_nqns
         if not node_bdev_names:
             ret = rpc_client.get_bdevs()
             if ret:
-                node_bdev_names = [b['name'] for b in ret]
+                for b in ret:
+                    node_bdev_names[b['name']] = b
+                    for al in b['aliases']:
+                        node_bdev_names[al] = b
             else:
                 node_bdev_names = []
 
@@ -619,8 +622,9 @@ def check_lvol_on_node(lvol_id, node_id, node_bdev_names=None, node_lvols_nqns=N
     if not node_bdev_names:
         node_bdev_names = {}
         ret = rpc_client.get_bdevs()
-        for bdev in ret:
-            node_bdev_names[bdev['name']] = bdev
+        if ret:
+            for bdev in ret:
+                node_bdev_names[bdev['name']] = bdev
 
     if not node_lvols_nqns:
         node_lvols_nqns = {}

{sbcli_dev-17.4.44 → sbcli_dev-17.4.45}/simplyblock_core/env_var

@@ -1,5 +1,5 @@
 SIMPLY_BLOCK_COMMAND_NAME=sbcli-dev
-SIMPLY_BLOCK_VERSION=17.4.44
+SIMPLY_BLOCK_VERSION=17.4.45
 
 SIMPLY_BLOCK_DOCKER_IMAGE=public.ecr.aws/simply-block/simplyblock:main
 SIMPLY_BLOCK_SPDK_ULTRA_IMAGE=public.ecr.aws/simply-block/ultra:main-latest

{sbcli_dev-17.4.44 → sbcli_dev-17.4.45}/simplyblock_core/services/lvol_stat_collector.py

@@ -272,10 +272,10 @@ while True:
                     capacity_dict = {}
                     stats = []
                     logger.info("Getting lVol stats: %s from node: %s", lvol.uuid, snode.get_id())
-                    if lvol.lvol_uuid in all_node_lvols_stats[snode.get_id()]:
+                    if snode.get_id() in all_node_lvols_stats and lvol.lvol_uuid in all_node_lvols_stats[snode.get_id()]:
                         stats.append(all_node_lvols_stats[snode.get_id()][lvol.lvol_uuid])
 
-                    if lvol.lvol_uuid in all_node_bdev_names[snode.get_id()]:
+                    if snode.get_id() in all_node_bdev_names and lvol.lvol_uuid in all_node_bdev_names[snode.get_id()]:
                         capacity_dict = all_node_bdev_names[snode.get_id()][lvol.lvol_uuid]
 
                     if lvol.ha_type == "ha":

{sbcli_dev-17.4.44 → sbcli_dev-17.4.45}/simplyblock_core/storage_node_ops.py

@@ -2134,7 +2134,7 @@ def shutdown_storage_node(node_id, force=False):
     logger.info("Stopping SPDK")
     if health_controller._check_node_api(snode.mgmt_ip):
         snode_api = SNodeClient(snode.api_endpoint, timeout=30, retry=1)
-        results, err = snode_api.spdk_process_kill(snode.rpc_port)
+        snode_api.spdk_process_kill(snode.rpc_port)
 
     logger.info("Setting node status to offline")
     set_node_status(node_id, StorageNode.STATUS_OFFLINE)

{sbcli_dev-17.4.44 → sbcli_dev-17.4.45}/simplyblock_web/blueprints/snode_ops.py

@@ -204,7 +204,7 @@ def spdk_process_start():
             f"RPC_PORT={rpc_port}",
             f"ssd_pcie={ssd_pcie_params}",
             f"PCI_ALLOWED={ssd_pcie_list}",
-            f"TOTAL_HP={total_mem}",
+            f"TOTAL_HP={total_mem_mib}",
         ]
         # restart_policy={"Name": "on-failure", "MaximumRetryCount": 99}
     )