Bug 1112613 - [SNAPSHOT] : gluster snapshot delete doesnt provide option to delete all / multiple snaps of a given volume
Summary: [SNAPSHOT] : gluster snapshot delete doesnt provide option to delete all / mu...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: glusterd
Version: mainline
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: ---
Assignee: Sachin Pandit
QA Contact:
URL:
Whiteboard: SNAPSHOT
Depends On: 1048122
Blocks: 1145083
TreeView+ depends on / blocked
 
Reported: 2014-06-24 10:32 UTC by Sachin Pandit
Modified: 2015-05-14 17:42 UTC (History)
10 users (show)

Fixed In Version: glusterfs-3.7.0
Doc Type: Bug Fix
Doc Text:
Clone Of: 1048122
: 1145083 (view as bug list)
Environment:
Last Closed: 2015-05-14 17:26:04 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Comment 1 Anand Avati 2014-06-24 10:33:52 UTC
REVIEW: http://review.gluster.org/8162 (feature/snapshot : Provide an interface to delete all snapshots belonging to a system as-well-as to a particular volume.) posted (#1) for review on master by Sachin Pandit (spandit)

Comment 2 Anand Avati 2014-06-25 07:40:16 UTC
REVIEW: http://review.gluster.org/8162 (feature/snapshot : Provide an interface to delete all snapshots belonging to a system as-well-as to a particular volume.) posted (#2) for review on master by Sachin Pandit (spandit)

Comment 3 Anand Avati 2014-06-25 09:57:12 UTC
REVIEW: http://review.gluster.org/8162 (feature/snapshot : Provide an interface to delete all snapshots belonging to a system as-well-as to a particular volume.) posted (#3) for review on master by Sachin Pandit (spandit)

Comment 4 Anand Avati 2014-06-25 10:52:32 UTC
REVIEW: http://review.gluster.org/8162 (feature/snapshot : Provide an interface to delete all snapshots belonging to a system as-well-as to a particular volume.) posted (#4) for review on master by Sachin Pandit (spandit)

Comment 5 Anand Avati 2014-06-26 06:09:15 UTC
REVIEW: http://review.gluster.org/8162 (feature/snapshot : Provide an interface to delete all snapshots belonging to a system as-well-as to a particular volume.) posted (#5) for review on master by Sachin Pandit (spandit)

Comment 6 Anand Avati 2014-07-02 09:51:54 UTC
REVIEW: http://review.gluster.org/8162 (feature/snapshot : Interface to delete all snapshots belonging to a system as-well-as to a particular volume.) posted (#6) for review on master by Sachin Pandit (spandit)

Comment 7 Anand Avati 2014-07-08 06:43:37 UTC
REVIEW: http://review.gluster.org/8162 (feature/snapshot : Interface to delete all snapshots belonging to a system as-well-as to a particular volume.) posted (#7) for review on master by Sachin Pandit (spandit)

Comment 8 Anand Avati 2014-07-09 09:43:02 UTC
REVIEW: http://review.gluster.org/8162 (feature/snapshot : Interface to delete all snapshots belonging to a system as-well-as to a particular volume.) posted (#8) for review on master by Sachin Pandit (spandit)

Comment 9 Anand Avati 2014-07-15 06:24:50 UTC
REVIEW: http://review.gluster.org/8162 (feature/snapshot : Interface to delete all snapshots belonging to a system as-well-as to a particular volume.) posted (#9) for review on master by Sachin Pandit (spandit)

Comment 10 Anand Avati 2014-07-18 09:41:59 UTC
REVIEW: http://review.gluster.org/8162 (feature/snapshot : Interface to delete all snapshots belonging to a system as-well-as to a particular volume.) posted (#10) for review on master by Sachin Pandit (spandit)

Comment 11 Anand Avati 2014-07-18 09:47:16 UTC
REVIEW: http://review.gluster.org/8162 (feature/snapshot : Interface to delete all snapshots belonging to a system as-well-as to a particular volume.) posted (#11) for review on master by Sachin Pandit (spandit)

Comment 12 Anand Avati 2014-07-25 05:43:06 UTC
REVIEW: http://review.gluster.org/8162 (feature/snapshot : Interface to delete all snapshots belonging to a system as-well-as to a particular volume.) posted (#12) for review on master by Sachin Pandit (spandit)

