Merge pull request #585 from yuanyu-ghca/hpe_3par_20221028

Change to lower case and add digital judgment
sodafoundation · Oct 28, 2022 · dff317c · dff317c
2 parents f5b25eb + 8023432
commit dff317c
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/delfin/drivers/hpe/hpe_3par/component_handler.py b/delfin/drivers/hpe/hpe_3par/component_handler.py
@@ -63,7 +63,7 @@ def get_storage(self, context):
                 # Check the hardware and software health
                 # status of the storage system
                 re_str = self.ssh_handler.get_health_state()
-                if 'degraded' in re_str or 'failed' in re_str:
+                if 'degraded' in re_str.lower() or 'failed' in re_str.lower():
                     status = constants.StorageStatus.ABNORMAL
             except Exception:
                 status = constants.StorageStatus.ABNORMAL
@@ -287,7 +287,7 @@ def list_disks(self, storage_id):
                         model = inventory_map.get('disk_model')
                         firmware = inventory_map.get('disk_fw_rev')
                 speed = None
-                if disk.get('rpm'):
+                if str(disk.get('rpm')).isdigit():
                     speed = int(disk.get('rpm')) * units.k
                 disk_model = {
                     'name': disk.get('cagepos'),