Allow multiple backup deletes in the command line

Bug #1543056 reported by shilpa
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
python-cinderclient
Fix Released
Medium
shilpa

Bug Description

Currently only one backup can be deleted at a time. Allow multiple backups to be deleted in a single command.

shilpa (smanjara)
Changed in cinder:
assignee: nobody → shilpa (smanjara)
status: New → In Progress
Changed in cinder:
importance: Undecided → Medium
Revision history for this message
Sheel Rana (ranasheel2000) wrote :

Hi Shilpa,

This does not seem a bug rather addition to existing feature.
I would suggest to raise BP for same so that fix strategy can be discussed through spec, if required.

Also, could you please share if you are going to add this in cinder client or in cinder code itself?

Revision history for this message
shilpa (smanjara) wrote :

Hi Rana,

Here is the blueprint. https://blueprints.launchpad.net/cinder/+spec/python-cinderclient

This would be added in cinderclient.

shilpa (smanjara)
affects: cinder → python-cinderclient
Revision history for this message
vishal yadav (vishalcdac07) wrote :

Bulk delete of resources does not provide much advantage provided delete itself is async operation. Additionally there is overhead of tracking quota and resources hence monitoring of those bulk deletes.

Alternatively you can loop through backups and delete those one at a time.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-cinderclient (master)

Fix proposed to branch: master
Review: https://review.openstack.org/277895

Revision history for this message
Sheel Rana (ranasheel2000) wrote :

>Bulk delete of resources does not provide much advantage provided delete
> itself is async operation.
Actually delete backup is asyn operation. So, seems no monitoring required.

Revision history for this message
Sheel Rana (ranasheel2000) wrote :

BP for this is created in cinder and not in cinderclient. :(

updated BP link:
https://blueprints.launchpad.net/cinder/+spec/cli-backup-multiple-deletes

Revision history for this message
shilpa (smanjara) wrote :

>BP for this is created in cinder and not in cinderclient. :(

Thanks for pointing out. I have moved the BP to cinderclient.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-cinderclient (master)

Reviewed: https://review.openstack.org/277895
Committed: https://git.openstack.org/cgit/openstack/python-cinderclient/commit/?id=a3dca1599f1fe7add715a4d78318a7583941cd26
Submitter: Jenkins
Branch: master

commit a3dca1599f1fe7add715a4d78318a7583941cd26
Author: Shilpa Jagannath <email address hidden>
Date: Tue Feb 9 21:10:00 2016 +0530

    Allow "cinder backup-delete" to delete multiple backups in one request

    While "cinder snapshot-delete" and "cinder delete" allow multiple resources to be deleted in a single command, "cinder backup-delete" request can only delete one backup at a time. Adding this capability to backups in cinderclient. Enables "cinder backup-delete" to delete multiple backups in a single command.

    With this change the command can be run as below:

    cinder backup-delete <backup> [<backup>...]

    DocImpact
    Closes-Bug: #1543056
    Implements: blueprint cli-backup-multiple-deletes

    Change-Id: I767710bda3b7c358c6525c9a9f074010084e411d

Changed in python-cinderclient:
status: In Progress → Fix Released
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/python-cinderclient 1.6.0

This issue was fixed in the openstack/python-cinderclient 1.6.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

This issue was fixed in the openstack/python-cinderclient 1.6.0 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.