You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by ASF Subversion and Git Services <as...@urd.zones.apache.org> on 2013/04/02 14:11:06 UTC

Re: Review Request: CLOUDSTACK-301: Fix validation of VMware cluster associated with Nexus 1000v

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


Commit 009749fb796bc935f610921f1bdc71e8d993198d in branch refs/heads/master from Sateesh Chodapuneedi <sa...@apache.org>
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=009749f ]

CLOUDSTACK-301 Nexus 1000v DVS integration is not functional

Moved validateVSMCluster method from CiscoNexusVSMDeviceManagerImpl to CiscoNexusVSMElement.

Signed-off-by: Sateesh Chodapuneedi <sa...@apache.org>


- ASF Subversion and Git Services


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
> 
>