RE: Review request for storage motion on xenserver

2013-04-11 Thread Devdeep Singh
> -Original Message- > From: Devdeep Singh [mailto:devdeep.si...@citrix.com] > Sent: Friday, April 05, 2013 3:37 PM > To: dev@cloudstack.apache.org > Subject: RE: Review request for storage motion on xenserver > > Hi Edison, > > Thanks a lot for looking at the changes. I am

RE: Review request for storage motion on xenserver

2013-04-05 Thread Devdeep Singh
com] > Sent: Thursday, April 04, 2013 3:21 AM > To: dev@cloudstack.apache.org > Subject: RE: Review request for storage motion on xenserver > > Sorry for the late. Following is my comments on storage motion: > 1. We shouldn't touch volume state in virutalmachinemanager. I sp

RE: Review request for storage motion on xenserver

2013-04-03 Thread Edison Su
Sorry for the late. Following is my comments on storage motion: 1. We shouldn't touch volume state in virutalmachinemanager. I spend a lot of time trying to move volume related operations from virtualmachinemanager to storage subsystem, better to follow the same pattern. 2. The current storage