[Bugs] [Bug 1787122] New: glusterd allowing to set server.statedump-path to file, non-existent file and non-existent paths

bugzilla at redhat.com bugzilla at redhat.com
Tue Dec 31 10:17:20 UTC 2019


https://bugzilla.redhat.com/show_bug.cgi?id=1787122

            Bug ID: 1787122
           Summary: glusterd allowing to set server.statedump-path to
                    file, non-existent file and non-existent paths
           Product: GlusterFS
           Version: mainline
          Hardware: x86_64
                OS: Linux
            Status: NEW
         Component: glusterd
          Severity: medium
          Priority: low
          Assignee: bugs at gluster.org
          Reporter: ypadia at redhat.com
                CC: bmekala at redhat.com, bugs at gluster.org,
                    kiyer at redhat.com, rhs-bugs at redhat.com,
                    risjain at redhat.com, sheggodu at redhat.com,
                    srakonde at redhat.com, storage-qa-internal at redhat.com,
                    vbellur at redhat.com, ypadia at redhat.com
        Depends On: 1725809
  Target Milestone: ---
    Classification: Community



Description of problem:
server.statedump-path is the path where statedumps are stored, by default it is
/var/run/gluster. And can be set to any valid directory path. It was observed
that server.statedump-path was also accepting file, non-existent file and
non-existent paths as well. And statedump command was successful even when
statedumps with all the three invalid paths. 

Default:
################################################################################
# gluster v get rep all | grep server.statedump-path
server.statedump-path                   /var/run/gluster/                       
################################################################################

Setting it to a file and taking statedumps:(file exists)
################################################################################
# ls -l a.txt 
-rw-r--r--. 1 root root 0 Jul  1 16:38 a.txt
# gluster v set rep server.statedump-path /root/a.txt 
volume set: success
# gluster v statedump rep 
volume statedump: success
# cat a.txt
################################################################################

Setting it to a file which doesn't exists and taking statedumps: 
################################################################################
# gluster v set rep server.statedump-path /root/haha.txt 
volume set: success
# gluster v statedump rep 
volume statedump: success
# gluster v get rep all | grep server.statedump-path
server.statedump-path                   /root/haha.txt  
################################################################################


Setting to a dir which doesn't exists:
################################################################################
# gluster v set rep server.statedump-path /root/kshithij/
volume set: success
# gluster v statedump rep 
volume statedump: success
# gluster v get rep all | grep server.statedump-path
server.statedump-path                   /root/kshithij/      
################################################################################

Version-Release number of selected component (if applicable):
mainline

How reproducible:
constantly 

Steps to Reproduce:
1.Create a volume of type replica (1x3).
2.Set server.statedump-path to:
  a. A file
  b. A non-existent file
  c. A non-existent paths
3.Take statedunp using the gluster statedump command:
# gluster v statedump rep 

Actual results:
glusterd allowing to set server.statedump-path to file, non-existent file and
non-existent paths.

Expected results:
glusterd shouldn't allow to set server.statedump-path to file, non-existent
file and non-existent paths.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1725809
[Bug 1725809] glusterd allowing to set server.statedump-path to file,
non-existent file and non-existent paths
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.


More information about the Bugs mailing list