Bug 1381304

Summary: RFE: provide all properties of LVs/VGs/PVs that the liblvm library's objects provide
Product: [Community] LVM and device-mapper Reporter: Vratislav Podzimek <vpodzime>
Component: lvm2Assignee: Tony Asleson <tasleson>
lvm2 sub component: lvmdbusd QA Contact: cluster-qe <cluster-qe>
Status: CLOSED ERRATA Docs Contact:
Severity: unspecified    
Priority: unspecified CC: agk, dlehman, heinzm, jbrassow, msnitzer, prajnoha, tasleson, zkabelac
Version: 2.02.166Flags: rule-engine: lvm-technical-solution?
rule-engine: lvm-test-coverage?
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 2.02.167 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-09-13 15:40:59 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Vratislav Podzimek 2016-10-03 15:57:39 UTC 
Description of problem:
LVM DBus API doesn't provide all properties for LVs/VGs/PVs that the liblvm API provides (e.g. "data_percent" for an LV). This prevents users of the liblvm API (e.g. storaged) from moving to the new DBus API (via libblockdev-lvm-dbus).
Comment 1 Vratislav Podzimek 2016-10-05 09:07:00 UTC 
If liblvm provides all properties, it will be enough (for now) to add properties required by storaged. [1]

[1] https://github.com/storaged-project/storaged/blob/master/modules/lvm2/udiskslvmhelper.c#L122 (and the other show_*() functions)
Comment 2 Tony Asleson 2016-10-05 20:58:52 UTC 
Changes committed upstream:

https://git.fedorahosted.org/cgit/lvm2.git/commit/?id=a3f24aaf5cfe9983a59ce200d0a0d8050658c8b8

https://git.fedorahosted.org/cgit/lvm2.git/commit/?id=d54ffcfcd12981a8e046aca0a82d529221015b30

https://git.fedorahosted.org/cgit/lvm2.git/commit/?id=a8bb8dfb08d38cfda5493ca9d35d1cb3bf2d32bf
Comment 3 Vratislav Podzimek 2016-10-06 08:21:14 UTC 
(In reply to Tony Asleson from comment #2)
> Changes committed upstream:
> 
> https://git.fedorahosted.org/cgit/lvm2.git/commit/
> ?id=a3f24aaf5cfe9983a59ce200d0a0d8050658c8b8
> 
> https://git.fedorahosted.org/cgit/lvm2.git/commit/
> ?id=d54ffcfcd12981a8e046aca0a82d529221015b30
> 
> https://git.fedorahosted.org/cgit/lvm2.git/commit/
> ?id=a8bb8dfb08d38cfda5493ca9d35d1cb3bf2d32bf

Looks good to me. Thanks!