Cinder

eqlx delete snapshot when snapshot is not present

Bug #1480377 reported by Eric Harney on 2015-07-31

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Cinder	Fix Released	Low	Sean McGinnis	Cinder newton-rc1 "RC1"

Bug Description

The DellEQLSanISCSIDriver snapshot_delete operation fails if the snapshot is not present on the backend. This should check for "Snapshot ... does not exist" in the command output and pass the operation.

delete_volume appears to be broken in the same way. It tries to catch exception.VolumeNotFound, but nothing in the path called throws that exception.

Tags:

Rajini Karthik (rajini-karthik) on 2016-06-23

Changed in cinder:
assignee:	nobody → Rajini Ram (rajini-ram)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-09-08: Fix proposed to cinder (master)

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

Changed in cinder:
status:	New → In Progress

Sean McGinnis (sean-mcginnis) on 2016-09-08

Changed in cinder:
assignee:	Rajini Ram (rajini-ram) → Sean McGinnis (sean-mcginnis)
milestone:	none → newton-rc1
importance:	Undecided → Low

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-09-08: Fix merged to cinder (master)

Reviewed: https://review.openstack.org/367043
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=e34bc5dc5c6d2a97f695523751f547709f6aa5e6
Submitter: Jenkins
Branch: master

commit e34bc5dc5c6d2a97f695523751f547709f6aa5e6
Author: Sean McGinnis <email address hidden>
Date: Wed Sep 7 20:45:20 2016 -0500

Eqlx: Ignore missing snapshot on delete

    The eqlx driver would fail if requested to delete a snapshot that no
    longer exists on the array. This should be ignored and return success
    since it is in the desired state.

Change-Id: I4d1d8fa7579b22629b2cafa3fad7d0d080c849ed
Closes-bug: #1480377