Cannot filter share server by share-network-subnet

Bug #1959329 reported by Felipe Rodrigues
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Shared File Systems Service (Manila)
Triaged
Low
Unassigned
python-manilaclient
Fix Released
Medium
Franca Mgbogu

Bug Description

INTRO
====================================
The share server list operation has the possibility to request share servers that is on specific share network subnet using the field "--share-network-subnet". However, the list is not returning any servers.

REPRODUCE:
=========================

1. Create share in subnet <id>
2. List share server in subnets <id>
3. No share servers.

See: [1]

PROBLEM
========================

The code is setting the query field as "share_network_subnet" [2], but the manila API expects the field as "share_network_subnet_id".

Apparently, there is no tempest test to check the share server subnet filter. It'd be nice to have at least one.

VERSION
========================
python-manilaclient: latest
manila: latest

REFERENCE
================================
[1] https://paste.opendev.org/show/812410/
[2] https://github.com/openstack/python-manilaclient/blob/master/manilaclient/v2/shell.py#L4311

Changed in python-manilaclient:
assignee: nobody → Franca Mgbogu (franciedigital)
Changed in python-manilaclient:
status: New → In Progress
Revision history for this message
Goutham Pacha Ravi (gouthamr) wrote :

Adding manila here to highlight Felipe's request for a tempest test -- this test must be added in manila-tempest-plugin, but there's no separate launchpad tracker page for that repository.

Revision history for this message
Franca Mgbogu (franciedigital) wrote :

Okay. I will create a tempest test for this.

Revision history for this message
Goutham Pacha Ravi (gouthamr) wrote :

Fix proposed to branch: master (python-manilaclient)
https://review.opendev.org/c/openstack/python-manilaclient/+/838293

Vida Haririan (vhariria)
Changed in manila:
assignee: nobody → Franca Mgbogu (franciedigital)
milestone: none → zed-1
importance: Undecided → Low
Revision history for this message
Vida Haririan (vhariria) wrote :
Revision history for this message
Franca Mgbogu (franciedigital) wrote :

Noted! Thanks

Changed in manila:
milestone: zed-1 → zed-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-manilaclient (master)

Reviewed: https://review.opendev.org/c/openstack/python-manilaclient/+/838293
Committed: https://opendev.org/openstack/python-manilaclient/commit/27b8830309c186c12f6b87c7d9a045325ea03e16
Submitter: "Zuul (22348)"
Branch: master

commit 27b8830309c186c12f6b87c7d9a045325ea03e16
Author: Franca Mgbogu <email address hidden>
Date: Mon Apr 18 12:43:49 2022 +0100

    Fix share server listing with network subnet id

    Share server list operation is supposed to accept share
    network subnet ids as parameters for filtering share servers.
    The share specified share network subnet ids were being
    ignored. This change fixes this behavior by allowing that
    search to occur properly.

    Closes-Bug: #1959329

    Change-Id: I6453e9d20a0b4ace0d71c501c5a65c6736554f0d

Changed in python-manilaclient:
status: In Progress → Fix Released
Changed in manila:
milestone: zed-2 → zed-3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/python-manilaclient 4.1.0

This issue was fixed in the openstack/python-manilaclient 4.1.0 release.

Changed in python-manilaclient:
importance: Undecided → Medium
milestone: none → zed-client-release
Changed in manila:
milestone: zed-3 → zed-rc1
assignee: Franca Mgbogu (franciedigital) → nobody
Changed in manila:
assignee: nobody → Fernando Ferraz (fernando-ferraz)
Changed in manila:
milestone: zed-rc1 → antelope-1
Changed in manila:
milestone: antelope-1 → antelope-2
Changed in manila:
assignee: Fernando Ferraz (fernando-ferraz) → Felipe Rodrigues (felipefutty)
Changed in manila:
milestone: antelope-2 → antelope-rc1
Revision history for this message
Goutham Pacha Ravi (gouthamr) wrote :

Adding the "low-hanging-fruit" tag because the only thing remaining here is to create a tempest test to test the filtering change done..

Changed in manila:
milestone: antelope-rc1 → bobcat-1
assignee: Felipe Rodrigues (felipefutty) → nobody
tags: added: low-hanging-fruit
Vida Haririan (vhariria)
Changed in manila:
status: New → Triaged
Changed in manila:
milestone: bobcat-1 → bobcat-2
Changed in manila:
milestone: bobcat-2 → bobcat-3
Changed in manila:
milestone: bobcat-3 → caracal-1
Changed in manila:
milestone: caracal-1 → caracal-2
Changed in manila:
milestone: caracal-2 → caracal-rc1
Changed in manila:
milestone: caracal-rc1 → 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.