incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Musayev, Ilya" <imusa...@webmd.net>
Subject RE: Review Request: PATCH-5 for feature 'Support for VMware dvSwitch in CloudStack'.
Date Fri, 01 Feb 2013 17:12:10 GMT
Has this been committed yet? 

Thanks
ilya

-----Original Message-----
From: Sateesh Chodapuneedi [mailto:noreply@reviews.apache.org] On Behalf Of Sateesh Chodapuneedi
Sent: Thursday, January 31, 2013 8:47 PM
To: Murali Reddy; Kelven Yang
Cc: Sateesh Chodapuneedi; cloudstack
Subject: Review Request: PATCH-5 for feature 'Support for VMware dvSwitch in CloudStack'.


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

Review request for cloudstack, Murali Reddy and Kelven Yang.


Description
-------

This is 5/final patch for feature 'Support for VMware dvSwitch in CloudStack'.
Virtual switch type could be chosen at zone level or at cluster level for specific traffic
type.
UI functionality is also included.
autoExpand of dvPortGroup is available in code but disabled as its breaking because vCenter
4.1 does not support autoExpand feature.


This addresses bug CLOUDSTACK-657.


Diffs
-----

  plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/VmwareServerDiscoverer.java 684df54

  plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManager.java be3fe9f

  plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java
c450312 
  plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
96c4348 
  server/src/com/cloud/resource/ResourceManagerImpl.java f82424a 
  ui/scripts/system.js 2ce7672 
  ui/scripts/zoneWizard.js 26838a1 
  vmware-base/src/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java 50f9541 

Diff: https://reviews.apache.org/r/9210/diff/


Testing
-------

Testing
-------

Manual testing:-
1) Tested guest traffic over dvSwitch on a dedicated physical network. In this case management
and public traffic uses standard vSwitch on a common physical network.
2) Tested both guest traffic and public traffic over dvSwitch on a physical network.
3) Use optional parameters added to AddClusterCmd to override Zone level network traffic label.
Tested 2 clusters, one with standard vSwitch and other with dvSwitch.
4) Tested all 3 traffic types on single physical network with global parameter 'vmware.use.dvswitch'
set to false. This is default configuration scenario.


Added following tests,
1) Test fetching dvSwitch object from vCenter
2) Test for presence of dvPortGroup
3) Test presence of dvPortGroup
4) Test get existing dvPortGroup
5) fetch dvPortGroup configuration
6) Test compare dvPortGroup configuration
7) Test update dvPortGroup configuration


Thanks,

Sateesh Chodapuneedi

Mime
View raw message