Bug 1285230 - Data Tiering:File create terminates with "Input/output error" as split brain is observed
Summary: Data Tiering:File create terminates with "Input/output error" as split brain ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: tiering
Version: mainline
Hardware: Unspecified
OS: Unspecified
urgent
urgent
Target Milestone: ---
Assignee: Susant Kumar Palai
QA Contact: bugs@gluster.org
URL:
Whiteboard:
Depends On: 1275751 1286028 1286029 1291557 1358823
Blocks: 1260923 1290363
TreeView+ depends on / blocked
 
Reported: 2015-11-25 09:23 UTC by Susant Kumar Palai
Modified: 2016-07-21 14:39 UTC (History)
8 users (show)

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

Attachments (Terms of Use)

Comment 1 Vijay Bellur 2015-11-25 10:25:01 UTC 
REVIEW: http://review.gluster.org/12745 (feature/tier: Fix EIO during write) posted (#1) for review on master by Susant Palai (spalai)
Comment 2 Vijay Bellur 2015-11-26 09:16:16 UTC 
REVIEW: http://review.gluster.org/12745 (feature/tier: Fix EIO during write) posted (#2) for review on master by Susant Palai (spalai)
Comment 3 Vijay Bellur 2015-11-30 09:57:31 UTC 
REVIEW: http://review.gluster.org/12745 (feature/tier: Fix EIO during write) posted (#3) for review on master by Susant Palai (spalai)
Comment 4 Vijay Bellur 2015-12-01 07:29:12 UTC 
REVIEW: http://review.gluster.org/12745 (feature/tier: Fix EIO during write) posted (#4) for review on master by Susant Palai (spalai)
Comment 5 Vijay Bellur 2015-12-01 08:29:44 UTC 
REVIEW: http://review.gluster.org/12745 (feature/tier: Fix EIO during write) posted (#5) for review on master by Susant Palai (spalai)
Comment 6 Vijay Bellur 2015-12-01 10:59:19 UTC 
REVIEW: http://review.gluster.org/12745 (feature/tier: Fix EIO during write) posted (#6) for review on master by Susant Palai (spalai)
Comment 7 Vijay Bellur 2015-12-03 07:39:42 UTC 
REVIEW: http://review.gluster.org/12745 (feature/tier: Fix EIO during write) posted (#7) for review on master by Susant Palai (spalai)
Comment 8 Vijay Bellur 2015-12-07 12:40:13 UTC 
REVIEW: http://review.gluster.org/12894 (afr: refresh inode using fstat) posted (#1) for review on master by Ravishankar N (ravishankar)
Comment 9 Vijay Bellur 2015-12-08 05:01:50 UTC 
REVIEW: http://review.gluster.org/12745 (posix: fix posix_fgetxattr to return the correct error) posted (#8) for review on master by Susant Palai (spalai)
Comment 10 Vijay Bellur 2015-12-08 12:59:40 UTC 
REVIEW: http://review.gluster.org/12894 (afr: refresh inode using fstat) posted (#2) for review on master by Ravishankar N (ravishankar)
Comment 11 Vijay Bellur 2015-12-08 21:47:40 UTC 
REVIEW: http://review.gluster.org/12894 (afr: refresh inode using fstat) posted (#3) for review on master by Vijay Bellur (vbellur)
Comment 12 Vijay Bellur 2015-12-09 04:43:53 UTC 
REVIEW: http://review.gluster.org/12894 (afr: refresh inode using fstat) posted (#4) for review on master by Ravishankar N (ravishankar)
Comment 13 Vijay Bellur 2015-12-09 07:01:55 UTC 
COMMIT: http://review.gluster.org/12745 committed in master by Pranith Kumar Karampuri (pkarampu) 
------
commit 848bf0db0dde745e1f3a966db0143b7003bd49b5
Author: Susant Palai <spalai>
Date:   Wed Nov 25 05:07:03 2015 -0500

    posix: fix posix_fgetxattr to return the correct error
    
    posix_fgetxattr used to not updating op_ret
    and op_errno (initialized to -1 and ENOENT respectively)
    on success cases. Hence, it can return ENOENT even if all
    the opertions were sucessful.
    
    Change-Id: I5db3c1821208166a052d734b0be2553d009b8ee4
    BUG: 1285230
    Signed-off-by: Susant Palai <spalai>
    Reviewed-on: http://review.gluster.org/12745
    Tested-by: NetBSD Build System <jenkins.org>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu>
