Adding L2GW support for SRIOV scenario #118
Conversation
| NeutronSupportedPCIVendorDevs: ['8086:10ed','8086:10fb'] | ||
| NeutronPhysicalDevMappings: "physnet1:em2" | ||
| # Number of VFs that needs to be configured for a physical interface | ||
| NeutronSriovNumVFs: "em2:8" |
There was a problem hiding this comment.
shouldn't this be a per deployment setting? since nic names can change based on deployment
| # Number of VFs that needs to be configured for a physical interface | ||
| NeutronSriovNumVFs: "em2:8" | ||
| NovaPCIPassthrough: | ||
| - devname: "em2" |
There was a problem hiding this comment.
should this have spaces in front of it if it is a list of values for NovaPCIPassthrough?
| NeutronSriovNumVFs: "em2:8" | ||
| NovaPCIPassthrough: | ||
| - devname: "em2" | ||
| physical_network: "physnet1" |
There was a problem hiding this comment.
should this have '-' if it part of the list for NovaPCIPassthrough?
| NeutronTenantNetworkType: vlan | ||
| NeutronNetworkVLANRanges: physnet1:3001:3009 | ||
| ExtraConfig: | ||
| neutron::server::service_providers: [ 'L2GW:OpenDaylight:networking_odl.l2gateway.driver.OpenDaylightL2gwDriver:default' ] |
There was a problem hiding this comment.
this needs spaces in front of it as well
|
After talking to Feng, we will let the user choose the interface in the scenario description using the parameter sriov_iface, and it will update opnfv-environment.yaml file to overwrite the commented parameters related to that interface name. |
| neutron::server::service_providers: [ 'L2GW:OpenDaylight:networking_odl.l2gateway.driver.OpenDaylightL2gwDriver:default' ] | ||
|
|
||
| #enable DHCP on ODL | ||
| opendaylight::enable_dhcp: true |
There was a problem hiding this comment.
Is this line meant to be in ExtraConfig section? Also, why is it set here?
| NeutronMechanismDrivers: ['sriovnicswitch','openvswitch','opendaylight_v2'] | ||
| NeutronServicePlugins: "odl-router_v2,networking-l2gw" | ||
| NeutronNetworkType: 'vxlan' | ||
| NeutronTenantNetworkType: vlan |
There was a problem hiding this comment.
I dont see NeutronTenantNetworkType defined in our tht. I do see NeutronNetworkType, which is the tenant network type. That should be VLAN right? and remove line 16?
No description provided.