Specify the conf file when creating a volume.

Updates the tgt-admin command to actually use the config file we create for the volume being created. This fixes an issue introduced in 9785963 which breaks iscsi volume creation. Fixes LP Bug #1046985. Change-Id: I06e83c9ba351c2b4846b557a3abb170a51d506c0
openstack · Sep 6, 2012 · 0903603 · 0903603
1 parent 5d72e7a
commit 0903603
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/cinder/tests/test_iscsi.py b/cinder/tests/test_iscsi.py
@@ -92,7 +92,7 @@ def setUp(self):
         self.flags(iscsi_helper='tgtadm')
         self.flags(volumes_dir="./")
         self.script_template = "\n".join([
-        'tgt-admin --update iqn.2011-09.org.foo.bar:blaa',
+        'tgt-admin --conf ./blaa --update iqn.2011-09.org.foo.bar:blaa',
         'tgt-admin --delete iqn.2010-10.org.openstack:volume-blaa'])
 
 

diff --git a/cinder/volume/iscsi.py b/cinder/volume/iscsi.py
@@ -128,6 +128,8 @@ def create_iscsi_target(self, name, tid, lun, path, **kwargs):
 
         try:
             (out, err) = self._execute('tgt-admin',
+                                       '--conf',
+                                       volume_path,
                                        '--update',
                                        name,
                                        run_as_root=True)