openstack/vitrage
Code Issues Proposed changes

Merge "allow multiple template deletion"

Browse Source
This commit is contained in:
Zuul 2018-01-23 12:17:14 +00:00 committed by Gerrit Code Review
commit a2a44f6fd3
1 changed files with 15 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
vitrage/api/controllers/v1/template.py
View File

@ -181,20 +181,23 @@ class TemplateController(RootRestController):
}
@staticmethod
def _delete(uuid):
def _delete(uuids):
try:
storage = pecan.request.storage
templates = storage.templates.query(uuid=uuid)
if type(uuids) != list:
uuids = [uuids]
storage = pecan.request.storage.templates
templates = [t for _id in uuids for t in storage.query(uuid=_id)
if t.status != TStatus.DELETED]
if not templates:
raise VitrageError('template does not exists')
elif templates[0].status == TStatus.DELETED:
raise VitrageError('template is deleted')
elif templates[0].status == TStatus.ERROR:
storage.templates.update(uuid, "status", TStatus.DELETED)
elif templates[0].status == TStatus.ACTIVE:
storage.templates.update(uuid, "status", TStatus.DELETING)
pecan.request.client.call(pecan.request.context,
'delete_template')
raise VitrageError('Template not found')
for t in templates:
if t.status == TStatus.ERROR:
storage.update(t.uuid, "status", TStatus.DELETED)
else:
storage.update(t.uuid, "status", TStatus.DELETING)
pecan.request.client.call(pecan.request.context,
'delete_template')
except Exception as e:
LOG.exception('failed to delete template file %s ', e)
abort(404, str(e))