]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Rename cinder-rtstool to rtstool
authorEric Harney <eharney@redhat.com>
Tue, 19 Mar 2013 20:40:19 +0000 (16:40 -0400)
committerEric Harney <eharney@redhat.com>
Tue, 19 Mar 2013 21:29:38 +0000 (17:29 -0400)
Since rtstool is being distributed externally, it will be named
rtstool rather than cinder-rtstool.

Change-Id: Ieff072ab8315928ed02837aff63c08601b1d2acd

cinder/tests/test_iscsi.py
cinder/volume/iscsi.py
etc/cinder/rootwrap.d/volume.filters

index 4bd942852352c51192fff8f1153a9413964d57d5..7b1a327d47bcee4197c3e090d3a6739bb95c81f0 100644 (file)
@@ -134,6 +134,6 @@ class LioAdmTestCase(test.TestCase, TargetAdminTestCase):
         self.persist_tempdir = tempfile.mkdtemp()
         self.flags(iscsi_helper='lioadm')
         self.script_template = "\n".join([
-            'cinder-rtstool create '
+            'rtstool create '
                 '/foo iqn.2011-09.org.foo.bar:blaa test_id test_pass',
-            'cinder-rtstool delete iqn.2010-10.org.openstack:volume-blaa'])
+            'rtstool delete iqn.2010-10.org.openstack:volume-blaa'])
index ae664b767b4399adec7ce672c8d03c81b06d47b9..77efed4701e73b406e3513a55ab05aa190e0a47f 100644 (file)
@@ -327,16 +327,16 @@ class FakeIscsiHelper(object):
 class LioAdm(TargetAdmin):
     """iSCSI target administration for LIO using python-rtslib."""
     def __init__(self, execute=utils.execute):
-        super(LioAdm, self).__init__('cinder-rtstool', execute)
+        super(LioAdm, self).__init__('rtstool', execute)
 
         try:
-            self._execute('cinder-rtstool', 'verify')
+            self._execute('rtstool', 'verify')
         except (OSError, exception.ProcessExecutionError):
-            LOG.error(_('cinder-rtstool is not installed correctly'))
+            LOG.error(_('rtstool is not installed correctly'))
             raise
 
     def _get_target(self, iqn):
-        (out, err) = self._execute('cinder-rtstool',
+        (out, err) = self._execute('rtstool',
                                    'get-targets',
                                    run_as_root=True)
         lines = out.split('\n')
@@ -354,7 +354,7 @@ class LioAdm(TargetAdmin):
 
         LOG.info(_('Creating iscsi_target for volume: %s') % vol_id)
 
-        # cinder-rtstool requires chap_auth, but unit tests don't provide it
+        # rtstool requires chap_auth, but unit tests don't provide it
         chap_auth_userid = 'test_id'
         chap_auth_password = 'test_pass'
 
@@ -366,7 +366,7 @@ class LioAdm(TargetAdmin):
             extra_args.append(FLAGS.lio_initiator_iqns)
 
         try:
-            command_args = ['cinder-rtstool',
+            command_args = ['rtstool',
                             'create',
                             path,
                             name,
@@ -397,7 +397,7 @@ class LioAdm(TargetAdmin):
         iqn = '%s%s' % (FLAGS.iscsi_target_prefix, vol_uuid_name)
 
         try:
-            self._execute('cinder-rtstool',
+            self._execute('rtstool',
                           'delete',
                           iqn,
                           run_as_root=True)
@@ -424,7 +424,7 @@ class LioAdm(TargetAdmin):
 
         # Add initiator iqns to target ACL
         try:
-            self._execute('cinder-rtstool', 'add-initiator',
+            self._execute('rtstool', 'add-initiator',
                           volume_iqn,
                           auth_user,
                           auth_pass,
index 9103db8ccd844edc39930f9fd9957dd9ab451411..fdd63a8ef5f9bbbfdc6108c317f470932048050a 100644 (file)
@@ -6,7 +6,7 @@
 ietadm: CommandFilter, ietadm, root
 tgtadm: CommandFilter, tgtadm, root
 tgt-admin: CommandFilter, tgt-admin, root
-cinder-rtstool: CommandFilter, cinder-rtstool, root
+rtstool: CommandFilter, rtstool, root
 
 # cinder/volume/driver.py: 'vgs', '--noheadings', '-o', 'name'
 vgs: CommandFilter, vgs, root