Skip to content

Commit 2c31f33

Browse files
skrobulcardoe
authored andcommitted
fix(cinder-understack): workaround field mismatch from nova/cinder
The field that nova passes along in the nova ironic driver is 'initiator' but the NetApp cinder driver expects the data to be in the 'nqn' field, so copy it over for now and revisit this with upstream to have a stable interface here with other drivers.
1 parent 6b6ef30 commit 2c31f33

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

python/cinder-understack/cinder_understack/dynamic_netapp_driver.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -402,6 +402,9 @@ def extend_volume(self, volume, new_size):
402402

403403
def initialize_connection(self, volume, connector):
404404
"""Initialize connection to volume."""
405+
# TODO: the nova ironic driver sends the field 'initiator' but the NetApp
406+
# cinder driver expects the field to be 'nqn' so copy the field over
407+
connector["nqn"] = connector["initiator"]
405408
with self._volume_to_library(volume) as lib:
406409
return lib.initialize_connection(volume, connector)
407410

0 commit comments

Comments
 (0)