Bug 1334443 - SAMBA-VSS : Permission denied issue while restoring the directory from windows client 1 when files are deleted from windows client 2
Summary: SAMBA-VSS : Permission denied issue while restoring the directory from window...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: libgfapi
Version: 3.8.0
Hardware: Unspecified
OS: Unspecified
urgent
urgent
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact: Sudhir D
URL:
Whiteboard:
Depends On: 1334444
Blocks: 1333668
TreeView+ depends on / blocked
 
Reported: 2016-05-09 15:50 UTC by Raghavendra Talur
Modified: 2016-06-16 14:06 UTC (History)
7 users (show)

Fixed In Version: glusterfs-3.8rc2
Doc Type: Bug Fix
Doc Text:
Clone Of: 1334441
: 1334444 (view as bug list)
Environment:
Last Closed: 2016-06-16 14:06:02 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Comment 1 Niels de Vos 2016-05-10 12:27:36 UTC
Please add a public description of this bug.

Comment 2 Raghavendra Talur 2016-05-11 09:56:57 UTC
Problem: Snapshot restore does not work for files that were deleted from the volume using a different windows client.

Steps to reproduce:
Create files from windows client1 and take snapshots. Now from a different windows client(say client2) delete some of the files that we created. On windows client1 right click over the directory where files were created and click on "Restore Previous Version" to see all the snapshots. Opening the snapshot directory shows all the files that are created initially in windows client1 (including the deleted files). However, when restore directory is requested we get a "Permission Denied" error on those deleted files.

How reproducible:
Always

NOTE: This requires vss plugin to be enabled in Samba and all uss related volume options set.

Comment 3 Vijay Bellur 2016-05-11 09:58:21 UTC
REVIEW: http://review.gluster.org/14289 (gfapi: clear loc.gfid when retrying after ESTALE) posted (#1) for review on release-3.8 by Raghavendra Talur (rtalur)

Comment 4 Vijay Bellur 2016-05-13 05:52:43 UTC
COMMIT: http://review.gluster.org/14289 committed in release-3.8 by Raghavendra Talur (rtalur) 
------
commit df12ab0937ce44d0190d7eb8f272170bf9515390
Author: Raghavendra Talur <rtalur>
Date:   Mon May 9 21:06:07 2016 +0530

    gfapi: clear loc.gfid when retrying after ESTALE
    
    If an ESTALE is returned as result of a revalidate lookup, it means the
    previous gfid and inode are to be discarded and lookup has to be tried
    as a fresh one. A fresh lookup should not have loc.gfid set. We were
    creating a new inode and passing it down but not clearing loc.gfid.
    This patch fixes that.
    
    >Change-Id: Ib192ada0528b5fb5e49b4e2555f2bcab62710e2d
    >BUG: 1334444
    >Signed-off-by: Raghavendra Talur <rtalur>
    >Reviewed-on: http://review.gluster.org/14274
    >Smoke: Gluster Build System <jenkins.com>
    >NetBSD-regression: NetBSD Build System <jenkins.org>
    >Reviewed-by: Niels de Vos <ndevos>
    >CentOS-regression: Gluster Build System <jenkins.com>
    >Reviewed-by: Pranith Kumar Karampuri <pkarampu>
    >(cherry picked from commit b2f09e531029f573772a09572cee0f8e1855481b)
    
    Change-Id: Ib192ada0528b5fb5e49b4e2555f2bcab62710e2d
    BUG: 1334443
    Signed-off-by: Raghavendra Talur <rtalur>
    Reviewed-on: http://review.gluster.org/14289
    Smoke: Gluster Build System <jenkins.com>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    Reviewed-by: Rajesh Joseph <rjoseph>
    CentOS-regression: Gluster Build System <jenkins.com>

Comment 5 Niels de Vos 2016-06-16 14:06:02 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.