test_brick_lvm unit test issue

Bug #1676646 reported by Eric Harney on 2017-03-27
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Undecided
Unassigned

Bug Description

BrickLvmTestCase.test_volume_create_after_thin_creation has a "def executor" method defined, but it doesn't seem to work. You can add anything to it, and unit tests will still pass.

Also, assigning directly to self.vg._executor is likely not safe.

Found while looking into this pylint error, which is still another issue:
["No value passed for parameter 'root_helper' in function call", "LOG.error('Current state: %s', self.get_all_volume_groups())"]

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

Other bug subscribers