summaryrefslogtreecommitdiffstats
path: root/glustolibs-gluster/glustolibs/gluster/brickmux_ops.py
Commit message (Collapse)AuthorAgeFilesLines
* [Libfix] Fix check_brick_pid_matches_glusterfsd_pid() to use pgrepkshithijiyer2020-04-281-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Problem: On latest platforms pidof command is returning multiple pids as shown below: 27190 27078 26854 This is becasue it was returning glusterd,glusterfsd and glusterfs processes as well. The problem is that /usr/sbin/glusterd is a link to glusterfsd. 'pidof' has a new feature that pidof searches for the pattern in /proc/PID/cmdline, /proc/PID/stat and finally /proc/PID/exe. Hence pidof matches realpath of /proc/<pid_of_glusterd>/exe as /usr/sbin/glusterfsd and results in glusterd, glusterfs and glusterfsd pids being returned in output. Fix: Use pgrep instead of pidof to get glusterfsd pids. And change the split logic accordingly. Change-Id: I729e05c3f4cacf7bf826592da965a94a49bb6f33 Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
* [Libfix] Fix get_brick_processes_count() to use pgrepkshithijiyer2020-04-271-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | Problem: On latest platforms pidof command is returning multiple pids as shown below: 27190 27078 26854 This is becasue it was returning glusterd,glusterfsd and glusterfs processes as well. The problem is that /usr/sbin/glusterd is a link to glusterfsd. 'pidof' has a new feature that pidof searches for the pattern in /proc/PID/cmdline, /proc/PID/stat and finally /proc/PID/exe. Hence pidof matches realpath of /proc/<pid_of_glusterd>/exe as /usr/sbin/glusterfsd and results in glusterd, glusterfs and glusterfsd pids being returned in output. Fix: Use pgrep instead of pidof to get glusterfsd pids. And change the split logic accordingly. Change-Id: Ie215734387989f2d8cb19e4b4f7cddc73d2a5608 Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
* [TC] Gluster cli reset commands behaviour with brick muxkshithijiyer2019-12-231-7/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Test case steps: 1. Set cluster.brick-multiplex to enabled. 2. Create and start 2 volumes of type 1x3 and 2x3. 3. Check if cluster.brick-multiplex is enabled. 4. Reset the cluster using "gluster v reset all". 5. Check if cluster.brick-multiplex is disabled. 6. Create a new volume of type 2x3. 7. Set cluster.brick-multiplex to enabled. 8. Stop and start all three volumes. 9. Check the if pids match and check if more than one pids of glusterfsd is present. Additional library fix: Changing the command in check_brick_pid_matches_glusterfsd_pid() as it won't work when brick-mux is enabled. From: cmd = ("ps -eaf | grep glusterfsd | " "grep %s.%s | grep -v 'grep %s.%s'" % (volname, brick_node, volname, brick_node)) To: cmd = "pidof glusterfsd" Change-Id: If7bdde13071732b176a0a2289635319571872e47 Co-authored-by: Vitalii Koriakov <vkoriako@redhat.com> Signed-off-by: Vitalii Koriakov <vkoriako@redhat.com> Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
* Fix brickmux_ops libraryVinayak Papnoi2019-08-021-1/+1
| | | | | | | | | 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 get_brick_processes_count() to brickmux_ops.kshithijiyer2019-07-231-0/+18
| | | | | | | | | | | 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>
* is_brick_mux_enabled misspelling fixesVitalii Koriakov2018-09-051-2/+1
| | | | | Change-Id: I61da7bbe3f5b64f7c93b291a5ada52cde37997d9 Signed-off-by: Vitalii Koriakov <vkoriako@redhat.com>
* Added methods and fixed importVitalii Koriakov2018-09-051-29/+50
| | | | | Change-Id: Id8f86e027e1109b0351dd45e073b68bf31fe1de6 Signed-off-by: Vitalii Koriakov <vkoriako@redhat.com>
* Changed disabled to disableVitalii Koriakov2018-08-301-1/+1
| | | | | Change-Id: I616ffab2a797330778781a8245de87a0f5775df3 Signed-off-by: Vitalii Koriakov <vkoriako@redhat.com>
* Adding is_brick_mux_disabled methodVitalii Koriakov2018-08-301-1/+18
| | | | | Change-Id: I2b5e8cb9d02967736f18ed80c1aaff48ac684716 Signed-off-by: Vitalii Koriakov <vkoriako@redhat.com>
* Fix spelling mistake across the codebaseNigel Babu2018-08-071-3/+3
| | | | Change-Id: I46fc2feffe6443af6913785d67bf310838532421
* Fix cyclic import in brickmux_ops.pyNigel Babu2018-04-251-5/+5
| | | | Change-Id: I487822efd6c57b8c51b65b4eaf149eb67e96731b
* Library to check brick pid matches pid of glusterfsd when brick mux feature ↵Akarsha2018-04-191-0/+46
| | | | | | | is enabled Change-Id: Id8d42c72548d618ac7bd1563ca2035018b7c2c99 Signed-off-by: Akarsha <akrai@redhat.com>
* Adding function to enable brick muxAkarsha2018-04-171-0/+16
| | | | | Change-Id: Id1396eb5211f1e189b603fcab916711dbceb30af Signed-off-by: Akarsha <akrai@redhat.com>
* Added function to check brickmux is enabled or notAkarsha2018-03-271-0/+40
Change-Id: Ie115ecc42a38391b0e8687d7f3acf9904c18b0ee Signed-off-by: Akarsha <akrai@redhat.com>