curtin-dev team mailing list archive

[Ryan Harper <ryan.harper@xxxxxxxxxxxxx>]

Diff comments:

> diff --git a/curtin/commands/block_meta.py b/curtin/commands/block_meta.py
> index 7fbd89f..5849e3d 100644
> --- a/curtin/commands/block_meta.py
> +++ b/curtin/commands/block_meta.py
> @@ -910,8 +910,12 @@ def partition_handler(info, storage_config):
>          if multipath.is_mpath_device(disk):
>              udevadm_settle()  # allow partition creation to happen
>              # update device mapper table mapping to mpathX-partN
> -            util.subp(['kpartx', '-v', '-a', '-s', '-p', '-part', disk])
>              part_path = disk + "-part%s" % partnumber
> +            # sometimes multipath lib creates a block device instead of

I can reference the bug in the commit message here.  It has to do with
the number of paths in the multipath and the load of the system as I
read it.  The RHBZ mentions that there's some tooling changes to disable
the behavior (ie, always let udev do it).

> +            # a udev symlink, remove this and allow kpartx to create it
> +            if os.path.exists(part_path) and not os.path.islink(part_path):
> +                util.del_file(part_path)
> +            util.subp(['kpartx', '-v', '-a', '-s', '-p', '-part', disk])

Let me check if we verify it later in this method.

>          else:
>              part_path = block.dev_path(block.partition_kname(disk_kname,
>                                                               partnumber))


-- 
https://code.launchpad.net/~raharper/curtin/+git/curtin/+merge/382776
Your team curtin developers is subscribed to branch curtin:master.