openstack/cinder
Code Issues Proposed changes

Use oslo.privsep for 'lvcreate'

Browse Source 
Use olso.privsep rather than oslo.rootwrap for the 'lvcreate'

Change-Id: I9e8f74ef74ef3f7cbbcf7967304246e9dfafb5c0
Signed-off-by: Charles Short <chucks@redhat.com>
This commit is contained in:
Charles Short 2019-01-17 13:52:02 -05:00
parent 3902a2bf28
commit 303741a4b3
3 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
cinder/brick/local_dev/lvm.py
View File

@ -147,8 +147,7 @@ class LVM(executor.Executor):
return exists
def _create_vg(self, pv_list):
cmd = ['vgcreate', self.vg_name, ','.join(pv_list)]
self._execute(*cmd, root_helper=self._root_helper, run_as_root=True)
cinder.privsep.lvm.create_volume(self.vg_name, pv_list)
def _get_thin_pool_free_space(self, vg_name, thin_pool_name):
"""Returns available thin pool free space.

6
cinder/privsep/lvm.py
View File

@ -30,3 +30,9 @@ def udevadm_settle():
def lvrename(vg_name, lv_name, new_name):
processutils.execute(
'lvrename', vg_name, lv_name, new_name)
@cinder.privsep.sys_admin_pctxt.entrypoint
def create_vg(vg_name, pv_list):
cmd = ['vgcreate', vg_name, ','.join(pv_list)]
processutils.execute(*cmd)

3
etc/cinder/rootwrap.d/volume.filters
View File

@ -46,9 +46,6 @@ privsep-rootwrap-sys_admin: RegExpFilter, privsep-helper, root, privsep-helper,
# is updated appropriately.
drbdadm: CommandFilter, drbdadm, root
# cinder/brick/local_dev/lvm.py: 'vgcreate', vg_name, pv_list
vgcreate: CommandFilter, vgcreate, root
# cinder/brick/local_dev/lvm.py: 'lvcreate', '-L', sizestr, '-n', volume_name,..
# cinder/brick/local_dev/lvm.py: 'lvcreate', '-L', ...
lvcreate: EnvFilter, env, root, LC_ALL=C, lvcreate