[PATCH] Allow user-specified values for data alignment of new lvm pvs. (#1178705)
David Lehman
dlehman at redhat.com
Tue Jan 20 19:18:29 UTC 2015
On 01/20/2015 09:03 AM, Anne Mulhern wrote:
>
>
>
>
> ----- Original Message -----
>> From: "Brian C. Lane" <bcl at redhat.com>
>> To: anaconda-patches at lists.fedorahosted.org
>> Sent: Friday, January 16, 2015 7:50:50 PM
>> Subject: Re: [PATCH] Allow user-specified values for data alignment of new lvm pvs. (#1178705)
>>
>> On Fri, Jan 16, 2015 at 12:09:43PM -0600, David Lehman wrote:
>>> ---
>>> blivet/devicelibs/lvm.py | 7 +++++--
>>> blivet/formats/lvmpv.py | 11 +++++++----
>>> tests/devicelibs_test/lvm_test.py | 29 +++++++++++++++++++++++++++++
>>> 3 files changed, 41 insertions(+), 6 deletions(-)
>>>
>>> diff --git a/blivet/devicelibs/lvm.py b/blivet/devicelibs/lvm.py
>>> index efca4c8..269b9c3 100644
>>> --- a/blivet/devicelibs/lvm.py
>>> +++ b/blivet/devicelibs/lvm.py
>>> @@ -233,8 +233,11 @@ def lvm(args, capture=False, ignore_errors=False):
>>> if capture:
>>> return strip_lvm_warnings(out)
>>>
>>> -def pvcreate(device):
>>> - args = ["pvcreate", device]
>>> +def pvcreate(device, data_alignment=None):
>>> + args = ["pvcreate"]
>>> + if data_alignment is not None:
>>> + args.extend(["--dataalignment", "%dk" %
>>> data_alignment.convertTo(KiB)])
>>> + args.append(device)
>>
>> This could use a docstring so we know it is in KiB.
>>
>> Ack other than that.
>>
>> --
>> Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA
>> (PST8PDT)
>> _______________________________________________
>> anaconda-patches mailing list
>> anaconda-patches at lists.fedorahosted.org
>> https://lists.fedorahosted.org/mailman/listinfo/anaconda-patches
>>
>
> I can't match this patch against any current likely blivet branch.
>
> Maybe there's also another patch set in flight?
There is a very old one that I apparently forgot to push:
https://lists.fedorahosted.org/pipermail/anaconda-patches/2014-June/011443.html
It's queued in my clone now to go right before this proposed patch.
David
>
> - mulhern
> _______________________________________________
> anaconda-patches mailing list
> anaconda-patches at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/anaconda-patches
>
More information about the anaconda-patches
mailing list