----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10038/#review18611 -----------------------------------------------------------
Ship it! - Sateesh Chodapuneedi On March 20, 2013, 4:57 a.m., Sateesh Chodapuneedi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10038/ > ----------------------------------------------------------- > > (Updated March 20, 2013, 4:57 a.m.) > > > Review request for cloudstack, Murali Reddy and Kelven Yang. > > > Description > ------- > > Moved validateVSMCluster method from CiscoNexusVSMDeviceManagerImpl to > CiscoNexusVSMElement. > > > This addresses bug CLOUDSTACK-301. > > > Diffs > ----- > > > plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/VmwareServerDiscoverer.java > 94ba97d > > plugins/hypervisors/vmware/src/com/cloud/network/CiscoNexusVSMDeviceManagerImpl.java > e17d99d > > plugins/hypervisors/vmware/src/com/cloud/network/element/CiscoNexusVSMElement.java > daf7dd6 > > plugins/hypervisors/vmware/src/com/cloud/network/element/CiscoNexusVSMElementService.java > 1cc9faf > > Diff: https://reviews.apache.org/r/10038/diff/ > > > Testing > ------- > > Tested adding a VMware cluster associated with Nexus 1000v. > Tested deleting the VMware cluster. > > > Thanks, > > Sateesh Chodapuneedi > >
