x
/
group-based-policy
Code Issues Proposed changes

Fix APICManager Initialization 

Commit d612e2a0f9 changed
the initialization of APICManager, which resulted in an
unintentional re-ordering of parameters passed to it. This
caused the apic_system_id parameter to be used incorrectly
in the class, and therefore elements that used the paramter,
such as tenant name in ACI, were using the default value
instead ("openstack").

This patch ensures that the apic_system_id is passed as a
named parameter so that it will be used correctly as part
of initializaiton.

Change-Id: I0e2793c9c4ae8cf7aefd265db7f6ef5307b445f0
This commit is contained in:
Thomas Bachman 2017-07-28 21:26:03 +00:00
parent 3548a41500
commit 6687c7036b
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
gbpservice/neutron/services/grouppolicy/drivers/cisco/apic/apic_mapping.py
View File

@ -312,7 +312,7 @@ class ApicMappingDriver(api.ResourceMappingDriver,
ApicMappingDriver.manager = apic_manager.APICManager(
apic_model.ApicDbModel(), logging, network_config, apic_config,
apic_system_id,
apic_system_id=apic_system_id,
default_apic_model=('apic_ml2.neutron.plugins.ml2.drivers.'
'cisco.apic.apic_model'),
keystoneclientv3=keystoneclientv3)