broken code in brick/local_dev/lvm

Bug #1866854 reported by Eric Harney
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
New
Undecided
Unassigned

Bug Description

_create_vg() calls cinder.privsep.lvm.create_volume() which does not actually exist

https://opendev.org/openstack/cinder/src/commit/c49442889c14da8df1cf8d55cc83f73a21af4695/cinder/brick/local_dev/lvm.py#L150

There is also possible confusion here about creating lvs vs. vgs.

Revision history for this message
Sean McGinnis (sean-mcginnis) wrote :

Looks like the actual call hadn't changed.

https://opendev.org/openstack/cinder/commit/303741a4b3f8df40e99fd5daf427e104788b0069

I'm a little confused how that passed, and has been passing in gate tests for the last year. But obvious in that diff that the call should have been create_vg, not create_volume.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.