Hacking: Fix E226

Fix:
E226 missing whitespace around arithmetic operator

Change-Id: I9d0a9a93c635695b4962a88c9bc68dfc4385abe7

manila/share/drivers/veritas/veritas_isa.py

@@ -353,10 +353,11 @@ class ACCESSShareDriver(driver.ExecuteMixin, driver.ShareDriver):
         path = self._nfs_add_str
         provider = '%s:%s' % (self.host, self._port)
         data = {}
-        va_share_info = ("{\"share\":[{\"fileSystemPath\":\""+va_sharepath +
+        va_share_info = ("{\"share\":[{\"fileSystemPath\":\"" + va_sharepath +
                          "\",\"shareType\":\"NFS\",\"shareDetails\":" +
-                         "[{\"client\":\""+server+"\",\"exportOptions\":\"" +
-                         access_level+"\"}]}]}")
+                         "[{\"client\":\"" + server +
+                         "\",\"exportOptions\":\"" +
+                         access_level + "\"}]}]}")
 
         data["shareDetails"] = va_share_info
 
@@ -396,9 +397,9 @@ class ACCESSShareDriver(driver.ExecuteMixin, driver.ShareDriver):
         path = self._nfs_delete_str
         provider = '%s:%s' % (self.host, self._port)
         data = {}
-        va_share_info = ("{\"share\":[{\"fileSystemPath\":\""+va_sharepath +
+        va_share_info = ("{\"share\":[{\"fileSystemPath\":\"" + va_sharepath +
                          "\",\"shareType\":\"NFS\",\"shareDetails\":" +
-                         "[{\"client\":\""+server+"\"}]}]}")
+                         "[{\"client\":\"" + server + "\"}]}]}")
 
         data["shareDetails"] = va_share_info
         result = self._access_api(self.session, provider, path,

manila/tests/share/drivers/hpe/test_hpe_3par_mediator.py

@@ -1121,7 +1121,7 @@ class HPE3ParMediatorTestCase(test.TestCase):
         fstore_init_size = int(
             constants.GET_FSQUOTA['members'][0]['hardBlock'])
 
-        expected_capacity = (0-share_size) * units.Ki + fstore_init_size
+        expected_capacity = (0 - share_size) * units.Ki + fstore_init_size
         self.mock_object(self.mediator,
                          '_find_fstore',
                          mock.Mock(return_value=constants.EXPECTED_FSTORE))
@@ -1667,7 +1667,7 @@ class HPE3ParMediatorTestCase(test.TestCase):
                 constants.NFS_LOWER,
                 constants.EXPECTED_VFS,
                 constants.EXPECTED_SHARE_NAME,
-                clientip='+'+constants.EXPECTED_IP_1234,
+                clientip='+' + constants.EXPECTED_IP_1234,
                 fpg=constants.EXPECTED_FPG,
                 fstore=constants.EXPECTED_FSTORE,
                 comment=constants.EXPECTED_COMMENT),

manila/tests/share_group/test_api.py

@@ -319,7 +319,8 @@ class ShareGroupsAPITestCase(test.TestCase):
             db_driver, 'share_group_type_get',
             mock.Mock(return_value=share_group_type))
         self.mock_object(share_types, 'get_share_type',
-                         mock.Mock(side_effect=[share_type_1, share_type_1]*2))
+                         mock.Mock(side_effect=[share_type_1,
+                                                share_type_1] * 2))
         self.mock_object(db_driver, 'share_group_snapshot_get')
         kwargs = {
             'availability_zone': 'hypernova',

tox.ini

@@ -138,8 +138,7 @@ commands = alembic -c manila/db/migrations/alembic.ini revision -m ""{posargs}
 # E305 expected 2 blank lines after class or function definition, found 1
 # E731 do not assign a lambda expression, use a def
 # E117 over-indented
-# E226 missing whitespace around arithmetic operator
-ignore = E117,E123,E226,E305,E402,E731,E741,W503,W504,W605
+ignore = E117,E123,E305,E402,E731,E741,W503,W504,W605
 builtins = _
 # [H106] Don't put vim configuration in source files.
 # [H203] Use assertIs(Not)None to check for None.