share export location not updated with netapp driver

Bug #2017501 reported by Sylvan Le Deunff
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Shared File Systems Service (Manila)
In Progress
Low
Sylvan Le Deunff

Bug Description

Description
===========
When adding a subnet to a share network that is used by a share, the list of export path of this share is not updated. This bug is specific to NetApp multi SVM driver.

Steps to reproduce
==================

A chronological list of steps which will help reproduce the issue you hit:
* Configure manila to use NetApp drv_multi_svm share driver with option driver_handle_share_server=true
* Create a share network (this automatically create a default share network subnet)
* Create a share using this share network
* Create a share network subnet check
* Add a second subnet to this share network

Expected result
===============
The list of share's export paths should now contains 2 export paths, 1 in each subnet.

Actual result
=============
Export path list is not updated when the second subnet is created.

This is due to the following check: https://github.com/openstack/manila/blob/master/manila/share/drivers/netapp/dataontap/cluster_mode/lib_multi_svm.py#L2306

Tags: netapp
Changed in manila:
assignee: nobody → Sylvan Le Deunff (sylvanld)
Vida Haririan (vhariria)
tags: added: neta
tags: added: netapp
removed: neta
Revision history for this message
Carlos Eduardo (silvacarlose) wrote :

Hello, Sylvan! Thank you for the bug report. I see you have assigned yourself to this bug. Are you working on a fix? If not, we can ask the NetApp driver maintainers to take a look at this bug.

Changed in manila:
status: New → Triaged
importance: Undecided → Low
milestone: none → bobcat-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to manila (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/manila/+/882130

Changed in manila:
status: Triaged → In Progress
information type: Public → Public Security
Revision history for this message
Sylvan Le Deunff (sylvanld) wrote :

Hello, Carlos! Yes I just managed to submit the fix. It consists in removing the condition that seems needless.

information type: Public Security → Public
Changed in manila:
milestone: bobcat-2 → bobcat-3
Changed in manila:
milestone: bobcat-3 → bobcat-rc1
Changed in manila:
milestone: bobcat-rc1 → caracal-1
Changed in manila:
milestone: caracal-1 → caracal-2
Changed in manila:
milestone: caracal-2 → caracal-3
Changed in manila:
milestone: caracal-3 → dalmation-1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.