Bug 1388563
Summary: | [Eventing]: 'VOLUME_REBALANCE' event messages have an incorrect volume name | ||
---|---|---|---|
Product: | [Community] GlusterFS | Reporter: | Nithya Balachandran <nbalacha> |
Component: | distribute | Assignee: | Nithya Balachandran <nbalacha> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | |
Severity: | high | Docs Contact: | |
Priority: | unspecified | ||
Version: | 3.9 | CC: | amukherj, bugs, rhinduja, sanandpa, storage-qa-internal, vbellur |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | glusterfs-3.9.0 | Doc Type: | If docs needed, set a value |
Doc Text: | Story Points: | --- | |
Clone Of: | 1388010 | Environment: | |
Last Closed: | 2016-12-06 05:59:10 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | 1388010 | ||
Bug Blocks: | 1386472 |
Comment 1
Nithya Balachandran
2016-10-25 16:28:42 UTC
RCA: Gluster translators do not store the actual volume name anywhere. Each translator appends a specific string to the volume name and stores this value in this->name. For dht, the suffix is "-dht" so this->name actually contains <volname>-dht. The event framework requires the actual volume name to be sent. The rebalance code incorrectly used strtok to parse the volume name by using "-dht" as the delimiter. strtok () treats every char in the delim string as a delimiter. So the parsing fails for a volume which contains 'd', 'h', or 't' in its name. Fix: The code was rewritten to use strstr instead. --- Additional comment from Worker Ant on 2016-10-25 07:09:26 EDT --- COMMIT: http://review.gluster.org/15712 committed in master by Raghavendra G (rgowdapp) ------ commit f4efbf0e3092bd2181f62be1e1f30f202678c866 Author: N Balachandran <nbalacha> Date: Mon Oct 24 13:54:41 2016 +0530 cluster/dht: Incorrect volname in rebalance events The rebalance event code was using strtok to parse the volume name which is incorrect. Reworked the code to get the correct volume name using strstr. Change-Id: Ib5f3305a34e6bf1ecfef677d87c5aff96bdeb0e6 BUG: 1388010 Signed-off-by: N Balachandran <nbalacha> Reviewed-on: http://review.gluster.org/15712 NetBSD-regression: NetBSD Build System <jenkins.org> Smoke: Gluster Build System <jenkins.org> Reviewed-by: Raghavendra G <rgowdapp> CentOS-regression: Gluster Build System <jenkins.org> REVIEW: http://review.gluster.org/15725 (cluster/dht: Incorrect volname in rebalance events) posted (#1) for review on release-3.9 by N Balachandran (nbalacha) COMMIT: http://review.gluster.org/15725 committed in release-3.9 by Pranith Kumar Karampuri (pkarampu) ------ commit 3c27ddac02366c6e5b015c832d4656f520680ced Author: N Balachandran <nbalacha> Date: Tue Oct 25 22:02:16 2016 +0530 cluster/dht: Incorrect volname in rebalance events The rebalance event code was using strtok to parse the volume name which is incorrect. Reworked the code to get the correct volume name using strstr. > Change-Id: Ib5f3305a34e6bf1ecfef677d87c5aff96bdeb0e6 > BUG: 1388010 > Signed-off-by: N Balachandran <nbalacha> > Reviewed-on: http://review.gluster.org/15712 > NetBSD-regression: NetBSD Build System <jenkins.org> > Smoke: Gluster Build System <jenkins.org> > Reviewed-by: Raghavendra G <rgowdapp> > CentOS-regression: Gluster Build System <jenkins.org> (cherry picked from commit f4efbf0e3092bd2181f62be1e1f30f202678c866) Change-Id: Ia22d6a8ce23e9dcab8281b4995277cbb3a196fda BUG: 1388563 Signed-off-by: N Balachandran <nbalacha> Reviewed-on: http://review.gluster.org/15725 Smoke: Gluster Build System <jenkins.org> NetBSD-regression: NetBSD Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.org> Reviewed-by: Pranith Kumar Karampuri <pkarampu> glusterfs-3.9.0rc2 is released[1] and packages are available for different distributions[2] to test. [1] http://www.gluster.org/pipermail/maintainers/2016-October/001601.html [2] http://www.gluster.org/pipermail/maintainers/2016-October/001605.html and http://www.gluster.org/pipermail/maintainers/2016-October/001606.html Gluster 3.9 GA is released http://blog.gluster.org/2016/11/announcing-gluster-3-9/ |