Oh.. thanks for pinging. I'll look into.
(2013/05/31 9:42), Animesh Chaturvedi wrote:
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8294/ On January 29th, 2013, 9:43 a.m., *Hugo Trippaers* wrote: The change looks good. Could you update the patch to the latest state of master? On March 29th, 2013, 6:34 a.m., *Animesh Chaturvedi* wrote: Hiroaki can you update the patch to latetst master. Any updates Hiroaki? - Animesh On November 30th, 2012, 12:17 p.m., Hiroaki Kawai wrote: Review request for cloudstack. By Hiroaki Kawai. /Updated Nov. 30, 2012, 12:17 p.m./ Description There are 13 services defined in com.cloud.network.Network.Service , but db schema has 12 services with connecitivity missing in `cloud`.`physical_network_service_providers` . This patch will fix this. In the cloudstack api response, we'll see Connectivity service is enabled in NiciraNvp network service provider servicelist like this: { "listnetworkserviceprovidersresponse": { "count": 1, "networkserviceprovider": [ { "physicalnetworkid": "80bf64a1-c5ca-4eb1-810e-76917a317d59", "servicelist": [ "Connectivity" ], "name": "NiciraNvp", "destinationphysicalnetworkid": "0", "state": "Disabled", "id": "efb45f75-9fd7-43aa-9937-b5c871971c00" } ] } } Diffs * api/src/com/cloud/network/PhysicalNetworkServiceProvider.java (1b91311) * packaging/centos63/package.sh (83c0e5e) * server/src/com/cloud/network/dao/PhysicalNetworkServiceProviderVO.java (12b08af) * server/src/com/cloud/upgrade/dao/Upgrade40to41.java (5067f39) * setup/db/create-schema.sql (fff084e) * setup/db/db/schema-40to41.sql (PRE-CREATION) View Diff <https://reviews.apache.org/r/8294/diff/>
