| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
|
|
|
| |
Adding support for arbiter and distributed-arbiter volume types
as it was not present earlier in setup_volume().
Change-Id: I836e4865bb2066478d5f08254f37f115b3a1aac5
Signed-off-by: sayaleeraut <saraut@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Following changes were implemented:
- Delete unused imports and place used ones in the alphabetical order.
Imports are splitted into 3 groups: built-ins, third-parties and
local modules/libs.
- Make changes to support py3 in addition to py2.
- Minimize number of code lines keeping the same behaviour and improving
readability.
- Add possibility to get 'bound' (cls) methods using 'get_super_method'
staticmethod from base class. Before it was possible to call only
unbound (self) methods.
- Update 'test_add_brick.py' module as PoC for running base class bound
methods in both - py2 and py3. Now this module py2/3 compatible.
Change-Id: I1b66b3a91084b2487c26bec8763ab2b4e12ac482
Signed-off-by: Valerii Ponomarov <kiparis.kh@gmail.com>
|
|
|
|
|
|
|
|
|
| |
For following reasons:
- It is unused.
- It is not compatible with py3.
Signed-off-by: Valerii Ponomarov <kiparis.kh@gmail.com>
Change-Id: Ied6373c398406ffd08d7673892fde89ec6e04fe2
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Lots of test classes are wrapped by the 'runs_on' decorator.
This decorator replaces original test class with it's copy where
parent class is original test class if we use py3.
Such situation leads to the impossibility to use following approach in
py3:
super(SomeClass, some_class_instance).some_method()
And, the above approach is py2/3 compatible approach for calling
parent class's methods.
The problem we face is that we fall into the unexpected recursion here.
So, add 'get_super_method' to the base class, which detects such
situation and returns proper method of a proper parent class.
Also, fix test class located at 'glusterd/test_peer_status.py' module
to have proof of concept.
With this change 'test_peer_probe_status' test case becomes completely
py2/3 compatible.
Example of new method usage:
@runs_on([['distributed'], ['glusterfs']])
class TestDecoratedClass(GlusterBaseClass):
...
def setUp(self):
self.get_super_method(self, 'setUp')()
...
This approach must be used instead of existing calls of 'im_func'
function if we want to support both at once - python2 and python3.
Signed-off-by: Valerii Ponomarov <kiparis.kh@gmail.com>
Change-Id: I23f4462b64f9d4dd90812273f08fb756d073ab76
|
|
|
|
|
|
|
|
| |
Dict attribute called "iteritems()" is not supported in the py3.
So, replace it's usage with another similar attr called "items()".
Change-Id: I130b7f67f0a2d5da5ed6c3d792f5ff024ba148f4
Signed-off-by: Valerii Ponomarov <kiparis.kh@gmail.com>
|
|
|
|
|
|
|
|
|
| |
Since egrep treats a pattern as extended regular expression
it is more suited to be used in this case. The current usage
of grep fails to identify the expression after the first pipe.
Change-Id: Id02f884356048c56066a406937f8d86456ee8bd7
Signed-off-by: Arjun Sharma <arjsharm@redhat.com>
|
|
|
|
|
|
|
| |
Change-Id: I5c9c6ce48a973c6c902186f4e62afda196faf442
Signed-off-by: vivek das <vdas@redhat.com>
Signed-off-by: srivickynesh <sselvan@redhat.com>
Signed-off-by: vivek das <vdas@redhat.com>
|
|
|
|
|
|
|
|
| |
NfsGaneshaVolumeBaseClass and NfsGaneshaIOBaseClass are
essentially not needed and act as redundant.
Change-Id: I45f95e018daa4bf9575f4e831111f91615085bdc
Signed-off-by: Arjun Sharma <arjsharm@redhat.com>
|
|
|
|
|
|
|
| |
The method 'refresh_config' can be used instead of 'run_refresh_config'
Change-Id: I6c70b4dff1fd4531fdf6eaa4f044d2d1992d8151
Signed-off-by: Jilju Joy <jijoy@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
At present there are 2 different functions for prerequisites
and create session merging them and adding checks to make the
functions more effective when called from a conftest.py file.
By doing so we also avoid the possibility of running the same
function twice which might lead to problems.
For example when commands like:
$ gluster system:: execute gsec_create
it would make 2 entries in authorized keys files leading
to workers getting into faulty state.
Note:
This patch should only be merged after
merging the below mentioned patches:
[1] https://review.gluster.org/#/c/glusto-tests/+/23291/
[2] https://review.gluster.org/#/c/glusto-tests/+/23293/
Change-Id: I3f8cda7b589c86de4dab8c66d033b364dd495b9b
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
THIS PATCH IS DEPENDENT ON THE BELOW PATCH:
https://review.gluster.org/#/c/glusto-tests/+/23291/
Removing the below functions as a more generic function
is added in lib_utils:
georep_ssh_keygen
georep_ssh_copyid
georep_groupadd
georep_geoaccount
georep_geoaccount_setpasswd
Adding georep_mountbroker_remove_user to
remove users from mountbroker which
will be needed in teardown.
Change-Id: I5b484b93cbc1660cdf223cfa4bac2a32acc8fc77
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adding the following functions tot lib_utils.py:
1.ssh_keygen(): Creates a pair of ssh private
and public key if not present.
2.set_passwd(): Sets password for a given username.
3.ssh_copy_id(): Copies the default ssh public key.
4.group_add(): Creates a group in all the servers.
Modifying add_user() to create and add user to a
specific group. Also modifying the code in add_user()
to take list of servers.
Change-Id: I58cdef967293204073aa912fa98adb9606f6885d
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
|
| |
Change-Id: I0f32dc73d960f8bfeb662643e736291ffb74cb81
Signed-off-by: Roch-elle <rallan@redhat.com>
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The line 316 of heal_libs.py library contains a function called
'wait_for_self_heal_daemons_to_be_online' which makes use of the
'volname' and 'timeout' arguements for string formatting.
The method used for the string formatting was wrong which causes
an error 'TypeError: not enough arguments for format string'.
Due to this, all the test cases which make use of heal_libs.py
will result in a TypeError scenario.
This patch will fix the TypeError for the string formatting.
Change-Id: I73abe573d2ccb60b74ccd6ae268b950640d75600
Signed-off-by: Vinayak Papnoi <vpapnoi@redhat.com>
|
|
|
|
|
|
|
|
|
| |
The function 'is_brick_mux_enabled' has an incorrect check for
the ValueError. The format specifier was incorrect and missing.
It has been added with this fix.
Change-Id: Idac536de8b9916207ebdda1ba6c981837f01a220
Signed-off-by: Vinayak Papnoi <vpapnoi@redhat.com>
|
|
|
|
|
|
|
|
| |
Adding function check_gluster_shared_volume() to
check if gluster_shared_storage is present or absent.
Change-Id: Ifa646187a0ef351f3dd37b54bb0a3a8a7c8b2500
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently there is a library to reset the volume entirely.
There needs to be a library to reset a particular volume option
because if a user has set a lot of other volume options then it
becomes a tedious job to reset and set the other options explicitly.
The library function 'reset_volume_option' takes particular
volume options as a string and resets it.
Change-Id: I2049ea4a49d321d921d940f34e124d1f34bfa068
Signed-off-by: Vinayak Papnoi <vpapnoi@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Adding get_brick_processes_count() which
can be used to get the number of brick processes
running on a given node would be needed for
brickmux testcases to check if brickmux is
honored or not.
Change-Id: I6ec608b8f9d715d0f21e95fc3c0293eed287b16c
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
| |
The function 'create_shared_storage' is redundant as the same function is
available as part of the shared-storage library.
This part will be removed from the geo-rep library with this fix.
To create/enable shared storage, make use of the shared_storage_ops.py library.
Change-Id: Ibefca73140e1218f0aa9331f061ef4fc8d9360e3
Signed-off-by: Vinayak Papnoi <vpapnoi@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Added more checking mechanism for crashes,
as sometimes coredumps may not get collected by systems
due to settings and hence can miss out on any crashes hit
Change-Id: Id45a844bd0d7877d335c06287d4db873776e89db
Signed-off-by: nchilaka <nchilaka@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adding library for the glusterfind operations such as:
* glusterfind create
* glusterfind delete
* glusterfind pre
* glusterfind post
* glusterfind query
* glusterfind list
Change-Id: I94372f8c4a14cc94d68451dec1763401d9d66d90
Signed-off-by: Vinayak Papnoi <vpapnoi@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
The function do the following in a nutshell:
1. find_and_replace_in_file: Find and replace
a given pattern in a specific file.
2. check_if_pattern_in_file: Check if a give
pattern is in seen in file or not.
Change-Id: Ib035f54490592f2be32a30f0e8a10e7847812990
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
|
| |
Change-Id: Ib557ce211f1616ef261bf414a2c0b3864a5be24e
Signed-off-by: Ambarish Soman <asoman@redhat.com>
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
| |
Change-Id: Ib8a832901430da30f5f49686ba78b8fb2b76b8e7
Signed-off-by: Jilju Joy <jijoy@redhat.com>
|
|
|
|
|
|
|
|
| |
* Code change to do peer probe if not already done
* Changes to avoid prerequeste of password less ssh between nodes
Change-Id: I8260a80ad8c75a6b3d569cc7ee652e569ec9153a
Signed-off-by: Jilju Joy <jijoy@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
* Removed dependency on gdeploy for setup and teardown of ganesha cluster
* Correction of pylint errors
* Added new method to set acl
* Added new method to set root-squash
* Added new method for refresh-config
* Removed setUp and tearDown methods from NfsGaneshaClusterSetupClass
Change-Id: I1266fe7c09e1fed148ca222712e15932a5ad928c
|
|
|
|
|
|
|
| |
Added new config file for georep
Change-Id: I544ee35f8cd6a2f2ed744f72caa28fa4fd63e5b3
Signed-off-by: Roch-elle <rallan@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
get_online_bricks_list() used to fail in case of node
down secnarios with KeyError exception adding code to
catch the exception and provide brick list with bricks
from gluster v status.
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
Change-Id: Ia26208a52e4197050421bc34b9b8cdaf74ac4da6
|
|
|
|
|
| |
Change-Id: I7a043753b8ae2ae46ee944e04ddd9137f177960f
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
| |
Change-Id: I6db42f6762fe323e8ea0af6b7a9ecbe9af9f5b59
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
|
|
|
|
|
| |
Change-Id: I2acf835a4cf7301c64c4c8a9423f78672cdf9aa4
Signed-off-by: Vitalii Koriakov <vkoriako@redhat.com>
|
|
|
|
| |
Change-Id: I1525d783c223dd3bb3099419bcd9a5efcf0f0085
|
|
|
|
| |
Change-Id: I4c0b5c9e39d1e7903a64ad9fe91c2feb6ecf7f8e
|
|
|
|
|
| |
Change-Id: I953975d66c333c141b03b4eaa7118a93859be878
Signed-off-by: Vitalii Koriakov <vkoriako@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Either nfs-ganesha or gnfs can be enabled. Both cannot co-exist.
If 'nfs_ganesha' enable option is set to True in the configuration file,
volume will be exported through nfs-ganesha,
else it will be exported through gnfs
Change-Id: I26e5aa8b29d54fc5d41d77fea7951df66e4de1e7
|
|
|
|
|
| |
Change-Id: I2183252ee881b5f9c8c74af62c7d8fd2cafb8fc9
Signed-off-by: Vitalii Koriakov <vkoriako@redhat.com>
|
|
|
|
|
| |
Change-Id: I8baf97066c5ae47e2a298581eeb74374fc09a21e
Signed-off-by: Vitalii Koriakov <vkoriako@redhat.com>
|
|
|
|
|
| |
Change-Id: I2f491f7cadd92846730c6528920c9fa8c2c9edce
Signed-off-by: Vitalii Koriakov <vkoriako@redhat.com>
|
|
|
|
|
| |
Change-Id: Ie134790ea05919a4b396657c11cbc3fc7a7fc529
Signed-off-by: Vitalii Koriakov <vkoriako@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add the base class for gluster-block which includes functions that
helps in automating the test cases on client and target side.
These functions will help in block creation, block discovery,
login on the client, getting mpath, mounting the block, logout.
The teardown function will take care of unmounting
blocks, logging out and block deletion and volume cleanup.
The .yml file for block should be like: Example
gluster_block_args_info:
volname: testvol #Should be same as volume name in glusterfs config fle
servers: #Server names should be same as names in glusterfs config file
- 192.168.100.163
- 192.168.100.192
size: 2GiB
ha: 3
auth: enable
prealloc:
num_of_blocks: 2 #Number of blocks to be created with this configuration
ring-buffer: 16
blockname: block_testing #Base name of the blocks to be created.
Change-Id: I8cd354fc991cd21c542cfd7f03399cc23c9f7917
Signed-off-by: Bhumika Goyal <bgoyal@redhat.com>
|
|
|
|
|
|
| |
This reverts commit 2d425798657a725e02d63ff631c5b87187ea4ca4.
Change-Id: I388fe7ff11e28e027fe9006512e35ec57ff10d9c
|
|
|
|
| |
Change-Id: I1310b9da1262b06e77a621831d599cd4077c31ed
|
|
|
|
|
| |
Change-Id: I61da7bbe3f5b64f7c93b291a5ada52cde37997d9
Signed-off-by: Vitalii Koriakov <vkoriako@redhat.com>
|
|
|
|
|
| |
Change-Id: Id8f86e027e1109b0351dd45e073b68bf31fe1de6
Signed-off-by: Vitalii Koriakov <vkoriako@redhat.com>
|
|
|
|
|
| |
Change-Id: I616ffab2a797330778781a8245de87a0f5775df3
Signed-off-by: Vitalii Koriakov <vkoriako@redhat.com>
|
|
|
|
|
| |
Change-Id: I2b5e8cb9d02967736f18ed80c1aaff48ac684716
Signed-off-by: Vitalii Koriakov <vkoriako@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
block_libs.py:
Module for some gluster-block related helper
functions like block info validation, checking if block exists etc.
This module will call the functions in block_ops and the result will be
used for further operations.
These functions are either called from GlusterBlockBaseClass or can
be called directly from testcases.
block_ops.py:
Module for gluster-block helper functions like block_create, block_delete,
block_modify, block_info, block_list.
The functions here will either be called from GlusterBlockBaseClass or
block_libs.py or can be called directly from the testcases.
Change-Id: I5d82ac32bbf27a2a6597ad275ff09b3601615f2b
Signed-off-by: Bhumika Goyal <bgoyal@redhat.com>
|
|
|
|
| |
Change-Id: I46fc2feffe6443af6913785d67bf310838532421
|
|
|
|
|
| |
Change-Id: I0a439b8defeb6b8e72694f1d97e1309a7ba988fa
Signed-off-by: Prasad Desala <tdesala@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
It could have failed without anyone noticing.
Added 'xfail' - do we expect to fail in deletion
(and changed tests accordingly)
On the way, ensure stdout and stderr are logged in case of such failures.
Change-Id: Ibdf7a43cadb0393707a6c68c19a664453a971eb1
Signed-off-by: Yaniv Kaul <ykaul@redhat.com>
|