Bug 1342558

Summary:	No information about data_lv and metadata_lv in a cache pool DBus object
Product:	[Community] LVM and device-mapper	Reporter:	Vratislav Podzimek <vpodzime>
Component:	lvm2	Assignee:	Tony Asleson <tasleson>
lvm2 sub component:	lvmdbusd	QA Contact:	cluster-qe <cluster-qe>
Status:	CLOSED ERRATA	Docs Contact:
Severity:	low
Priority:	low	CC:	agk, anprice, bmarzins, bmr, dwysocha, heinzm, jbrassow, jonathan, lvm-team, msnitzer, prajnoha, prockai, tasleson, zkabelac
Version:	2.02.154	Keywords:	FutureFeature
Target Milestone:	Fedora	Flags:	agk: lvm-technical-solution+ rule-engine: lvm-test-coverage?
Target Release:	2016-06
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:	2.02.156	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-09-13 15:53:27 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:
Embargoed:

Description Vratislav Podzimek 2016-06-03 14:02:53 UTC

Description of problem:
When I have a cache pool LV, its DBus representation doesn't provide information about the data_lv and metadata_lv objects. In case of a thin pool these both are provided so it should be the same for cache pools.

Version-Release number of selected component (if applicable):
lvm2-2.02.154-1

How reproducible:
100%

Steps to Reproduce:
1. create a cache pool LV
2. try to get it's data_lv and metadata_lv over DBus

Actual results:
information missing

Expected results:
information provided

Additional info:
These two pieces of information should probably go under a new 'CachePool' interface similar to 'ThinPool'.

Comment 1 Tony Asleson 2016-06-10 21:11:14 UTC

Committed upstream:
https://git.fedorahosted.org/cgit/lvm2.git/commit/?id=e04705b305626d37cbd076b47d6ad37c9b1b282e

Comment 2 Vratislav Podzimek 2016-06-13 07:21:32 UTC

(In reply to Tony Asleson from comment #1)
> Committed upstream:
> https://git.fedorahosted.org/cgit/lvm2.git/commit/
> ?id=e04705b305626d37cbd076b47d6ad37c9b1b282e

Looks good to me, thanks!

Comment 3 Alasdair Kergon 2016-06-13 11:12:44 UTC

Included in 2.02.156.

https://www.redhat.com/archives/lvm-devel/2016-June/msg00029.html

commit e04705b305626d37cbd076b47d6ad37c9b1b282e
Author: Tony Asleson <tasleson>
Date:   Mon Jun 6 15:03:02 2016 -0500

    lvmdbusd: Add data_lv and metadata_lv to cache pool interface