Table of networks or routers does't support pagination

Bug #1746184 reported by Wangliangyu
28
This bug affects 5 people
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
In Progress
High
Unassigned

Bug Description

The neutron service support the pagination for networks and routers.
But the horizon does not support it.

Akihiro Motoki (amotoki)
tags: added: neutron pagination
Akihiro Motoki (amotoki)
Changed in horizon:
status: New → Confirmed
importance: Undecided → High
milestone: none → rocky-1
Wangliangyu (wangly)
Changed in horizon:
assignee: nobody → Wangliangyu (wangly)
Akihiro Motoki (amotoki)
Changed in horizon:
milestone: rocky-1 → rocky-2
Revision history for this message
Akihiro Motoki (amotoki) wrote :

Clearing the assignee due to inactivity.

Changed in horizon:
assignee: Wangliangyu (wangly) → nobody
milestone: rocky-2 → none
Changed in horizon:
assignee: nobody → XiaojueGuan (xiaojuegaun)
Changed in horizon:
assignee: XiaojueGuan (xiaojuegaun) → nobody
Changed in horizon:
assignee: nobody → Marek Lyčka (mareklycka)
Bui Hong Ha (telescreen)
tags: added: sts
Changed in horizon:
assignee: Marek Lyčka (mareklycka) → Erlon R. Cruz (sombrafam)
Akihiro Motoki (amotoki)
tags: removed: sts
Changed in horizon:
assignee: Erlon R. Cruz (sombrafam) → Heather Lemon (hypothetical-lemon)
Changed in horizon:
assignee: Heather Lemon (hypothetical-lemon) → nobody
Revision history for this message
Heather Lemon (hypothetical-lemon) wrote :
Changed in horizon:
assignee: nobody → Heather Lemon (hypothetical-lemon)
Revision history for this message
Heather Lemon (hypothetical-lemon) wrote :

I am currently still working on this bug. Fixing failing zuul tests.

Revision history for this message
Christian Rohmann (christian-rohmann) wrote :

Heather thank you for looking into this!
Your change is currently marked abandoned though (https://review.opendev.org/c/openstack/neutron/+/743953/).

I observe the same (no pagination, only 20 records shown) issue with designate and when managing i.e recordsets via

https://cloud.example.com/horizon/ngdetails/OS::Designate::Zone/ab284296-d0ec-4bb6-bfe9-b5e9baba9d42

Should I raise another bug for is this or do you believe this might somehow relate to the same root cause when talking to the API?

Revision history for this message
Heather Lemon (hypothetical-lemon) wrote :

Hi Christian,

Yes that MR was abandoned after being recommended to split the MR into 2 separate issues.
"It's better to split this patch to two independent ones. One for support pagination and sorting for agents."
This is the PR I've been dealing with.
https://review.opendev.org/c/openstack/neutron/+/791044

However, there are still zuul failures.

The issue is because the following function is not modified with the new arguments.
https://opendev.org/openstack/neutron/src/branch/master/neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py#L1183
https://review.opendev.org/c/openstack/neutron/+/743953/

I'm having a coworker take over this pagination network feature from me already.

| Should I raise another bug for is this or do you believe this might somehow relate to the same root cause when talking to the API?

Yes, the issue is with the API. It needs to be fixed in the core neutron package.

Here are a list of links you might be interested in.
https://bugs.launchpad.net/ubuntu/+source/python-openstackclient/+bug/1936419
https://bugs.launchpad.net/horizon/+bug/1746184
https://bugs.launchpad.net/neutron/+bug/1877254
https://bugs.launchpad.net/python-neutronclient/+bug/1936216
https://storyboard.openstack.org/#!/story/1646814

Thanks,
Heather Lemon

Changed in horizon:
assignee: Heather Lemon (hypothetical-lemon) → nobody
Revision history for this message
Christian Rohmann (christian-rohmann) wrote (last edit ):

Thanks Heather for this explicit response.

Apparently the bug with designate-dashboard was already known and is kind of long standing:

https://bugs.launchpad.net/designate-dashboard/+bug/1729261

Changed in horizon:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.opendev.org/c/openstack/horizon/+/806454
Committed: https://opendev.org/openstack/horizon/commit/196de449b6f7be3707cfd08fd1099c4ed792fba0
Submitter: "Zuul (22348)"
Branch: master

commit 196de449b6f7be3707cfd08fd1099c4ed792fba0
Author: Rodrigo Barbieri <email address hidden>
Date: Wed Aug 25 17:52:29 2021 -0300

    Implement pagination in admin/proj network tab

    Added pagination support to the networks page under
    Project and Admin Dashboard.

    To accomplish so, the method network_list_for_tenant
    in api/neutron.py that is used for listing networks under
    the Project Dashboard was refactored to merge the different
    requests for shared, non-shared and external networks while
    tracking the pagination for each of those requests, so for
    instance when all the shared networks are listed and paginated
    then the non-shared networks are queried and so on.

    For the Admin dashboard all network types are retrieved under
    a single request so it is a simpler pagination logic.

    Partial-Bug: #1746184
    Change-Id: I96a2d6cabed47c89bdc02ec922d7f9451e5ec025

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.