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

(Updated Aug. 1, 2013, 1:41 a.m.)


Review request for cloudstack, Sebastien Goasguen and Hugo Trippaers.


Changes
-------

I made an update to refactor gre controller:

+ fix errors on preview patch
+ add OvsProvider to configPhysicalNetwork phase when deploying zone
+ remove sdn.ovs.controller to Global Setting
+ fix getLabel() //also work for multiNic servers


Repository: cloudstack-git


Description
-------

I made an update to refactor gre controller:

+ remove ovs_devices table, because we'll have an ODL plugin separately.
+ move command/answer to new package: com.cloud.agent.api
+ add Connectivity service checking
+ add new NetworkProvider: Ovs
+ add L3 services to Ovs Capabilities
+ add L3 services prototype code.

Next step:
+ L3 services implement with VirtualRouter
+ ODL plugin


Diffs (updated)
-----

  api/src/com/cloud/network/Network.java a06208b 
  api/src/com/cloud/network/OvsProvider.java PRE-CREATION 
  api/src/com/cloud/network/element/VirtualRouterElementService.java ea971b8 
  api/src/org/apache/cloudstack/api/ResponseGenerator.java d8d07cb 
  
api/src/org/apache/cloudstack/api/command/admin/router/ConfigureOvsElementCmd.java
 PRE-CREATION 
  
api/src/org/apache/cloudstack/api/command/admin/router/ListOvsElementsCmd.java 
PRE-CREATION 
  api/src/org/apache/cloudstack/api/response/OvsProviderResponse.java 
PRE-CREATION 
  client/tomcatconf/applicationContext.xml.in 60f1e30 
  client/tomcatconf/commands.properties.in 6ec4d92 
  engine/schema/src/com/cloud/network/dao/OvsProviderDao.java PRE-CREATION 
  engine/schema/src/com/cloud/network/dao/OvsProviderDaoImpl.java PRE-CREATION 
  engine/schema/src/com/cloud/network/element/OvsProviderVO.java PRE-CREATION 
  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
 30b0521 
  
plugins/network-elements/ovs/src/com/cloud/agent/api/OvsCreateGreTunnelAnswer.java
 PRE-CREATION 
  
plugins/network-elements/ovs/src/com/cloud/agent/api/OvsCreateGreTunnelCommand.java
 PRE-CREATION 
  
plugins/network-elements/ovs/src/com/cloud/agent/api/OvsCreateTunnelAnswer.java 
PRE-CREATION 
  
plugins/network-elements/ovs/src/com/cloud/agent/api/OvsCreateTunnelCommand.java
 PRE-CREATION 
  
plugins/network-elements/ovs/src/com/cloud/agent/api/OvsDeleteFlowCommand.java 
PRE-CREATION 
  
plugins/network-elements/ovs/src/com/cloud/agent/api/OvsDestroyBridgeCommand.java
 PRE-CREATION 
  
plugins/network-elements/ovs/src/com/cloud/agent/api/OvsDestroyTunnelCommand.java
 PRE-CREATION 
  
plugins/network-elements/ovs/src/com/cloud/agent/api/OvsFetchInterfaceAnswer.java
 PRE-CREATION 
  
plugins/network-elements/ovs/src/com/cloud/agent/api/OvsFetchInterfaceCommand.java
 PRE-CREATION 
  
plugins/network-elements/ovs/src/com/cloud/agent/api/OvsSetTagAndFlowAnswer.java
 PRE-CREATION 
  
plugins/network-elements/ovs/src/com/cloud/agent/api/OvsSetTagAndFlowCommand.java
 PRE-CREATION 
  
plugins/network-elements/ovs/src/com/cloud/agent/api/OvsSetupBridgeCommand.java 
PRE-CREATION 
  plugins/network-elements/ovs/src/com/cloud/agent/api/StartupOvsCommand.java 
PRE-CREATION 
  plugins/network-elements/ovs/src/com/cloud/network/element/OvsElement.java 
0ea6b52 
  
plugins/network-elements/ovs/src/com/cloud/network/element/OvsElementService.java
 b55fe6b 
  
plugins/network-elements/ovs/src/com/cloud/network/guru/OvsGuestNetworkGuru.java
 bbdf110 
  
plugins/network-elements/ovs/src/com/cloud/network/ovs/OvsTunnelManagerImpl.java
 b1ecaac 
  scripts/vm/hypervisor/xenserver/ovstunnel a4760b2 
  server/src/com/cloud/api/ApiResponseHelper.java 090d9c2 
  server/src/com/cloud/network/NetworkServiceImpl.java d4f9030 
  server/src/com/cloud/network/element/VirtualRouterElement.java 1916678 
  server/src/com/cloud/server/ManagementServerImpl.java 36b3879 
  setup/db/create-schema.sql 143023a 
  setup/db/db/schema-410to420.sql b48ae6a 
  tools/apidoc/gen_toc.py 33a7e75 
  ui/scripts/zoneWizard.js e2ad910 

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


Testing (updated)
-------

Test done with following cases:

+ choose GRE method, VLAN range when deploy advanced zone
+ create new Network Offering and choose Ovs for VirtualNetworking service, 
VirtualRouter for other services.
+ create a new Guest network using this new NetworkOffering
+ deploy VMs on this Guest network
+ stop, restart, destroy VMs
+ deploy, stop, restart, destroy VMs on two hosts.


Thanks,

tuna

Reply via email to