curtin-dev team mailing list archive

[Jean-Noël AVILA <jn.avila@xxxxxxx>]

Jean-Noël AVILA has proposed merging ~jnavila/curtin:lvm_int_bytes into curtin:master.

Requested reviews:
  curtin developers (curtin-dev)

For more details, see:
https://code.launchpad.net/~jnavila/curtin/+git/curtin/+merge/396875

When using automatic install script in subiquity with automated size
computation, the output is a double with no decimal, but lvcreate
fails with values of bytes presenting a decimal point.
-- 
Your team curtin developers is requested to review the proposed merge of ~jnavila/curtin:lvm_int_bytes into curtin:master.

diff --git a/curtin/commands/block_meta.py b/curtin/commands/block_meta.py
index 25910a0..05f3bd8 100644
--- a/curtin/commands/block_meta.py
+++ b/curtin/commands/block_meta.py
@@ -1326,7 +1326,7 @@ def lvm_partition_handler(info, storage_config):
 
         if info.get('size'):
             size = util.human2bytes(info["size"])
-            cmd.extend(["--size", "{}B".format(size)])
+            cmd.extend(["--size", "{}B".format(int(size))])
         else:
             cmd.extend(["--extents", "100%FREE"])