← Back to team overview

curtin-dev team mailing list archive

Re: [Merge] ~mwhudson/curtin:lp-1878041 into curtin:master

 

Yeah, the two issues came up in the same report but I should probably fix them in separate branches. Will untangle presently.

Diff comments:

> diff --git a/curtin/commands/block_meta.py b/curtin/commands/block_meta.py
> index e29c1e4..5557e7d 100644
> --- a/curtin/commands/block_meta.py
> +++ b/curtin/commands/block_meta.py
> @@ -692,6 +692,10 @@ def calc_partition_info(disk, partition, logical_block_size_bytes):
>      if partition.startswith('dm-'):
>          pp = partition
>          pp_start_sec, pp_size_sec = calc_dm_partition_info(partition)
> +        if pp_start_sec is not None:
> +            pp_start_sec //= logical_block_size_bytes // 512
> +        if pp_start_sec is not None:
> +            pp_size_sec //= logical_block_size_bytes // 512

Oh yes, I see the other branch of the if has the same adjustment. Too much late night coding. I should add a test for this bit too.

>      else:
>          pp = os.path.join(disk, partition)
>          # XXX: sys/block/X/{size,start} is *ALWAYS* in 512b value


-- 
https://code.launchpad.net/~mwhudson/curtin/+git/curtin/+merge/396538
Your team curtin developers is requested to review the proposed merge of ~mwhudson/curtin:lp-1878041 into curtin:master.


References