openstack/puppet-neutron
Code Issues Proposed changes

Cisco nexus config manifest - obsolete parameter (switch_replay_count).

Browse Source 
Due to fix: https://bugs.launchpad.net/networking-cisco/+bug/1469839,
the ml2_cisco replay count parameter is obsolete and no longer used.
This needs to be reflected in the tripleo-heat-templates as well as
puppet-neutron manifests.

Change-Id: I4967443fc978a6a0aba58cd3606ec3beeb49cbf1
Closes-bug:  #1551387
This commit is contained in:
Leon Zachery
2016-02-29 14:28:34 -05:00
parent 906e8dd2f5
commit f0801dc4c0
1 changed files with 10 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
manifests/plugins/ml2/cisco/nexus.pp
View File

@@ -59,9 +59,12 @@
# Defaults to 0
#
# [*switch_replay_count*]
# (optional) Number of times to attempt config replay with switch.
# This variable depends on switch_heartbeat_time being enabled.
# Defaults to 3
# (optional) This configuration item is OBSOLETE. The Nexus driver replay
# behavior is to continue to attempt to connect to the down Nexus device
# with a period equal to the heartbeat time interval. This was previously:
# Number of times to attempt config replay with switch.
# This variable depends on switch_heartbeat_time being enabled.
# Defaults to $::os_service_default
#
# [*provider_vlan_auto_create*]
# (optional) A flag indicating whether OpenStack networking should manage the
@@ -102,7 +105,7 @@ class neutron::plugins::ml2::cisco::nexus (
$provider_vlan_name_prefix = 'p-',
$persistent_switch_config = false,
$switch_heartbeat_time = 0,
$switch_replay_count = 3,
$switch_replay_count = $::os_service_default,
$provider_vlan_auto_create = true,
$provider_vlan_auto_trunk = true,
$vxlan_global_config = true,
@@ -110,6 +113,9 @@ class neutron::plugins::ml2::cisco::nexus (
) {
include ::neutron::plugins::ml2::cisco
if ! is_service_default($switch_replay_count) {
warning('The switch_replay_count parameter is obsolete. The Nexus driver will always attempt replay on reconnect, if enabled.')
}
neutron_plugin_ml2 {
'ml2_cisco/managed_physical_network' : value => $managed_physical_network;
'ml2_cisco/vlan_name_prefix' : value => $vlan_name_prefix;