On Wed, May 29, 2013 at 01:51:55PM +0000, Sateesh Chodapuneedi wrote: > > -----Original Message----- > > From: Chip Childers [mailto:chip.child...@sungard.com] > > Sent: 29 May 2013 00:25 > > To: Sateesh Chodapuneedi > > Cc: dev@cloudstack.apache.org > > Subject: Re: [MERGE] Storage migration support VMware into MASTER > > > > Specific code comments in the attachment. > > Hi Chip, > Thanks for reviewing the branch. > Kindly find my comments in the attachment. I have updated the branch per my > comments. > > Regards, > Sateesh
Looks good to me now! Also, thanks for explaining some of the bits. Especially the point about VMware API call consolidation. I should have remembered that myself frankly. +1 from me to merge.
