]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Add os-brick's scsi_id command to Cinder rootwrap
authorSean McGinnis <sean_mcginnis@dell.com>
Sat, 5 Sep 2015 18:04:05 +0000 (13:04 -0500)
committerSean McGinnis <sean_mcginnis@dell.com>
Sun, 6 Sep 2015 02:15:58 +0000 (21:15 -0500)
FC drivers are failing on calls to scsi_id. This command
is in the os-brick rootwrap filters, but that file is not
being pulled in to Cinder yet.

As a temporary fix until that work can be completed, this
patch adds the missing command directly to volume.filters
to allow this command to run. Follow up work will be needed
to remove this and other legacy brick commands from the
volume.filters file and instead pull in os-brick's
os-brick.filters values.

Change-Id: Iba8e5c82f8d203759cda0e21de5fdf2404b18e69
Partial-bug: #1479842

etc/cinder/rootwrap.d/volume.filters

index 552cabc97bedaae7d6d2e562079683d062bf7424..6737e283cb7481d315297b50ccacc9ab78827419 100644 (file)
@@ -22,6 +22,12 @@ vgs_lvmconf: EnvFilter, env, root, LVM_SYSTEM_DIR=, LC_ALL=C, vgs
 lvs_lvmconf: EnvFilter, env, root, LVM_SYSTEM_DIR=, LC_ALL=C, lvs
 lvdisplay_lvmconf: EnvFilter, env, root, LVM_SYSTEM_DIR=, LC_ALL=C, lvdisplay
 
+# os-brick library commands
+# TODO(smcginnis) This is a temporary fix. Need to pull in os-brick
+# os-brick.filters file instead and clean out stale brick values from
+# this file.
+scsi_id: CommandFilter, /lib/udev/scsi_id, root
+
 # cinder/volumes/drivers/srb.py: 'pvresize', '--setphysicalvolumesize', sizestr, pvname
 pvresize: CommandFilter, pvresize, root