Comment 14 Vijay Bellur 2015-12-09 07:26:46 UTC 
REVIEW: http://review.gluster.org/12916 (storage/posix: fix dict leak in posix_fgetxattr) posted (#1) for review on master by Susant Palai (spalai)
Comment 15 Vijay Bellur 2015-12-09 08:59:53 UTC 
REVIEW: http://review.gluster.org/12916 (storage/posix: fix dict leak in posix_fgetxattr.) posted (#2) for review on master by Susant Palai (spalai)
Comment 16 Vijay Bellur 2015-12-09 09:11:12 UTC 
REVIEW: http://review.gluster.org/12916 (storage/posix: fix dict leak in posix_fgetxattr) posted (#3) for review on master by Susant Palai (spalai)
Comment 17 Vijay Bellur 2015-12-09 11:38:24 UTC 
REVIEW: http://review.gluster.org/12894 (afr: refresh inode using fstat) posted (#5) for review on master by Ravishankar N (ravishankar)
Comment 18 Vijay Bellur 2015-12-09 12:55:17 UTC 
REVIEW: http://review.gluster.org/12894 (afr: refresh inode using fstat) posted (#6) for review on master by Ravishankar N (ravishankar)
Comment 19 Vijay Bellur 2015-12-09 13:03:17 UTC 
REVIEW: http://review.gluster.org/12916 (storage/posix: fix dict leak in posix_fgetxattr) posted (#4) for review on master by Susant Palai (spalai)
Comment 20 Vijay Bellur 2015-12-10 04:46:44 UTC 
REVIEW: http://review.gluster.org/12894 (afr: refresh inode using fstat) posted (#7) for review on master by Ravishankar N (ravishankar)
Comment 21 Vijay Bellur 2015-12-10 10:32:02 UTC 
COMMIT: http://review.gluster.org/12916 committed in master by Pranith Kumar Karampuri (pkarampu) 
------
commit 3949541b21b1d03ab37c7ab39f95581d553251d4
Author: Susant Palai <spalai>
Date:   Wed Dec 9 03:53:45 2015 -0500

    storage/posix: fix dict leak in posix_fgetxattr
    
    Change-Id: I8c53805993570e6f37786dde2103cb884c026791
    BUG: 1285230
    Signed-off-by: Susant Palai <spalai>
    Reviewed-on: http://review.gluster.org/12916
    Tested-by: Gluster Build System <jenkins.com>
    Tested-by: NetBSD Build System <jenkins.org>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu>
Comment 22 Vijay Bellur 2015-12-11 16:19:26 UTC 
REVIEW: http://review.gluster.org/12894 (afr: refresh inode using fstat) posted (#8) for review on master by Dan Lambright (dlambrig)
Comment 23 Vijay Bellur 2015-12-14 10:47:35 UTC 
REVIEW: http://review.gluster.org/12894 (afr: refresh inode using fstat) posted (#9) for review on master by Ravishankar N (ravishankar)
Comment 24 Vijay Bellur 2015-12-15 16:52:36 UTC 
REVIEW: http://review.gluster.org/12894 (afr: refresh inode using fstat) posted (#10) for review on master by Ravishankar N (ravishankar)
Comment 25 Vijay Bellur 2015-12-16 16:12:16 UTC 
REVIEW: http://review.gluster.org/12894 (afr: refresh inode using fstat) posted (#11) for review on master by Ravishankar N (ravishankar)
Comment 26 Vijay Bellur 2015-12-18 06:15:04 UTC 
REVIEW: http://review.gluster.org/12894 (afr: refresh inode using fstat) posted (#12) for review on master by Ravishankar N (ravishankar)
Comment 27 Vijay Bellur 2015-12-18 09:42:00 UTC 
REVIEW: http://review.gluster.org/12894 (afr: refresh inode using fstat) posted (#13) for review on master by Ravishankar N (ravishankar)
Comment 28 Vijay Bellur 2015-12-21 04:04:42 UTC 
COMMIT: http://review.gluster.org/12894 committed in master by Pranith Kumar Karampuri (pkarampu) 
------
commit d5a9e765543fd6eec5d522e4f3f062bef97aefed
Author: Ravishankar N <ravishankar>
Date:   Mon Dec 7 18:03:03 2015 +0530

    afr: refresh inode using fstat
    
    For fd based operations (fgetxattr, readv etc.) if an inode refresh is
    required, do so using fstat instead of lookup. This is because the file
    might have been deleted by another client before refresh but posix
    mandates that FOPS using already open fds must still succeed.
    
    Change-Id: Id5f71c3af4892b648eb747f363dffe6208e7ac09
    BUG: 1285230
    Signed-off-by: Ravishankar N <ravishankar>
    Reviewed-on: http://review.gluster.org/12894
    Reviewed-by: Pranith Kumar Karampuri <pkarampu>
    Tested-by: Gluster Build System <jenkins.com>
    Tested-by: NetBSD Build System <jenkins.org>
Comment 29 Ravishankar N 2015-12-21 04:08:36 UTC 
All patches for this BZ are merged, moving this to MODIFIED.
Comment 30 Niels de Vos 2016-06-16 13:46:40 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.