Bug 1250864 - ec returns EIO error in cases where a more specific error could be returned
Summary: ec returns EIO error in cases where a more specific error could be returned
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: disperse
Version: 3.7.3
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Xavi Hernandez
QA Contact:
URL:
Whiteboard:
Depends On: 1245276
Blocks: glusterfs-3.7.4
TreeView+ depends on / blocked
 
Reported: 2015-08-06 07:51 UTC by Xavi Hernandez
Modified: 2015-09-09 09:39 UTC (History)
2 users (show)

Fixed In Version: glusterfs-3.7.4
Doc Type: Bug Fix
Doc Text:
Clone Of: 1245276
Environment:
Last Closed: 2015-09-09 09:39:07 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Xavi Hernandez 2015-08-06 07:51:32 UTC
+++ This bug was initially created as a clone of Bug #1245276 +++

Description of problem:

ec xlator uses EIO error to report some failures that could be better identified by an ENOMEM or EINVAL for example. This needs to be fixed to allow better error handling on the application side.

Version-Release number of selected component (if applicable): mainline


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

--- Additional comment from Anand Avati on 2015-07-22 16:07:36 CEST ---

REVIEW: http://review.gluster.org/11741 (cluster/ec: Minimize usage of EIO error) posted (#1) for review on master by Xavier Hernandez (xhernandez)

--- Additional comment from Anand Avati on 2015-07-27 15:58:59 CEST ---

REVIEW: http://review.gluster.org/11741 (cluster/ec: Minimize usage of EIO error) posted (#2) for review on master by Xavier Hernandez (xhernandez)

Comment 1 Anand Avati 2015-08-06 07:58:17 UTC
REVIEW: http://review.gluster.org/11852 (cluster/ec: Minimize usage of EIO error) posted (#1) for review on release-3.7 by Xavier Hernandez (xhernandez)

Comment 2 Anand Avati 2015-08-08 15:37:00 UTC
COMMIT: http://review.gluster.org/11852 committed in release-3.7 by Xavier Hernandez (xhernandez) 
------
commit 0e0c3ec84c73bc5fcda16ed743f9d3dc0a582c03
Author: Xavier Hernandez <xhernandez>
Date:   Tue Jul 21 18:05:06 2015 +0200

    cluster/ec: Minimize usage of EIO error
    
    >Change-Id: I82e245615419c2006a2d1b5e94ff0908d2f5e891
    >BUG: 1245276
    >Signed-off-by: Xavier Hernandez <xhernandez>
    >Reviewed-on: http://review.gluster.org/11741
    >Tested-by: Gluster Build System <jenkins.com>
    >Reviewed-by: Pranith Kumar Karampuri <pkarampu>
    >Tested-by: NetBSD Build System <jenkins.org>
    
    Change-Id: Ifd3d63f88a686a2963c5ba2e62110249f84f338d
    BUG: 1250864
    Signed-off-by: Xavier Hernandez <xhernandez>
    Reviewed-on: http://review.gluster.org/11852
    Reviewed-by: Pranith Kumar Karampuri <pkarampu>
    Tested-by: NetBSD Build System <jenkins.org>
    Tested-by: Gluster Build System <jenkins.com>

Comment 3 Kaushal 2015-09-09 09:39:07 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.7.4, please open a new bug report.

glusterfs-3.7.4 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://thread.gmane.org/gmane.comp.file-systems.gluster.devel/12496
[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.