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

Review request for cloudstack.


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 

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


Testing
-------


Thanks,

Hiroaki Kawai

Reply via email to