diff options
author | kshithijiyer <kshithij.ki@gmail.com> | 2020-08-14 12:44:48 +0530 |
---|---|---|
committer | Arthy Loganathan <aloganat@redhat.com> | 2020-08-17 05:58:44 +0000 |
commit | 6694d72c786b754d76bc18854174accfb2c23efa (patch) | |
tree | a70513f9d3e5bea1d429cb95045fdb016537ca4b /tests/functional/dht | |
parent | 30c45e121dbb2d7da289328e8a76111691a44f46 (diff) |
[Libfix] Fix python3 getfattr() issues
Problem:
Due to patch [1] which was sent for issue #24
causes a large number of testcases to fail
or get stuck in the latest DHT run.
Solution:
Make changes sot that getfattr command
sends back the output in text wherever needed.
Links:
[1] https://review.gluster.org/#/c/glusto-tests/+/24841/
Change-Id: I6390e38130b0699ceae652dee8c3b2db2ef3f379
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
Diffstat (limited to 'tests/functional/dht')
-rw-r--r-- | tests/functional/dht/test_dht_custom_xattr.py | 17 | ||||
-rw-r--r-- | tests/functional/dht/test_directory_custom_extended_attributes.py | 21 |
2 files changed, 22 insertions, 16 deletions
diff --git a/tests/functional/dht/test_dht_custom_xattr.py b/tests/functional/dht/test_dht_custom_xattr.py index a0a17958b..fa2ad8cdb 100644 --- a/tests/functional/dht/test_dht_custom_xattr.py +++ b/tests/functional/dht/test_dht_custom_xattr.py @@ -44,7 +44,8 @@ class TestDhtCustomXattrClass(GlusterBaseClass): for mount_object in self.mounts: for fname in self.files_and_soft_links: attr_val = get_fattr(mount_object.client_system, - fname, 'user.foo') + fname, 'user.foo', + encode='text') self.assertEqual(attr_val, xattr_val, "Custom xattr not found from mount.") g.log.info("Custom xattr found on mount point.") @@ -59,7 +60,7 @@ class TestDhtCustomXattrClass(GlusterBaseClass): for fname in files: attr_val = get_fattr(node, "{}/{}".format(brick_path, fname), - 'user.foo') + 'user.foo', encode='text') self.assertEqual(attr_val, xattr_val, "Custom xattr not visible on bricks") g.log.info("Custom xattr found on bricks.") @@ -104,7 +105,7 @@ class TestDhtCustomXattrClass(GlusterBaseClass): for mount_object in self.mounts: for fname in list_of_all_files: ret = get_fattr(mount_object.client_system, - fname, 'user.foo') + fname, 'user.foo', encode='text') self.assertIsNone(ret, "Custom attribute visible at mount " "point even after deletion") @@ -118,7 +119,7 @@ class TestDhtCustomXattrClass(GlusterBaseClass): if fname.split('/')[3] in files_on_bricks] for fname in files: ret = get_fattr(node, "{}/{}".format(brick_path, fname), - 'user.foo') + 'user.foo', encode='text') self.assertIsNone(ret, "Custom attribute visible on " "brick even after deletion") @@ -216,13 +217,13 @@ class TestDhtCustomXattrClass(GlusterBaseClass): self.set_xattr_user_foo(self.list_of_files, 'bar2') # Check if custom xattr is set to all the regular files - self.check_custom_xattr_visible('bar2') + self.check_custom_xattr_visible("bar2") # Change the custom xattr on all the regular files self.set_xattr_user_foo(self.list_of_files, 'ABC') # Check if xattr is set to all the regular files - self.check_custom_xattr_visible('ABC') + self.check_custom_xattr_visible("ABC") # Delete Custom xattr from all regular files self.delete_xattr_user_foo(self.list_of_files) @@ -238,13 +239,13 @@ class TestDhtCustomXattrClass(GlusterBaseClass): self.set_xattr_user_foo(list_of_softlinks, 'bar2') # Check if custom xattr is set to all the regular files - self.check_custom_xattr_visible('bar2') + self.check_custom_xattr_visible("bar2") # Change the custom xattr on all the regular files self.set_xattr_user_foo(list_of_softlinks, 'ABC') # Check if xattr is set to all the regular files - self.check_custom_xattr_visible('ABC') + self.check_custom_xattr_visible("ABC") # Delete Custom xattr from all regular files self.delete_xattr_user_foo(list_of_softlinks) diff --git a/tests/functional/dht/test_directory_custom_extended_attributes.py b/tests/functional/dht/test_directory_custom_extended_attributes.py index b391593b1..fd1493622 100644 --- a/tests/functional/dht/test_directory_custom_extended_attributes.py +++ b/tests/functional/dht/test_directory_custom_extended_attributes.py @@ -117,7 +117,8 @@ class TestDirectoryCustomExtendedAttributes(GlusterBaseClass): mount_point, folder_name) ret = get_fattr(mount_point.client_system, mount_point.mountpoint, - 'trusted.glusterfs.pathinfo') + 'trusted.glusterfs.pathinfo', + encode="text") self.assertIsNotNone(ret, "trusted.glusterfs.pathinfo is not " "presented on %s:%s" % @@ -139,7 +140,7 @@ class TestDirectoryCustomExtendedAttributes(GlusterBaseClass): g.log.debug('Check xarttr user.foo on %s:%s', mount_point.client_system, folder_name) ret = get_fattr(mount_point.client_system, folder_name, - 'user.foo') + 'user.foo', encode="text") self.assertEqual(ret, 'bar2', "Xattr attribute user.foo is not presented on " "mount point %s and directory %s" % @@ -153,7 +154,8 @@ class TestDirectoryCustomExtendedAttributes(GlusterBaseClass): brick_path = dir_prefix.format(root=brick_dir, client_index=mount_index) - ret = get_fattr(brick_server, brick_path, 'user.foo') + ret = get_fattr(brick_server, brick_path, 'user.foo', + encode="text") g.log.debug('Check custom xattr for directory on brick %s:%s', brick_server, brick_path) @@ -177,7 +179,8 @@ class TestDirectoryCustomExtendedAttributes(GlusterBaseClass): g.log.debug('Looking if custom extra attribute user.foo is ' 'presented on mount or on bricks after deletion') self.assertIsNone(get_fattr(mount_point.client_system, - folder_name, 'user.foo'), + folder_name, 'user.foo', + encode="text"), "Xattr user.foo is presented on mount point" " %s:%s after deletion" % (mount_point.mountpoint, folder_name)) @@ -277,7 +280,7 @@ class TestDirectoryCustomExtendedAttributes(GlusterBaseClass): g.log.debug('Check mountpoint and bricks for custom xattribute') self.assertEqual('bar2', get_fattr(mount_point.client_system, linked_folder_name, - 'user.foo'), + 'user.foo', encode="text"), 'Custom xattribute is not presented on ' 'mount point %s:%s' % (mount_point.client_system, linked_folder_name)) @@ -297,7 +300,8 @@ class TestDirectoryCustomExtendedAttributes(GlusterBaseClass): continue self.assertEqual(get_fattr(brick_server, brick_path, - 'user.foo'), 'bar2', + 'user.foo', encode="text"), + 'bar2', "Actual: custom attribute not " "found on brick %s:%s" % ( brick_server, brick_path)) @@ -319,7 +323,8 @@ class TestDirectoryCustomExtendedAttributes(GlusterBaseClass): "after deletion", mount_point.client_system, linked_folder_name) self.assertIsNone(get_fattr(mount_point.client_system, - linked_folder_name, 'user.foo'), + linked_folder_name, 'user.foo', + encode="text"), "Expected: xattr user.foo to be not presented on" " %s:%s" % (mount_point.client_system, linked_folder_name)) @@ -339,7 +344,7 @@ class TestDirectoryCustomExtendedAttributes(GlusterBaseClass): continue self.assertIsNone(get_fattr(brick_server, brick_path, - 'user.foo'), + 'user.foo', encode="text"), "Extended custom attribute is presented on " "%s:%s after deletion" % (brick_server, brick_path)) |