curtin-dev team mailing list archive
-
curtin-dev team
-
Mailing list archive
-
Message #01354
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