cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Abhinandan Prateek" <aprat...@apache.org>
Subject Review Request 30020: CLOUDSTACK-8164: Look for all host to avoid NPE
Date Mon, 19 Jan 2015 03:31:58 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30020/
-----------------------------------------------------------

Review request for cloudstack and Rohit Yadav.


Bugs: CLOUDSTACK-8164
    https://issues.apache.org/jira/browse/CLOUDSTACK-8164


Repository: cloudstack-git


Description
-------

CLOUDSTACK-8164: Look for all host to avoid NPE


Diffs
-----

  engine/storage/src/org/apache/cloudstack/storage/helper/VMSnapshotHelperImpl.java a1bcd45d3a20dd8965d3aa4cb8f630d7eb0b13f9

  server/src/com/cloud/vm/snapshot/VMSnapshotManagerImpl.java 6590a748a31647d5bc83f48d140a04d90bedb068


Diff: https://reviews.apache.org/r/30020/diff/


Testing
-------

Tested, a proper error is now shown when the snapshot on disabled host is deleted instead
of NPE.


Thanks,

Abhinandan Prateek


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message