Bug 1415581 - RFE : Create trash directory only when its is enabled
Summary: RFE : Create trash directory only when its is enabled
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: trash-xlator
Version: 3.10
Hardware: All
OS: All
medium
medium
Target Milestone: ---
Assignee: Jiffin
QA Contact:
URL:
Whiteboard:
Depends On: 1264849
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-01-23 06:43 UTC by Anoop C S
Modified: 2017-03-06 17:44 UTC (History)
5 users (show)

Fixed In Version: glusterfs-3.10.0
Doc Type: Enhancement
Doc Text:
Clone Of: 1264849
Environment:
Last Closed: 2017-03-03 08:56:15 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Anoop C S 2017-01-23 06:43:24 UTC
+++ This bug was initially created as a clone of Bug #1264849 +++

Description of problem:

The creation of trash directory will performed only when it is enabled. And similarly operations such as rename , deletion will be prevented only when it is enabled. If creation fails (by some how) then trash xlator will off by throwing a warning message. The same is applicable for internal op directory.

--- Additional comment from Vijay Bellur on 2015-10-12 18:30:59 IST ---

REVIEW: http://review.gluster.org/12256 (features/trash : create trash directory only when it is enabled) posted (#2) for review on master by jiffin tony Thottan (jthottan)

--- Additional comment from Vijay Bellur on 2016-03-28 15:14:13 IST ---

REVIEW: http://review.gluster.org/12256 (features/trash : create trash directory only when it is enabled) posted (#3) for review on master by jiffin tony Thottan (jthottan)

--- Additional comment from Mike McCune on 2016-03-29 04:20:27 IST ---

This bug was accidentally moved from POST to MODIFIED via an error in automation, please see mmccune with any questions

--- Additional comment from Niels de Vos on 2016-05-10 15:02:09 IST ---

This didnt make it for GlusterFS 3.8, moving out of the tracking tree.

--- Additional comment from Worker Ant on 2016-08-30 01:39:42 IST ---

REVIEW: http://review.gluster.org/12256 (features/trash : create trash directory only when it is enabled) posted (#4) for review on master by jiffin tony Thottan (jthottan)

--- Additional comment from Worker Ant on 2016-08-30 12:20:58 IST ---

REVIEW: http://review.gluster.org/12256 (features/trash : create trash directory only when it is enabled) posted (#5) for review on master by jiffin tony Thottan (jthottan)

--- Additional comment from Worker Ant on 2016-12-14 15:03:07 IST ---

REVIEW: http://review.gluster.org/12256 (features/trash: Create trash directory only when it is enabled) posted (#6) for review on master by Anoop C S (anoopcs)

--- Additional comment from Worker Ant on 2016-12-21 11:07:43 IST ---

REVIEW: http://review.gluster.org/12256 (features/trash: Create trash directory only when it is enabled) posted (#7) for review on master by jiffin tony Thottan (jthottan)

--- Additional comment from Worker Ant on 2016-12-22 10:58:40 IST ---

REVIEW: http://review.gluster.org/12256 (features/trash: Create trash directory only when it is enabled) posted (#8) for review on master by Anoop C S (anoopcs)

--- Additional comment from Worker Ant on 2017-01-17 12:31:45 IST ---

REVIEW: http://review.gluster.org/12256 (features/trash: Create trash directory only when it is enabled) posted (#9) for review on master by jiffin tony Thottan (jthottan)

--- Additional comment from Worker Ant on 2017-01-18 18:09:28 IST ---

REVIEW: http://review.gluster.org/12256 (features/trash: Create trash directory only when it is enabled) posted (#10) for review on master by jiffin tony Thottan (jthottan)

--- Additional comment from Worker Ant on 2017-01-19 20:46:03 IST ---

REVIEW: http://review.gluster.org/12256 (features/trash: Create trash directory only when it is enabled) posted (#11) for review on master by jiffin tony Thottan (jthottan)

--- Additional comment from Worker Ant on 2017-01-20 17:42:25 IST ---

COMMIT: http://review.gluster.org/12256 committed in master by Jeff Darcy (jdarcy) 
------
commit 07b9853ad0c92b341be33a6cd632013c416221c8
Author: Jiffin Tony Thottan <jthottan>
Date:   Mon Sep 28 17:20:19 2015 +0530

    features/trash: Create trash directory only when it is enabled
    
    Previously trash directory was being created as part of volume
    start operation. And also the user/admin could not delete this
    directory from volume even if it is not needed. This patch will
    fix the same. From now onwards creation and enforcement on trash
    directory will come into pictrure only when trash translator is
    enabled. Similarly exact same behaviour is reflected on internal-op
    directory inside trash directory.
    
    Change-Id: I3e58316a7b299a691885e458c960438bec2220fb
    BUG: 1264849
    Signed-off-by: Jiffin Tony Thottan <jthottan>
    Reviewed-on: http://review.gluster.org/12256
    Smoke: Gluster Build System <jenkins.org>
    Tested-by: Anoop C S <anoopcs>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Reviewed-by: Anoop C S <anoopcs>
    Reviewed-by: Jeff Darcy <jdarcy>

Comment 1 Anoop C S 2017-02-07 10:28:39 UTC
Somehow Worker Ant failed to update:

https://review.gluster.org/#/c/16454/ merged in release-3.10.

Comment 2 Anoop C S 2017-03-03 08:56:15 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.10.0, please open a new bug report.

glusterfs-3.10.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://lists.gluster.org/pipermail/gluster-devel/2017-February/052173.html
[2] https://www.gluster.org/pipermail/gluster-users/

Comment 3 Shyamsundar 2017-03-06 17:44:03 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.10.0, please open a new bug report.

glusterfs-3.10.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://lists.gluster.org/pipermail/gluster-users/2017-February/030119.html
[2] https://www.gluster.org/pipermail/gluster-users/


Note You need to log in before you can comment on or make changes to this bug.