Bug 1337113
Summary: | Modified volume options are not syncing once glusterd comes up. | ||
---|---|---|---|
Product: | [Community] GlusterFS | Reporter: | Atin Mukherjee <amukherj> |
Component: | glusterd | Assignee: | Atin Mukherjee <amukherj> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | |
Severity: | high | Docs Contact: | |
Priority: | unspecified | ||
Version: | 3.7.11 | CC: | bsrirama, bugs, kaushal |
Target Milestone: | --- | Keywords: | Regression, Triaged |
Target Release: | --- | ||
Hardware: | x86_64 | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | glusterfs-3.7.12 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | 1335531 | Environment: | |
Last Closed: | 2016-06-28 12:18:31 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: | 1335357, 1335531, 1337114 | ||
Bug Blocks: |
Description
Atin Mukherjee
2016-05-18 10:40:46 UTC
REVIEW: http://review.gluster.org/14410 (glusterd: copy real_path from older brickinfo during brick import) posted (#1) for review on release-3.7 by Atin Mukherjee (amukherj) COMMIT: http://review.gluster.org/14410 committed in release-3.7 by Kaushal M (kaushal) ------ commit 93b72a135a7793b5b71c0eb6498c5fe529827ca6 Author: Atin Mukherjee <amukherj> Date: Wed May 11 18:24:40 2016 +0530 glusterd: copy real_path from older brickinfo during brick import Backport of http://review.gluster.org/14306 In glusterd_import_new_brick () new_brickinfo->real_path will not be populated for the first time and hence if the underlying file system is bad for the same brick, import will fail resulting in inconsistent configuration data. Fix is to populate real_path from old brickinfo object. Also there were many cases where we were unnecessarily calling realpath() and that may cause in failure. For eg - if a remove brick is executed with a brick whoose underlying file system has crashed, remove-brick fails since realpath() call fails. We'd need to call realpath() here as the value is of no use.Hence passing construct_realpath as _gf_false in glusterd_volume_brickinfo_get_by_brick () is a must in such cases. Change-Id: I7ec93871dc9e616f5d565ad5e540b2f1cacaf9dc BUG: 1337113 Signed-off-by: Atin Mukherjee <amukherj> Reviewed-on: http://review.gluster.org/14306 Smoke: Gluster Build System <jenkins.com> NetBSD-regression: NetBSD Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.com> Reviewed-by: Kaushal M <kaushal> Reviewed-on: http://review.gluster.org/14410 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.7.12, please open a new bug report. glusterfs-3.7.12 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] https://www.gluster.org/pipermail/gluster-devel/2016-June/049918.html [2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user |