Merge remote-tracking branch 'ldoktor/bitshift'

Signed-off-by: Cleber Rosa <[email protected]>
avocado-framework · Jun 26, 2019 · 17fbd21 · 17fbd21
2 parents 689c385 + 8f4c027
commit 17fbd21
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/virttest/storage.py b/virttest/storage.py
@@ -476,12 +476,10 @@ def is_disk_size_enough(required, available):
         """Check if available disk size is enough for the data copy."""
         minimum_disk_free = 1.2 * required
         if available < minimum_disk_free:
-            # bitshift trick to do a quick convert (MB) for the log
-            bitshift = 20
-            logging.error("Free space: %s MB", (available >> bitshift))
-            logging.error("Backup size: %s MB", (required >> bitshift))
+            logging.error("Free space: %s MB", (available / 1048576.))
+            logging.error("Backup size: %s MB", (required / 1048576.))
             logging.error("Minimum free space acceptable: %s MB",
-                          (minimum_disk_free >> bitshift))
+                          (minimum_disk_free / 1048576.))
             logging.error("Available disk space is not enough. Skipping...")
             return False
         return True