sbcli-pre 25.3.5__tar.gz → 25.3.7__tar.gz

{sbcli-pre-25.3.5 → sbcli-pre-25.3.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: sbcli-pre
-Version: 25.3.5
+Version: 25.3.7
 Summary: CLI for managing SimplyBlock cluster
 Home-page: https://www.simplyblock.io/
 Author: Hamdy

{sbcli-pre-25.3.5 → sbcli-pre-25.3.7}/sbcli_pre.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: sbcli-pre
-Version: 25.3.5
+Version: 25.3.7
 Summary: CLI for managing SimplyBlock cluster
 Home-page: https://www.simplyblock.io/
 Author: Hamdy

{sbcli-pre-25.3.5 → sbcli-pre-25.3.7}/simplyblock_core/env_var

@@ -1,5 +1,5 @@
 SIMPLY_BLOCK_COMMAND_NAME=sbcli-pre
-SIMPLY_BLOCK_VERSION=25.3.5
+SIMPLY_BLOCK_VERSION=25.3.7
 
 SIMPLY_BLOCK_DOCKER_IMAGE=public.ecr.aws/simply-block/simplyblock:R25.3-PRE-Hotfix
 SIMPLY_BLOCK_SPDK_ULTRA_IMAGE=public.ecr.aws/simply-block/ultra:R25.3-PRE-latest

{sbcli-pre-25.3.5 → sbcli-pre-25.3.7}/simplyblock_core/services/lvol_monitor.py

@@ -83,17 +83,18 @@ while True:
                             sec_node.mgmt_ip, sec_node.rpc_port,
                             sec_node.rpc_username, sec_node.rpc_password, timeout=3, retry=2)
                         ret = sec_rpc_client.get_bdevs()
-                        for bdev in ret:
-                            sec_node_bdev_names[bdev['name']] = bdev
+                        if ret:
+                            for bdev in ret:
+                                sec_node_bdev_names[bdev['name']] = bdev
 
                         ret = sec_rpc_client.subsystem_list()
-                        for sub in ret:
-                            sec_node_lvols_nqns[sub['nqn']] = sub
+                        if ret:
+                            for sub in ret:
+                                sec_node_lvols_nqns[sub['nqn']] = sub
 
                 for lvol in db_controller.get_lvols_by_node_id(snode.get_id()):
 
-                    if lvol.io_error:
-                        logger.debug(f"Skipping LVol health check because of io_error {lvol.get_id()}")
+                    if lvol.status == LVol.STATUS_IN_CREATION:
                         continue
 
                     if lvol.status == lvol.STATUS_IN_DELETION:
@@ -116,12 +117,10 @@ while True:
                         continue
 
                     passed = True
-
-                    if lvol.ha_type == 'single':
-                        ret = health_controller.check_lvol_on_node(
-                            lvol.get_id(), lvol.node_id, node_bdev_names, node_lvols_nqns)
-                        if not ret:
-                            passed = False
+                    ret = health_controller.check_lvol_on_node(
+                        lvol.get_id(), lvol.node_id, node_bdev_names, node_lvols_nqns)
+                    if not ret:
+                        passed = False
 
 
                     if lvol.ha_type == "ha":
@@ -132,19 +131,20 @@ while True:
                             if not ret:
                                 passed = False
 
+                    if snode.lvstore_status == "ready":
 
-                    logger.info(f"LVol: {lvol.get_id()}, is healthy: {passed}")
-                    set_lvol_health_check(lvol, passed)
-                    if passed:
-                        set_lvol_status(lvol, LVol.STATUS_ONLINE)
+                        logger.info(f"LVol: {lvol.get_id()}, is healthy: {passed}")
+                        set_lvol_health_check(lvol, passed)
+                        if passed:
+                            set_lvol_status(lvol, LVol.STATUS_ONLINE)
 
-                for snap in db_controller.get_snapshots_by_node_id(snode.get_id()):
-                    if snap.snap_bdev in node_bdev_names:
-                        logger.info(f"Checking bdev: {snap.snap_bdev} ... ok")
-                        set_snapshot_health_check(snap, True)
-                    else:
-                        logger.error(f"Checking bdev: {snap.snap_bdev} ... failed")
-                        set_snapshot_health_check(snap, False)
-                        passed = False
+                        for snap in db_controller.get_snapshots_by_node_id(snode.get_id()):
+                            if snap.snap_bdev in node_bdev_names:
+                                logger.info(f"Checking bdev: {snap.snap_bdev} ... ok")
+                                set_snapshot_health_check(snap, True)
+                            else:
+                                logger.error(f"Checking bdev: {snap.snap_bdev} ... failed")
+                                set_snapshot_health_check(snap, False)
+                                passed = False
 
     time.sleep(constants.LVOL_MONITOR_INTERVAL_SEC)

{sbcli-pre-25.3.5 → sbcli-pre-25.3.7}/simplyblock_core/storage_node_ops.py

@@ -2877,7 +2877,10 @@ def recreate_lvstore_on_sec(snode):
         node.lvstore_status = "in_creation"
         node.write_to_db()
 
-        lvol_list = db_controller.get_lvols_by_node_id(node.get_id())
+        lvol_list = []
+        for lv in db_controller.get_lvols_by_node_id(node.get_id()):
+            if lv.status not in [LVol.STATUS_IN_DELETION, LVol.STATUS_IN_CREATION]:
+                lvol_list.append(lv)
 
         ret, err = _create_bdev_stack(snode, node.lvstore_stack, primary_node=node)
         if err:
@@ -2960,7 +2963,10 @@ def recreate_lvstore(snode):
     sec_node = db_controller.get_storage_node_by_id(snode.secondary_node_id)
     sec_node_api = SNodeClient(sec_node.api_endpoint)
 
-    lvol_list = db_controller.get_lvols_by_node_id(snode.get_id())
+    lvol_list = []
+    for lv in db_controller.get_lvols_by_node_id(snode.get_id()):
+        if lv.status not in [LVol.STATUS_IN_DELETION, LVol.STATUS_IN_CREATION]:
+            lvol_list.append(lv)
 
     prim_node_suspend = False
     if sec_node:

{sbcli-pre-25.3.5 → sbcli-pre-25.3.7}/simplyblock_web/blueprints/web_api_snapshot.py

@@ -2,6 +2,7 @@
 # encoding: utf-8
 
 import logging
+import time
 
 from flask import Blueprint
 from flask import request
@@ -45,7 +46,9 @@ def list_snapshots():
     for snap in snaps:
         if snap.cluster_id != cluster_id:
             continue
-        data.append(snap.get_clean_dict())
+        d = snap.get_clean_dict()
+        d["created_at"] = str(snap.created_at)
+        data.append(d)
     return utils.get_response(data)