]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Remove RecursivePermDirFixture useless cleanup
authorCedric Brandily <zzelle@gmail.com>
Fri, 24 Apr 2015 19:20:40 +0000 (21:20 +0200)
committerCedric Brandily <zzelle@gmail.com>
Fri, 24 Apr 2015 19:20:40 +0000 (21:20 +0200)
This change removes a useless cleanup in RecursivePermDirFixture:
previously RecursivePermDirFixture reverts permission changes on
directories, but the cleanup is useless as directories are provided
by TempDir.

Change-Id: I76c8dbefe3b42ec34a50fb164b9cbc25f4ac4245

neutron/tests/functional/agent/linux/helpers.py

index 611a423045df9812e77d36422574b53d3437ff22..6e003b515a09b385e531651adc1df4c032739b35 100644 (file)
@@ -50,16 +50,9 @@ class RecursivePermDirFixture(fixtures.Fixture):
             perms = os.stat(current_directory).st_mode
             if perms & self.least_perms != self.least_perms:
                 os.chmod(current_directory, perms | self.least_perms)
-                self.addCleanup(self.safe_chmod, current_directory, perms)
             previous_directory = current_directory
             current_directory = os.path.dirname(current_directory)
 
-    def safe_chmod(self, path, mode):
-        try:
-            os.chmod(path, mode)
-        except OSError:
-            pass
-
 
 def get_free_namespace_port(tcp=True, namespace=None):
     """Return an unused port from given namespace