]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
CiscoFCSanLookupSerive uses extra argument in init
authorXing Yang <xing.yang@emc.com>
Sun, 26 Oct 2014 21:29:26 +0000 (17:29 -0400)
committerXing Yang <xing.yang@emc.com>
Sun, 26 Oct 2014 21:53:31 +0000 (17:53 -0400)
commit5cc9d0cffa44467051bdc01d64b889277d149359
tree89073abf944a5ef16acfe73b0e603d7b359cfe0a
parentfc87da7eeb3451e139ee71b31095d0b9093332ce
CiscoFCSanLookupSerive uses extra argument in init

This patch fixed two issues with the __init__ routine in
CiscoFCSanLookupService:

1. There's an extra argument in super(CiscoFCSanLookupService,
self).__init__(self, **kwargs). It should be changed to
super(CiscoFCSanLookupService, self).__init__(**kwargs).

2. The last line 'self.fabric_configs = ""' should be removed.
self.fabric_configs was created in self.create_configuration() in the
middle of the __init__ routine. It shouldn't be cleared out at the end
of the __init__ routine.

Change-Id: I2623f3b51af34d14849015e52e3676271c9fe414
Closes-Bug: #1385974
cinder/zonemanager/drivers/cisco/cisco_fc_san_lookup_service.py