"next links" broken for limited share replicas api

Bug #2023754 reported by Maurice Escher
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Shared File Systems Service (Manila)
Fix Released
Medium
christian okeke

Bug Description

Hi,

the link to get the next share replica when using pagination is broken, it reports the URL with underscore 'share_replicas' whereas it should be with hyphen 'share-replicas':

https://paste.opendev.org/show/820352/

Now I cannot use that link to get the next share replica(s).

Briefly looked into the code, the `_get_next_link` method uses the `_collection_name` which is not right in this case, since the collection name and URL route are different for share replicas.

BR,
Maurice

credit to: kayrus

description: updated
Vida Haririan (vhariria)
tags: added: replication
Vida Haririan (vhariria)
tags: added: low-hanging-fruit
Changed in manila:
importance: Undecided → Medium
Vida Haririan (vhariria)
Changed in manila:
assignee: nobody → Maurice Escher (maurice-escher)
status: New → Triaged
Changed in manila:
assignee: Maurice Escher (maurice-escher) → nobody
Revision history for this message
Vida Haririan (vhariria) wrote :
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/+/911709

Changed in manila:
status: Triaged → In Progress
Changed in manila:
milestone: none → caracal-rc1
Changed in manila:
assignee: nobody → christian okeke (ccokeke)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to manila (master)

Reviewed: https://review.opendev.org/c/openstack/manila/+/911709
Committed: https://opendev.org/openstack/manila/commit/c8236141f86fc609ee9901a42bbcffe60bd66e40
Submitter: "Zuul (22348)"
Branch: master

commit c8236141f86fc609ee9901a42bbcffe60bd66e40
Author: Okeke Christian <email address hidden>
Date: Thu Mar 7 14:03:13 2024 +0100

    Fix the link to get the next share replica

    The url to get the next share replica is broken when performing
    pagination. The new API uses underscore to represent resource.
    This fix resolves the broken link for share replicas by using alias.

    Closes-bug: #2023754
    Change-Id: I31e32c0bcfb60b53121da9a96df3b44ad2b3ac67

Changed in manila:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to manila (stable/2024.1)

Fix proposed to branch: stable/2024.1
Review: https://review.opendev.org/c/openstack/manila/+/913803

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.