Bug 1214048 - IO touched a file undergoing migration fails for tiered volumes
Summary: IO touched a file undergoing migration fails for tiered volumes
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: tiering
Version: mainline
Hardware: x86_64
OS: Linux
high
high
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact: bugs@gluster.org
URL:
Whiteboard:
: 1212084 (view as bug list)
Depends On:
Blocks: qe_tracker_everglades 1212084 1219608 1260923
TreeView+ depends on / blocked
 
Reported: 2015-04-21 20:24 UTC by Dan Lambright
Modified: 2016-06-16 12:53 UTC (History)
7 users (show)

Fixed In Version: glusterfs-3.8rc2
Clone Of:
: 1219608 (view as bug list)
Environment:
Last Closed: 2016-06-16 12:53:42 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Dan Lambright 2015-04-21 20:24:50 UTC
Description of problem:

If a read IO occurs against a file that has reached rebalance phase 2, we redirect the IO to the destination. For tiered volumes, when we try to reopen the file (on the destination), the lower level DHT receives the open call and fails; it does not have a "cached subvol". Fix it to "teach" the lower level DHT of the new location by sending a locate first in this case.

Comment 1 Anand Avati 2015-04-21 20:52:13 UTC
REVIEW: http://review.gluster.org/10324 (uster/dht: tiered volumes may not allow access to files undergoing migration [WIP]) posted (#1) for review on master by Dan Lambright (dlambrig)

Comment 2 Anand Avati 2015-04-22 10:16:33 UTC
REVIEW: http://review.gluster.org/10324 (guster/dht: tiered volumes may not allow access to files undergoing migration [WIP]) posted (#2) for review on master by Dan Lambright (dlambrig)

Comment 3 Anand Avati 2015-04-23 16:16:04 UTC
REVIEW: http://review.gluster.org/10324 (guster/dht: tiered volumes may not allow access to files undergoing migration) posted (#3) for review on master by Dan Lambright (dlambrig)

Comment 4 Anand Avati 2015-04-29 19:39:19 UTC
REVIEW: http://review.gluster.org/10324 (guster/dht: tiered volumes may not allow access to files undergoing migration) posted (#4) for review on master by Dan Lambright (dlambrig)

Comment 5 Anand Avati 2015-05-04 14:38:27 UTC
REVIEW: http://review.gluster.org/10324 (guster/dht: tiered volumes may not allow access to files undergoing migration) posted (#5) for review on master by Dan Lambright (dlambrig)

Comment 6 Anand Avati 2015-05-05 16:46:32 UTC
COMMIT: http://review.gluster.org/10324 committed in master by Raghavendra G (rgowdapp) 
------
commit 8965ac78bfd26e7f45c9579894191817afafa15e
Author: Dan Lambright <dlambrig>
Date:   Wed Apr 29 15:35:56 2015 -0400

    guster/dht: tiered volumes may not allow access to files undergoing migration
    
    If a read IO occurs against a file that has reached rebalance
    phase 2, we redirect the IO to the destination. For tiered
    volumes, when we try to reopen the file (on the destination),
    the lower level DHT receives the open call and fails; it does
    not have a "cached subvol". Fix is to "teach" the lower level
    DHT of the new location by sending a locate before the open.
    
    Change-Id: Ia4acb0035ff1da15f6a8f9ed54f43c76e8b98f5f
    BUG: 1214048
    Signed-off-by: Dan Lambright <dlambrig>
    Signed-off-by: root <root.lab.eng.bos.redhat.com>
    Signed-off-by: Dan Lambright <dlambrig>
    Reviewed-on: http://review.gluster.org/10324
    Tested-by: NetBSD Build System
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Raghavendra G <rgowdapp>
    Tested-by: Raghavendra G <rgowdapp>

Comment 7 Niels de Vos 2015-05-15 13:07:31 UTC
This change should not be in "ON_QA", the patch posted for this bug is only available in the master branch and not in a release yet. Moving back to MODIFIED until there is an beta release for the next GlusterFS version.

Comment 8 Dan Lambright 2015-05-20 05:51:19 UTC
*** Bug 1212084 has been marked as a duplicate of this bug. ***

Comment 9 Nagaprasad Sathyanarayana 2015-10-25 14:44:16 UTC
Fix for this BZ is already present in a GlusterFS release. You can find clone of this BZ, fixed in a GlusterFS release and closed. Hence closing this mainline BZ as well.

Comment 10 Niels de Vos 2016-06-16 12:53:42 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.