Bug 1211183

+++ This bug was initially created as a clone of Bug #1187481 +++

Description of problem:

lvm2app has regressed with regard to the "lv_attr" property: lvm_lv_get_property does no longer deliver a value for it.

This means that storaged treats all logical volumes as inactive and doesn't recognize thin pools and snapshots anymore, among other things. 

Switching from lvm_lv_get_property to lvm_lv_get_attr is not enough.  The latter returns incomplete attributes where 'active' and 'open' are set to "unknown".

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

lvm2-2.02.115-1.fc21.x86_64

How reproducible:

Always.

Steps to Reproduce:
1. Create a thin pool with Cockpit

Actual results:

Thin pool is created but shown as a inactive plain logical volume.

Expected results:

This pool is shown as a active thin pool

Additional info:

https://www.redhat.com/archives/lvm-devel/2015-January/msg00075.html

--- Additional comment from Marius Vollmer on 2015-01-30 02:45:55 EST ---

Looks like lvm2app is unmaintained and storaged should be ported away from it.  Would it make sense to remove lvm2app from Fedora (and maybe the world)?

--- Additional comment from Peter Rajnoha on 2015-01-30 04:03:32 EST ---

Currently, there are several changes in the reporting area of LVM2 code. One of those changes is that we're trying to reuse the same info and status ioctl result for one reporting line, removing a need to call several ioctl per one line of report output which can save some time and resources.

Unfortunately, one of the recent changes, where we tried to convert the lv_attr (and hence the lvm_lv_get_attr lvm2app counterpart), we introduced a regression that got unnoticed and it caused some of these "info" and "status" attributes in lv_attr (and the string returned by lvm_lv_get_attr) to have "unknown" (the "X") value.

Sorry for that! It should be fixed by this patch now:

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

I'll do a new update with this fix in for f21.

--- Additional comment from Marius Vollmer on 2015-01-30 04:11:07 EST ---

> It should be fixed by this patch now:

Great, thanks!

From reading the patch, I think this doesn't fully fix the regression, unfortunately.

lvm_lv_get_property can't find the "lv_attr" attribute at all anymore, because it's type was changed from LVS to LVSSTATUSINFO, as far as I can tell.

We can easily update storaged to use lvm_lv_get_attr, though.

What do you propose?

--- Additional comment from Marius Vollmer on 2015-01-30 04:14:03 EST ---

Correction: snapshots are recognized.

--- Additional comment from Peter Rajnoha on 2015-01-30 04:34:04 EST ---

Just looking at the lvm_lv_get_property. You can also use lvm_lv_get_attr - internally, it's the same code which gets the value - in case lvm_lv_get_attr it's direct call to acquire the attr string. The lvm_lv_get_property is the general one for all fields which has some extra wrappers to handle this in general way, but underneath, for the lv_attr, it returns exactly the same as lvm_lv_get_attr.

--- Additional comment from Marius Vollmer on 2015-01-30 05:04:15 EST ---

> You can also use lvm_lv_get_attr - internally, it's the same code which gets the value 

Yes, but we need to change storaged for that, and rush the new version into Fedora.  That's fine, but if you are going to fix the lvm_lv_get_property regression as well, we don't need to even do that.

--- Additional comment from Peter Rajnoha on 2015-01-30 05:08:58 EST ---

Yes, if you wait a bit, I think I can fix that. Hopefully today.

--- Additional comment from Peter Rajnoha on 2015-01-30 05:22:54 EST ---

So this one should fix the issue with lvm_lv_get_property:

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

I've tested that on my machine, but I'll do a scratch for you to test too, just for sure.

--- Additional comment from Peter Rajnoha on 2015-01-30 05:36:12 EST ---

F21 scratch build here:
http://koji.fedoraproject.org/koji/taskinfo?taskID=8781186

--- Additional comment from Marius Vollmer on 2015-01-30 05:44:50 EST ---

> F21 scratch build here:

Awesome!  Testing...

--- Additional comment from Marius Vollmer on 2015-01-30 05:55:14 EST ---

(In reply to Marius Vollmer from comment #10)
> > F21 scratch build here:
> 
> Awesome!  Testing...

Works as expected.  Lvm_lv_get_property returns good values for "lv_attr"  and storaged recognizes volumes as thin pools again.

Thanks!

--- Additional comment from Peter Rajnoha on 2015-01-30 06:35:50 EST ---

OK, there's going to be a new upstream release today anyway, so I'll do rawhide and F21 pkgs then.

--- Additional comment from Fedora Update System on 2015-01-30 12:42:54 EST ---

lvm2-2.02.116-3.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/lvm2-2.02.116-3.fc21

--- Additional comment from Fedora Update System on 2015-01-31 19:26:24 EST ---

Package lvm2-2.02.116-3.fc21:
* should fix your issue,
* was pushed to the Fedora 21 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing lvm2-2.02.116-3.fc21'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2015-1504/lvm2-2.02.116-3.fc21
then log in and leave karma (feedback).

--- Additional comment from Fedora Update System on 2015-02-02 12:23:39 EST ---

lvm2-2.02.116-3.fc21 has been pushed to the Fedora 21 stable repository.  If problems still persist, please make note of it in this bug report.

--- Additional comment from Marius Vollmer on 2015-02-03 03:02:46 EST ---

Thanks, excellent service, will shop again!