You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ra...@apache.org on 2017/02/21 00:16:39 UTC
[2/2] git commit: updated refs/heads/master to 1d1b503
Merge pull request #1924 from niteshsarda/CS-50213
CLOUDSTACK-9766 : Executing deleteSnapshot api with already deleted sIf we try to delete the snapshot which is already deleted, then no proper error appears in the log and it just try to delete the snapshot which is already deleted.
Steps to reproduce :
-------
1-create a snapshot
2-delete the snapshot
3-try to delete snapshot which is deleted in step 2
Expected Result
-------------
Result should show proper error message. Request for deleting already deleted snapshot should not be placed.
* pr/1924:
CLOUDSTACK-9766 : Executing deleteSnapshot api with already deleted snapshot does not throw any exception or failure message
Signed-off-by: Rajani Karuturi <ra...@accelerite.com>
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/1d1b503d
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/1d1b503d
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/1d1b503d
Branch: refs/heads/master
Commit: 1d1b503dec6472590ed4f4e73968258f526ba0ca
Parents: 3582c65 f8ad3e8
Author: Rajani Karuturi <ra...@accelerite.com>
Authored: Tue Feb 21 05:46:17 2017 +0530
Committer: Rajani Karuturi <ra...@accelerite.com>
Committed: Tue Feb 21 05:46:18 2017 +0530
----------------------------------------------------------------------
.../src/com/cloud/storage/dao/SnapshotDao.java | 2 ++
.../src/com/cloud/storage/dao/SnapshotDaoImpl.java | 16 ++++++++++++++++
.../cloud/storage/snapshot/SnapshotManagerImpl.java | 10 +++++++---
.../cloud/storage/snapshot/SnapshotManagerTest.java | 5 ++---
4 files changed, 27 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/1d1b503d/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java
----------------------------------------------------------------------
diff --cc server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java
index 2c7b4a0,5bc56b7..92937ba
mode 100644,100755..100755
--- a/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java
+++ b/server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java