curtin-dev team mailing list archive

[Michael Hudson-Doyle <mwhudsonlp@xxxxxxxxxxx>]

Michael Hudson-Doyle has proposed merging ~mwhudson/curtin:lp-1923487 into curtin:master.

Commit message:
block_meta: pass --yes to lvcreate alongside --wipesignatures=y

Otherwise it will not actually wipe the signatures it finds.

LP: #1923487


Requested reviews:
  curtin developers (curtin-dev)
Related bugs:
  Bug #1923487 in curtin: "multipath / FCP installation fails on particular LPAR"
  https://bugs.launchpad.net/curtin/+bug/1923487

For more details, see:
https://code.launchpad.net/~mwhudson/curtin/+git/curtin/+merge/401009
-- 
Your team curtin developers is requested to review the proposed merge of ~mwhudson/curtin:lp-1923487 into curtin:master.

diff --git a/curtin/commands/block_meta.py b/curtin/commands/block_meta.py
index cf6bc02..29f3969 100644
--- a/curtin/commands/block_meta.py
+++ b/curtin/commands/block_meta.py
@@ -1324,7 +1324,7 @@ def lvm_partition_handler(info, storage_config):
         cmd = ["lvcreate", volgroup, "--name", name, "--zero=y"]
         release = distro.lsb_release()['codename']
         if release not in ['precise', 'trusty']:
-            cmd.extend(["--wipesignatures=y"])
+            cmd.extend(["--wipesignatures=y", "--yes"])
 
         if info.get('size'):
             size = util.human2bytes(info["size"])