Skip to content

Conversation

cardoe
Copy link
Contributor

@cardoe cardoe commented Aug 29, 2025

Fixes the following error:

2025-08-29 20:50:10.174 7 ERROR cinder.api.v3.attachments File "/var/lib/openstack/lib/python3.10/site-packages/cinder_understack/dynamic_netapp_driver.py", line 447, in create_export
2025-08-29 20:50:10.174 7 ERROR cinder.api.v3.attachments return lib.create_export(context, volume, connector)
2025-08-29 20:50:10.174 7 ERROR cinder.api.v3.attachments
2025-08-29 20:50:10.174 7 ERROR cinder.api.v3.attachments File "/var/lib/openstack/lib/python3.10/site-packages/cinder/volume/volume_utils.py", line 1563, in trace_method_logging_wrapper
2025-08-29 20:50:10.174 7 ERROR cinder.api.v3.attachments return f(*args, **kwargs)
2025-08-29 20:50:10.174 7 ERROR cinder.api.v3.attachments
2025-08-29 20:50:10.174 7 ERROR cinder.api.v3.attachments TypeError: NetAppNVMeStorageLibrary.create_export() takes 3 positional arguments but 4 were given

Fixes the following error:

2025-08-29 20:50:10.174 7 ERROR cinder.api.v3.attachments   File "/var/lib/openstack/lib/python3.10/site-packages/cinder_understack/dynamic_netapp_driver.py", line 447, in create_export
2025-08-29 20:50:10.174 7 ERROR cinder.api.v3.attachments     return lib.create_export(context, volume, connector)
2025-08-29 20:50:10.174 7 ERROR cinder.api.v3.attachments
2025-08-29 20:50:10.174 7 ERROR cinder.api.v3.attachments   File "/var/lib/openstack/lib/python3.10/site-packages/cinder/volume/volume_utils.py", line 1563, in trace_method_logging_wrapper
2025-08-29 20:50:10.174 7 ERROR cinder.api.v3.attachments     return f(*args, **kwargs)
2025-08-29 20:50:10.174 7 ERROR cinder.api.v3.attachments
2025-08-29 20:50:10.174 7 ERROR cinder.api.v3.attachments TypeError: NetAppNVMeStorageLibrary.create_export() takes 3 positional arguments but 4 were given
@cardoe cardoe requested a review from a team August 29, 2025 21:13
@skrobul skrobul added this pull request to the merge queue Sep 1, 2025
Merged via the queue into main with commit ce16d76 Sep 1, 2025
38 checks passed
@skrobul skrobul deleted the cinder-attach branch September 1, 2025 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants