| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Problem:
Testcases test_mount_snap_delete and test_restore_online_vol
were failing in the latest runs with the below traceback
```
Traceback (most recent call last):
File "/usr/share/glustolibs/io/scripts/file_dir_ops.py", line 1246, in <module>
rc = args.func(args)
File "/usr/share/glustolibs/io/scripts/file_dir_ops.py", line 374, in create_files
base_file_name, file_types)
File "/usr/share/glustolibs/io/scripts/file_dir_ops.py", line 341, in _create_files
ret = pool.map(lambda file_tuple: _create_file(*file_tuple), files)
File "/usr/lib64/python3.6/multiprocessing/pool.py", line 266, in map
return self._map_async(func, iterable, mapstar, chunksize).get()
File "/usr/lib64/python3.6/multiprocessing/pool.py", line 644, in get
raise self._value
File "/usr/lib64/python3.6/multiprocessing/pool.py", line 119, in worker
result = (True, func(*args, **kwds))
File "/usr/lib64/python3.6/multiprocessing/pool.py", line 44, in mapstar
return list(map(*args))
File "/usr/share/glustolibs/io/scripts/file_dir_ops.py", line 341, in <lambda>
ret = pool.map(lambda file_tuple: _create_file(*file_tuple), files)
File "/usr/share/glustolibs/io/scripts/file_dir_ops.py", line 270, in _create_file
with open(file_abs_path, "w+") as new_file:
FileExistsError: [Errno 17] File exists: '/mnt/testvol_distributed_glusterfs/file1.txt'
```
This was because I/O logic was trying to create 2 files with the same
name from 2 clients.
Fix:
Modify logic to use counters to create files with different names.
Change-Id: I2896736d28f6bd17435f941088fd634347e3f4fd
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Test Cases in this module tests the
USS functionality of snapshots snapd
on cloned volume and validated snapshots
are present inside .snaps directory by
terminating snapd on one by one nodes
and validating .snaps directory is still accessible.
Change-Id: I98d48268e7c5c5952a7f0f544960203d8634b7ac
Signed-off-by: Sri Vignesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For the non-tiered volume types, In few test cases while bringing
bricks offline, collecting both hot_tier_bricks and cold_tier_bricks
and it is not needed to collect hot and cold tier bricks.
Removing the hot and cold tiered bricks and collecting only bricks
of the particular volume as mentioned below.
Removing below section
```
bricks_to_bring_offline_dict = (select_bricks_to_bring_offline(
self.mnode, self.volname))
bricks_to_bring_offline = list(filter(None, (
bricks_to_bring_offline_dict['hot_tier_bricks'] +
bricks_to_bring_offline_dict['cold_tier_bricks'] +
bricks_to_bring_offline_dict['volume_bricks'])))
```
Modifying as below for bringing bricks offline.
```
bricks_to_bring_offline = bricks_to_bring_offline_dict['volume_bricks']
```
Change-Id: Icb1dc4a79cf311b686d839f2c9390371e42142f7
Signed-off-by: Bala Konda Reddy M <bala12352@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Problem:
test_snap_delete_multiple fails on I/O validation across all automation
runs constantly with the below trace back:
Traceback (most recent call last):
File "/usr/share/glustolibs/io/scripts/file_dir_ops.py", line 1246, in <module>
rc = args.func(args)
File "/usr/share/glustolibs/io/scripts/file_dir_ops.py", line 374, in create_files
base_file_name, file_types)
File "/usr/share/glustolibs/io/scripts/file_dir_ops.py", line 341, in _create_files
ret = pool.map(lambda file_tuple: _create_file(*file_tuple), files)
File "/usr/lib64/python2.7/multiprocessing/pool.py", line 250, in map
return self.map_async(func, iterable, chunksize).get()
File "/usr/lib64/python2.7/multiprocessing/pool.py", line 554, in get
raise self._value
IOError: [Errno 17] File exists: '/mnt/testvol_distributed_glusterfs/testfile42.txt'
Fix:
Change the I/O to use --base-file-name parameter when running the I/O
scripts.
Change-Id: Ic5a8222f4fafeac4ac9aadc9c4d23327711ed9f0
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When a snapshot is restored, that snapshot gets removed.
USS makes use of a '.snaps' directory in the mount point
where all the activated snapshots can be listed.
So the restored snapshot should not be listed under the
'.snaps' directory regardless of it being activated or
deactivated.
Steps:
* Perform I/O on mounts
* Enable USS on volume
* Validate USS is enabled
* Create a snapshot
* Activate the snapshot
* Perform some more I/O
* Create another snapshot
* Activate the second
* Restore volume to the second snapshot
* From mount point validate under .snaps
- first snapshot should be listed
- second snapshot should not be listed
Change-Id: I5630d8aad6b4758d49e8d4f53497073c78a00a6b
Co-authored-by: Sunny Kumar <sunkumar@redhat.com>
Signed-off-by: Sunny Kumar <sunkumar@redhat.com>
Signed-off-by: Vinayak Papnoi <vpapnoi@redhat.com>
|
|
|
|
|
|
|
| |
Move cases from teardown class to teardown in snapshot
Change-Id: I7b33fa2728665fad000a5ad881f6690d40913f22
Signed-off-by: Sri Vignesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add sleep in test_snap_delete_original_volume.py
after cloning a volume added sleep and then starting
the volume. Changed io to write in one mount point
otherwise seeing issues with validate io.
removed baseclass cleanup because original volume is
already cleaned up in testcase.
Change-Id: I7bf9686384e238e1afe8491013a3058865343eee
Signed-off-by: Sri Vignesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Please refer to commit message of patch [1].
[1] https://review.gluster.org/#/c/glusto-tests/+/24140/
Change-Id: I5319ce497ca3359e0e7dbd9ece481bada1ee2205
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
|
|
|
|
| |
used library functions to wait for glusterd to start
and wait for peer to connect and made modifications in teardown part
to rectified statements to correct values
Change-Id: I40b4362ae1491acf75681c7623c16c53213bb1b9
Signed-off-by: Sri Vignesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There can be a case where due to a failed tearDown of a previous
test case, the snapshot creation might fail because of redundant
snapshot of the same name present.
This fix includes the changed snapshot names along with tearDown
for deleting the snapshots created in the test case
"test_snap_info_glusterd_restart.py".
Changing the names of the test case function to something
relevant to the test case.
Making use of the method "wait_for_glusterd_to_start" in places
where glusterd is restarted.
Change-Id: I6cb50dc84d306194b6bd363daf7ae0ebd6bb12ee
Signed-off-by: Vinayak Papnoi <vpapnoi@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There can be a case where due to a failed tearDown of a previous
test case, the snapshot creation might fail. This can happen when
a redundant snapshot with the same name is present.
This fix includes the changed snapshot names along with tearDown
for deleting the snapshots created in the test case
"test_snap_list_after_restart.py".
This fix also includes the method 'wait_for_glusterd_to_start'
where there is a restart of glusterd.
Adding a check for validation of snapshots using snapname.
Change-Id: If8b48a12bd067ad54dba742eeb88444beaf5f153
Signed-off-by: Vinayak Papnoi <vpapnoi@redhat.com>
|
|
|
|
|
|
|
|
| |
Added library to cleanup lv created after snapshot clone and made
modifications with cleanup.
Change-Id: I71a437bf99eac1170895510475ddb30748771672
Signed-off-by: Sri Vignesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The test case 'test_256_snapshots.py' contains hard-coded commands
which can be run using the existing library functions from the
snap_ops.py library, such as:
* snap_create
* set_snap_config
This patch includes usage of appropriate functions in place of the
hard-coded commands along with various cosmetic changes.
Change-Id: I6da7444d903efd1be6582c8fea037a5c4fddc111
Signed-off-by: Vinayak Papnoi <vpapnoi@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
'ExecutionError' must not be used inside the test case function.
Replacing 'ExecutionError' raised in test case function with
assert statements.
Changed the name of the snapshots being created to something
specific to the test case to avoid redundancy with the other
test cases.
Segregating the setUp and tearDown functions.
Adding the description for the test case function.
Changing the test case name for something specific to the
component (snapshot).
Fixing the typo errors and various cosmetic changes.
Change-Id: I24f271cdb7a180ecde8efa21185b4a8f807be203
Signed-off-by: Vinayak Papnoi <vpapnoi@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Checking the status of snapshot informations
before and after restarting glusterd.
Test case:
1. Create volume
2. Create two snapshots with description
3. Check snapshot status informations with snapname,
volume name and without snap name/volname.
4. Restart glusterd on all nodes
5. Follow step3 again and validate snapshot
History of the patch:
The testcase was failing in CentOS-ci due to bug [1], However this
bug was defered for fix. Post updating the code with the workaround
mentioned below, it was observed that glusterd restart was failing
due to bug [2]. And now as the both the bugs are fixed this patch
is safe to merge.
Workaround:
Now the only possible workaround for this is to
modify the function get_snap_info_by_volname()
to not use --xml option and directly run the command which
will dump the output as string which can be directly
used in the testcase. This modification in the library function
will not impact any other testcase.
Links:
[1] https://bugzilla.redhat.com/show_bug.cgi?id=1590797
[2] https://bugzilla.redhat.com/show_bug.cgi?id=1782200
Change-Id: I26ac0aaa5f6c8849fd9de41f506d6d13fc55e166
Co-authored-by: srivickynesh <sselvan@redhat.com>
Signed-off-by: srivickynesh <sselvan@redhat.com>
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
|
|
|
| |
Please refer to the commit message of the below patch:
https://review.gluster.org/#/c/glusto-tests/+/23902/
Change-Id: I1df0324dac2da5aad4064cc72ef77dcb5bf67e4f
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Removing script_local_path as both script_local_path and
cls.script_upload_path hold the same values which makes
each script slow. This will help decrease the execution
time of the test suite.
PoC:
$cat test.py
a = ("/usr/share/glustolibs/io/scripts/"
"file_dir_ops.py")
b = ("/usr/share/glustolibs/io/scripts/"
"file_dir_ops.py")
$time python test.py
real 0m0.063s
user 0m0.039s
sys 0m0.019s
$cat test.py
a = ("/usr/share/glustolibs/io/scripts/"
"file_dir_ops.py")
$time python test.py
real 0m0.013s
user 0m0.009s
sys 0m0.003s
Code changes needed:
From:
script_local_path = ("/usr/share/glustolibs/io/scripts/"
"file_dir_ops.py")
cls.script_upload_path = ("/usr/share/glustolibs/io/scripts/"
"file_dir_ops.py")
ret = upload_scripts(cls.clients, script_local_path)
To:
cls.script_upload_path = ("/usr/share/glustolibs/io/scripts/"
"file_dir_ops.py")
ret = upload_scripts(cls.clients, cls.script_upload_path)
Change-Id: I7908b3b418bbc929b7cc3ff81e3675310eecdbeb
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
| |
Change-Id: I2e85670e50e3dab8727295c34aa6ec4f1326c19d
Signed-off-by: Valerii Ponomarov <kiparis.kh@gmail.com>
|
|
|
|
|
| |
Change-Id: Ie408d7972452123b63eb5cc17c61bc319a99e304
Signed-off-by: Valerii Ponomarov <kiparis.kh@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
The tearDown function was incomplete where, since the test case has
enabled the shared storage, it needs to be disabled as part of the
tear down.
This fix includes the addition of the necessary tearDown parts along
with some cosmetic changes.
Change-Id: Id421c840a1c7606ecf185c9520ca436d47911f45
Signed-off-by: Vinayak Papnoi <vpapnoi@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
The test case makes use of 'is_shared_volume_unmounted' function
which is redundant as the purpose can be solved via a very similar
function 'is_shared_volume_mounted'. This change has been implemented.
There are various cosmetic changes that have been implemented as well.
Change-Id: I560b464b4bcc436658db49c0a5ed8c7aadacfb6a
Signed-off-by: Vinayak Papnoi <vpapnoi@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Removing 'ExecutionError' raised inside the test case function.
Adding the description for the test case function.
Fixing the typo errors and various cosmetic changes.
Change-Id: Ica505417f482c7a622be5da70c1053913cf3052c
Signed-off-by: Vinayak Papnoi <vpapnoi@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Removing 'ExecutionError' raised inside the test case function.
Adding the description for the test case function.
Fixing the typo errors and various cosmetic changes.
Change-Id: I496dc1df067870d43e5c3c97fc27da4294862eba
Signed-off-by: Vinayak Papnoi <vpapnoi@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Delete multiple snapshots of the original and cloned volumes
simultaneously.
Change-Id: Ic5165b896d2adf4cb51ad4252b7caad7bb6f0892
Signed-off-by: srivickynesh <sselvan@redhat.com>
Signed-off-by: Sri <sselvan@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Test Cases in this module tests the
Creation of clone from snapshot of one volume.
add-brick to cloned volume and perform rebalance
operation on cloned volume
Change-Id: I34de5c53bbf68b1c80962295e4e4d0535da78990
Signed-off-by: srivickynesh <sselvan@redhat.com>
Signed-off-by: Sri Vignesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Test Cases in this module tests the
snapshot scheduler command such as initialise
scheduler, enable scheduler, status of scheduler.
Change-Id: Id18e83b6a6bc37820f7b2d23f86e7d68bbddef0e
Signed-off-by: srivickynesh <sselvan@redhat.com>
Signed-off-by: Sri Vignesh <sselvan@redhat.com>
|
|
|
|
| |
Change-Id: I46fc2feffe6443af6913785d67bf310838532421
|
|
|
|
|
| |
Change-Id: Ia177213ca46af917dc83e1482c6061d425e30545
Signed-off-by: Sri Vignesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
| |
The purpose of this test is to validate snapshot
hard and soft max-limt options.
Change-Id: I000b840421620094287ba393367cad7c8df8e23a
Signed-off-by: srivickynesh <sselvan@redhat.com>
|
|
|
|
|
|
|
| |
The purpose of this test is to validate snapshot create
Change-Id: Icacafc48739e6322fb925e117df63462d690251f
Signed-off-by: srivickynesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
| |
This testcase contains tests which verifies the snapshot max-hardlimit
and soft-limit and deletion of snapshots for a volume
Change-Id: Iff00475abdaf01b2beac72545774f477e99a13ce
Signed-off-by: srivickynesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
| |
No functional change, just make the tests a bit more readable.
It could be moved to a decorator later on, wrapping tests.
Change-Id: I484bb8b46907ee8f33dfcf4c960737a21819cd6a
Signed-off-by: Yaniv Kaul <ykaul@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Test Cases in this module tests the
snapshot activation and deactivation status
when glusterd is down.
Change-Id: I9ae97ceba40ce2511b1b731d5e02990f78f424e9
Signed-off-by: srivickynesh <sselvan@redhat.com>
Signed-off-by: Vignesh <vignesh@localhost.localdomain>
|
|
|
|
|
|
|
|
| |
This test case will validate snap restore on online volume.
When we try to restore online volume it should fail
Change-Id: If542a6052ee6e9eec7d5e1268f480f047916e66c
Signed-off-by: Sunny Kumar <sunkumar@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Test Cases in this module tests the USS
where .snaps folder is already created and
checks for behaviour before
enabling uss and after disabling uss
Change-Id: I9c6dcca1198b7fed37103cc21f3ecba72bfa20a5
Signed-off-by: srivickynesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Test Cases in this module tests the
snapshot creation and listing Invalid names
and parameters.
Change-Id: Iab3dd71a83176d850780c0834f95d4e8ac1e8997
Signed-off-by: srivickynesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
| |
Create a clone and check for self heal operation on
the cloned volume.
Change-Id: Icf61f996fcd503a6c0d0bf936900858b715a4742
Signed-off-by: srivickynesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
|
| |
This test cases validates snapshot delete behaviour
when glusterd is down on one node. When we bring up the brought
down node, post handshake number of snaps should be consistent.
Change-Id: If7ed6c3f384658faa4eb3de8577e38d3fe55f980
Signed-off-by: Sunny Kumar <sunkumar@redhat.com>
|
|
|
|
|
|
|
|
| |
Test Cases in this module tests the snapshot scheduler
for deleting existing scheduled jobs
Change-Id: Ibd1a00fb336f279d8c13a89ae513a914977f593d
Signed-off-by: srivickynesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Test Cases in this module tests the
Creation of clone from snapshot of volume
and delete snapshot and original volume.
Validate cloned volume is not affected.
Change-Id: Ic61defe96fed6adeab21b3715878ac8093156645
Signed-off-by: srivickynesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Test Cases in this module tests the
snapshot deletion with snapname, with volumename
and delete all snapshot commands.
Change-Id: I1e4e361e58b35744e08e63c48b43d9e8caf2e953
Signed-off-by: srivickynesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Test Cases in this module tests the
snapshot listing before and after
glusterd restart.
Change-Id: I7cabe284c52974256e1a48807a5fc1787789583a
Signed-off-by: srivickynesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
| |
Test Cases in this module tests the
uss functionality while io is going on.
Change-Id: Ie7bf440b02980a0606bf4c4061f5a1628179c128
Signed-off-by: srivickynesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
| |
This test cases will validate snapshot scheduler behaviour
when we enable/disable scheduler.
Change-Id: Ia6f01a9853aaceb05155bfc92cccba686d320e43
Signed-off-by: Sunny Kumar <sunkumar@redhat.com>
|
|
|
|
|
|
|
|
| |
This test case will validate USS behaviour when we
enable USS on the volume when brick is down.
Change-Id: I9be021135c1f038a0c6949ce2484b47cd8634c1e
Signed-off-by: Sunny Kumar <sunkumar@redhat.com>
|
|
|
|
|
|
|
|
| |
Activated snaps should get listed in .snap directory while deactivated
snap should not.
Change-Id: I04a61c49dcbc9510d60cc8ee6b1364742271bbf0
Signed-off-by: Sunny Kumar <sunkumar@redhat.com>
|
|
|
|
|
|
|
|
| |
Create 10 clones of snapshot and verify gluster volume list
information
Change-Id: Ibd813680d1890e239deaf415469f7f4dccfa6867
Signed-off-by: srivickynesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
| |
Test Cases in this module tests the snapshot information after
glusterd is restarted.
Change-Id: I7c5e761d8a8cd261841d064dbd94093e1c5b6edd
Signed-off-by: srivickynesh <sselvan@redhat.com>
|
|
|
|
|
|
|
|
| |
Information of snapshots taken for a specified snapshot/volume
or all snapshots present in the system
Change-Id: Ibe355053848b234e0892c0b7f68bfed053f8867a
Signed-off-by: srivickynesh <sselvan@redhat.com>
|
|
|
|
|
| |
Change-Id: I70dc1930dabebc8e0af4c83c8b70842f92f9865a
Signed-off-by: srivickynesh <sselvan@redhat.com>
|