]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/devicehealth: do not cast int options to int
authorKefu Chai <kchai@redhat.com>
Sat, 26 Dec 2020 08:52:19 +0000 (16:52 +0800)
committerKefu Chai <kchai@redhat.com>
Sun, 27 Dec 2020 14:50:10 +0000 (22:50 +0800)
since all of them are already typed, there is no need to cast them to int anymore.

Signed-off-by: Kefu Chai <kchai@redhat.com>
src/pybind/mgr/devicehealth/module.py

index 6a76a251fbd06e0abd5533db21fe963acfe1014b..c2aaf48ec79529f3ca5accd47db1156fa5e9d57e 100644 (file)
@@ -255,7 +255,7 @@ class Module(MgrModule):
                     next_scrape = now
                 else:
                     # align to scrape interval
-                    scrape_frequency = int(self.scrape_frequency) or 86400
+                    scrape_frequency = self.scrape_frequency or 86400
                     seconds = (last_scrape - datetime.utcfromtimestamp(0)).total_seconds()
                     seconds -= seconds % scrape_frequency
                     seconds += scrape_frequency
@@ -274,7 +274,7 @@ class Module(MgrModule):
                     self.set_store('last_scrape', last_scrape.strftime(TIME_FORMAT))
 
             # sleep
-            sleep_interval = int(self.sleep_interval) or 60
+            sleep_interval = self.sleep_interval or 60
             self.log.debug('Sleeping for %d seconds', sleep_interval)
             ret = self.event.wait(sleep_interval)
             self.event.clear()
@@ -384,7 +384,7 @@ class Module(MgrModule):
     def put_device_metrics(self, ioctx, devid, data):
         assert devid
         old_key = datetime.utcnow() - timedelta(
-            seconds=int(self.retention_period))
+            seconds=self.retention_period)
         prune = old_key.strftime(TIME_FORMAT)
         self.log.debug('put_device_metrics device %s prune %s' %
                        (devid, prune))
@@ -460,8 +460,8 @@ class Module(MgrModule):
         self.log.info('Check health')
         config = self.get('config')
         min_in_ratio = float(config.get('mon_osd_min_in_ratio'))
-        mark_out_threshold_td = timedelta(seconds=int(self.mark_out_threshold))
-        warn_threshold_td = timedelta(seconds=int(self.warn_threshold))
+        mark_out_threshold_td = timedelta(seconds=self.mark_out_threshold)
+        warn_threshold_td = timedelta(seconds=self.warn_threshold)
         checks = {}
         health_warnings = {
             DEVICE_HEALTH: [],