Bug 1343368
Summary: | Input / Output when chmoding files on NFS mount point | ||
---|---|---|---|
Product: | [Community] GlusterFS | Reporter: | Manikandan <mselvaga> |
Component: | nfs | Assignee: | Manikandan <mselvaga> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | 3.8.0 | CC: | bugs, jthottan, jules, lemonnier.k, ndevos, skoduri, smohan |
Target Milestone: | --- | Keywords: | Triaged |
Target Release: | --- | ||
Hardware: | x86_64 | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | glusterfs-3.8.0 | Doc Type: | If docs needed, set a value |
Doc Text: | Story Points: | --- | |
Clone Of: | 1343362 | Environment: | |
Last Closed: | 2016-06-16 12:33:36 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: | 1318204, 1343362 | ||
Bug Blocks: | 1336753, 1343367 |
Description
Manikandan
2016-06-07 08:35:29 UTC
REVIEW: http://review.gluster.org/14660 (nfs : store sattr properly in nfs3_setattr() call) posted (#1) for review on release-3.8 by Manikandan Selvaganesh (mselvaga) COMMIT: http://review.gluster.org/14660 committed in release-3.8 by Niels de Vos (ndevos) ------ commit ee86027acfe44dc070a8bfe9e37864c645276370 Author: Jiffin Tony Thottan <jthottan> Date: Mon Jun 6 18:10:09 2016 +0530 nfs : store sattr properly in nfs3_setattr() call nfs3_setattr stores the input arguments in cs->stbuf. However, inode/entry resolution code overwrites cs->stbuf after a successful resolution, thereby overwriting the input arguments with iatt values stored on backend. Hence operations like chmod/chown turns out to be a NOP. Specifically following are the functions that overwrite cs->stbuf: nfs3_fh_resolve_inode_lookup_cbk nfs3_fh_resolve_entry_lookup_cbk Since we resort to inode resolution only when inode is not found in inode table and lru limit guards the number of inodes in itable, we run into this issue only when the data set is bigger than lru limit of itable. Fix is to store input arguments in a member other than cs->stbuf. Thanks Du for suggesting the fix Backport of http://review.gluster.org/#/c/14657/ > Change-Id: I7caef48839d4f177c3557d7823fc1d35c8294939 > BUG: 1318204 > Signed-off-by: Jiffin Tony Thottan <jthottan> > Signed-off-by: Manikandan Selvaganesh <mselvaga> Change-Id: I7caef48839d4f177c3557d7823fc1d35c8294939 BUG: 1343368 Signed-off-by: Manikandan Selvaganesh <mselvaga> Reviewed-on: http://review.gluster.org/14660 Tested-by: Nigel Babu <nigelb> Smoke: Gluster Build System <jenkins.com> NetBSD-regression: NetBSD Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.com> Reviewed-by: jiffin tony Thottan <jthottan> Reviewed-by: Niels de Vos <ndevos> 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 *** Bug 1343367 has been marked as a duplicate of this bug. *** |