Re: Review Request 11984: Fix primary datastore NPE/incorrect db entry/exception propagation for KVM on cloudstack

2013-07-10 Thread edison su
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11984/#review22980 --- Ship it! Ship It! - edison su On July 2, 2013, 11:09 p.m., Venka

Re: Review Request 11984: Fix primary datastore NPE/incorrect db entry/exception propagation for KVM on cloudstack

2013-07-02 Thread Venkata Siva Vijayendra Bhamidipati
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11984/ --- (Updated July 2, 2013, 11:09 p.m.) Review request for cloudstack, Chip Childers

Re: Review Request 11984: Fix primary datastore NPE/incorrect db entry/exception propagation for KVM on cloudstack

2013-07-02 Thread Venkata Siva Vijayendra Bhamidipati
> On June 20, 2013, 10:37 p.m., edison su wrote: > > engine/api/src/org/apache/cloudstack/engine/subsystem/api/storage/DataStoreLifeCycle.java, > > line 44 > > > > > > If stoagemanager wants to delete datastore, then j