api.nova.Server makes suboptimal calls to Glance

Bug #1508554 reported by Timur Sufiev
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
In Progress
Medium
Ivan Kolodyazhny

Bug Description

Each access to Server.image_name property may result in a call to Glance API (see https://github.com/openstack/horizon/blob/8.0.0/openstack_dashboard/api/nova.py#L118) which is suboptimal and leads to many unnecessary requests when Server() is invoked from server_list() function.

Timur Sufiev (tsufiev-x)
Changed in horizon:
assignee: nobody → Timur Sufiev (tsufiev-x)
Timur Sufiev (tsufiev-x)
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/238189

Changed in horizon:
status: New → In Progress
Timur Sufiev (tsufiev-x)
tags: added: large-scale
Changed in horizon:
assignee: Timur Sufiev (tsufiev-x) → Paul Karikh (pkarikh)
Changed in horizon:
assignee: Paul Karikh (pkarikh) → Timur Sufiev (tsufiev-x)
Timur Sufiev (tsufiev-x)
Changed in horizon:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on horizon (master)

Change abandoned by Rob Cresswell (<email address hidden>) on branch: master
Review: https://review.openstack.org/238189
Reason: Abandoning for now, hasn't been updated in 3 months. Feel free to restore and update if you get time to work on the patch. Thanks!

Akihiro Motoki (amotoki)
Changed in horizon:
assignee: Timur Sufiev (tsufiev-x) → nobody
status: In Progress → Confirmed
Changed in horizon:
assignee: nobody → Anton Arefiev (aarefiev)
status: Confirmed → In Progress
Revision history for this message
Akihiro Motoki (amotoki) wrote :

The reported issue can be addressed by a fix for bug 1711486. bug 1711486 was reported later but it describes the situation when an issue reported here happens. A fix is already proposed, so I mark this as a duplicate of bug 1711486.

Changed in horizon:
assignee: Anton Arefiev (aarefiev) → Ivan Kolodyazhny (e0ne)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Ivan Kolodyazhny (<email address hidden>) on branch: master
Review: https://review.openstack.org/238189
Reason: This review is > 4 months without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to horizon (master)

Reviewed: https://review.openstack.org/238189
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=deb55b8411434b32a2ea92f1a825e7dea36deea5
Submitter: Zuul
Branch: master

commit deb55b8411434b32a2ea92f1a825e7dea36deea5
Author: BubaVV <email address hidden>
Date: Tue May 29 18:26:33 2018 +0300

    Call Glance list with certain image ids

    This patch uses glance list filtering, which allow to query images
    for the list of required ids. See [1] for details.

    Patch rebased to master. Added test for proposed Glance API request

    [1] https://developer.openstack.org/api-ref/image/v2/#show-images

    Related-Bug: #1508554
    Co-Authored-By: Ivan Kolodyazhny <email address hidden>
    Co-Authored-By: Vadym Markov <email address hidden>
    Change-Id: Iac485c1b448011fec97ba5bfaa83851914b294d9

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to horizon (stable/rocky)

Related fix proposed to branch: stable/rocky
Review: https://review.opendev.org/695593

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to horizon (stable/rocky)

Reviewed: https://review.opendev.org/695593
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=7f218c8e7d0ac78d066e397952ae8c8a9b7df985
Submitter: Zuul
Branch: stable/rocky

commit 7f218c8e7d0ac78d066e397952ae8c8a9b7df985
Author: BubaVV <email address hidden>
Date: Tue May 29 18:26:33 2018 +0300

    Call Glance list with certain image ids

    This patch uses glance list filtering, which allow to query images
    for the list of required ids. See [1] for details.

    Patch rebased to master. Added test for proposed Glance API request

    [1] https://developer.openstack.org/api-ref/image/v2/#show-images

    Related-Bug: #1508554
    Co-Authored-By: Ivan Kolodyazhny <email address hidden>
    Co-Authored-By: Vadym Markov <email address hidden>
    Change-Id: Iac485c1b448011fec97ba5bfaa83851914b294d9
    (cherry picked from commit deb55b8411434b32a2ea92f1a825e7dea36deea5)

tags: added: in-stable-rocky
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.