(#177): backup with compression enabled fails: unsupported operand ty…

…pe(s)
abbbi · Apr 24, 2024 · 45c2677 · 45c2677
1 parent e6e48c9
commit 45c2677
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/Changelog b/Changelog
@@ -1,3 +1,9 @@
+Version 2.9
+---------
+ * Fix: backup with compression enabled fails: unsupported operand type(s) for
+ +: 'int' and 'dict' (#177)
+
+
 Version 2.8
 ---------
  * Add packages compatible to fedora 39 to package build (#174)

diff --git a/libvirtnbdbackup/backup/disk.py b/libvirtnbdbackup/backup/disk.py
@@ -176,6 +176,7 @@ def backup(  # pylint: disable=too-many-arguments,too-many-branches, too-many-lo
                 writer.write(sTypes.TERM)
                 if args.compress:
                     logging.debug("Compressed size: %s", size)
+                    backupSize += size
                     if cSizes:
                         blockList = {}
                         blockList[size] = cSizes
@@ -184,7 +185,7 @@ def backup(  # pylint: disable=too-many-arguments,too-many-branches, too-many-lo
                         compressedSizes.append(size)
                 else:
                     assert size == save.length
-            backupSize += save.length
+                    backupSize += save.length
         else:
             if streamType == "raw":
                 writer.seek(save.offset)
@@ -215,7 +216,4 @@ def backup(  # pylint: disable=too-many-arguments,too-many-branches, too-many-lo
         partialfile.rename(targetFilePartial, targetFile)
         backupChecksum(fileStream, targetFile)
 
-    if args.compress:
-        backupSize = sum(blen for blen in compressedSizes)
-
     return backupSize, True