Bug 1271150 - libglusterfs : glusterd was not restarting after setting key=value length beyond PATH_MAX (4096) character
Summary: libglusterfs : glusterd was not restarting after setting key=value length bey...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: libglusterfsclient
Version: mainline
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Satish Mohan
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 1319649
TreeView+ depends on / blocked
 
Reported: 2015-10-13 09:10 UTC by Gaurav Kumar Garg
Modified: 2016-06-16 13:39 UTC (History)
9 users (show)

Fixed In Version: glusterfs-3.8rc2
Doc Type: Bug Fix
Doc Text:
Clone Of: 1269203
: 1319649 (view as bug list)
Environment:
Last Closed: 2016-06-16 13:39:57 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Comment 1 Vijay Bellur 2015-10-13 09:25:28 UTC
REVIEW: http://review.gluster.org/12346 (libglusterfs: read volfile based on the volfile size) posted (#1) for review on master by Gaurav Kumar Garg (ggarg)

Comment 2 Vijay Bellur 2015-10-13 14:15:29 UTC
REVIEW: http://review.gluster.org/12346 (libglusterfs: read volfile based on the volfile size) posted (#2) for review on master by Gaurav Kumar Garg (ggarg)

Comment 3 Vijay Bellur 2015-10-13 14:48:54 UTC
REVIEW: http://review.gluster.org/12346 (libglusterfs: read volume configuration file based on the file size) posted (#3) for review on master by Gaurav Kumar Garg (ggarg)

Comment 4 Vijay Bellur 2015-10-13 14:53:32 UTC
REVIEW: http://review.gluster.org/12346 (libglusterfs: restarting glusterd will fail while parsing volume info file.) posted (#4) for review on master by Gaurav Kumar Garg (ggarg)

Comment 5 Vijay Bellur 2015-10-13 15:59:01 UTC
REVIEW: http://review.gluster.org/12346 (libglusterfs: pass buffer size to gf_store_read_and_tokenize function) posted (#5) for review on master by Gaurav Kumar Garg (ggarg)

Comment 7 Vijay Bellur 2015-10-14 08:34:00 UTC
REVIEW: http://review.gluster.org/12346 (libglusterfs: pass buffer size to gf_store_read_and_tokenize function) posted (#6) for review on master by Gaurav Kumar Garg (ggarg)

Comment 8 Vijay Bellur 2015-10-14 20:42:34 UTC
COMMIT: http://review.gluster.org/12346 committed in master by Niels de Vos (ndevos) 
------
commit 816ca94f5dd49f34f395caf501de3c71f0ba113d
Author: Gaurav Kumar Garg <garg.gaurav52>
Date:   Tue Oct 13 14:40:55 2015 +0530

    libglusterfs: pass buffer size to gf_store_read_and_tokenize function
    
    Previously if user set an option where length of key=value goes beyond
    PATH_MAX (4096) character then tokenzing the option at the time of
    reading configuration file will fail.
    This is because of the we was having restraction in fgets to read maximum
    of PATH_MAX (4096) length of character.
    Consequence of this is when user try to restart glusterd, after setting
    key=value length beyond PATH_MAX (4096) character, glusterd will not restart.
    
    With this fix instead of PATH_MAX, consumer of gf_store_read_and_tokenize
    function will decide the size of the buffer length.
    
    Change-Id: I655a8ce982effdfff8f3e785ea31f543dbe39301
    BUG: 1271150
    Signed-off-by: Gaurav Kumar Garg <ggarg>
    Reviewed-on: http://review.gluster.org/12346
    Tested-by: NetBSD Build System <jenkins.org>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Anand Nekkunti <anekkunt>
    Reviewed-by: Niels de Vos <ndevos>

Comment 11 Niels de Vos 2016-06-16 13:39:57 UTC
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.8.0, please open a new bug report.

glusterfs-3.8.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://blog.gluster.org/2016/06/glusterfs-3-8-released/
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user


Note You need to log in before you can comment on or make changes to this bug.