Comment 13 Anand Avati 2014-07-25 10:23:50 UTC
COMMIT: http://review.gluster.org/8162 committed in master by Kaushal M (kaushal) 
------
commit 35c6ca05d8ee0e1be1b67ac64d32c21b195aaeea
Author: Sachin Pandit <spandit>
Date:   Mon Jun 23 09:35:52 2014 +0530

    feature/snapshot : Interface to delete all snapshots belonging
    to a system as-well-as to a particular volume.
    
    Problem :
    With the current design we can only delete a single snapshot.
    And the deletion of volume which contains snapshot is not allowed.
    Because of that user might be forced to delete all the snapshots
    manually before he is allowed to delete a volume.
    
    Solution:
    Following is the interface with which user can delete
    all the snapshots of a system or belonging to a particular volume.
    
            Syntax : gluster snapshot delete all
    
            *To delete all the snapshots present in a system
    
            Syntax : gluster snapshot delete volume <volname>
    
            *To deletes all the snapshot present in a volume specified.
    
    ========================================================================
    Sample Output:
    
    Case 1 : Deleting a single snapshot.
    [root@snapshot-24 glusterfs]# gluster snapshot delete snap1
    Deleting snap will erase all the information about the snap. Do you still want to continue? (y/n) y
    snapshot delete: snap1: snap removed successfully
    
    -----------------------------------------------------------------
    Case 2 : Deleting all the snapshots in a Volume.
    [root@snapshot-24 glusterfs]# gluster snapshot delete volume vol1
    Volume (vol1) contains 9 snapshot(s).
    Do you still want to continue and delete them?  (y/n) y
    snapshot delete: snap2: snap removed successfully
    snapshot delete: snap3: snap removed successfully
    snapshot delete: snap4: snap removed successfully
    snapshot delete: snap5: snap removed successfully
    .
    .
    .
    
    -----------------------------------------------------------------
    Case 3 : Deleting all the snapshots in a system.
    [root@snapshot-24 glusterfs]# gluster snapshot delete all
    System contains 4 snapshot(s).
    Do you still want to continue and delete them?  (y/n) y
    snapshot delete: snap7: snap removed successfully
    snapshot delete: snap8: snap removed successfully
    snapshot delete: snap9: snap removed successfully
    snapshot delete: snap10: snap removed successfully
    ========================================================================
    
    Change-Id: Ifec8e128ab2011cbbba208376b9c92cfbe7d8d71
    BUG: 1112613
    Signed-off-by: Sachin Pandit <spandit>
    Reviewed-on: http://review.gluster.org/8162
    Reviewed-by: Atin Mukherjee <amukherj>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Avra Sengupta <asengupt>
    Reviewed-by: Raghavendra Bhat <raghavendra>
    Reviewed-by: Kaushal M <kaushal>

Comment 14 Anand Avati 2014-07-28 04:27:31 UTC
REVIEW: http://review.gluster.org/8378 (doc : man page and admin-guide for newly introduced snapshot delete option.) posted (#1) for review on master by Sachin Pandit (spandit)

Comment 15 Anand Avati 2014-07-30 05:52:20 UTC
COMMIT: http://review.gluster.org/8378 committed in master by Kaushal M (kaushal) 
------
commit b56719fc478590815b6b8bc1e5a7dee5194555df
Author: Sachin Pandit <spandit>
Date:   Mon Jul 28 09:53:50 2014 +0530

    doc : man page and admin-guide for newly introduced snapshot delete option.
    
    Change-Id: Iab5e7f63d673a2040e8d83ab7280121ff468836e
    BUG: 1112613
    Signed-off-by: Sachin Pandit <spandit>
    Reviewed-on: http://review.gluster.org/8378
    Reviewed-by: Atin Mukherjee <amukherj>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Kaushal M <kaushal>

Comment 16 Anand Avati 2014-08-25 11:24:56 UTC
REVIEW: http://review.gluster.org/8534 (cli/snapshot : update of a snapshot delete syntax in documentation.) posted (#1) for review on master by Sachin Pandit (spandit)

Comment 17 Anand Avati 2014-08-25 11:26:30 UTC
REVIEW: http://review.gluster.org/8534 (cli/snapshot : update of a snapshot delete syntax in documentation.) posted (#2) for review on master by Sachin Pandit (spandit)

Comment 18 Anand Avati 2014-08-26 05:15:03 UTC
REVIEW: http://review.gluster.org/8534 (cli/snapshot : update of a snapshot delete syntax in documentation) posted (#3) for review on master by Sachin Pandit (spandit)

Comment 19 Anand Avati 2014-08-26 07:30:35 UTC
COMMIT: http://review.gluster.org/8534 committed in master by Kaushal M (kaushal) 
------
commit 71095db8f40888f6afc604c896c296dbe988d292
Author: Sachin Pandit <spandit>
Date:   Mon Aug 25 09:46:14 2014 +0530

    cli/snapshot : update of a snapshot delete syntax in documentation
    
    Change-Id: Id1a4b9684a8dd5750ee6eed841e3d5195407fb7e
    BUG: 1112613
    Signed-off-by: Sachin Pandit <spandit>
    Reviewed-on: http://review.gluster.org/8534
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Kaushal M <kaushal>

Comment 20 Niels de Vos 2015-05-14 17:26:04 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.0, please open a new bug report.

glusterfs-3.7.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://thread.gmane.org/gmane.comp.file-systems.gluster.devel/10939
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user

Comment 21 Niels de Vos 2015-05-14 17:35:27 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.0, please open a new bug report.

glusterfs-3.7.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://thread.gmane.org/gmane.comp.file-systems.gluster.devel/10939
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user

Comment 22 Niels de Vos 2015-05-14 17:37:49 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.0, please open a new bug report.

glusterfs-3.7.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://thread.gmane.org/gmane.comp.file-systems.gluster.devel/10939
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user

Comment 23 Niels de Vos 2015-05-14 17:42:30 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.0, please open a new bug report.

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