Skip to content

Commit

Permalink
Merge "Specify the conf file when creating a volume."
Browse files Browse the repository at this point in the history
  • Loading branch information
Jenkins authored and openstack-gerrit committed Sep 10, 2012
2 parents c3476b5 + 0a09bf5 commit 28a5b31
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion nova/tests/test_iscsi.py
Original file line number Diff line number Diff line change
Expand Up @@ -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'])


Expand Down
2 changes: 2 additions & 0 deletions nova/volume/iscsi.py
Original file line number Diff line number Diff line change
Expand Up @@ -127,6 +127,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)
Expand Down

0 comments on commit 28a5b31

Please sign in to comment.