Prevent additional API call when hostname not available

Bug #1341696 reported by Suthan Venkataramanaiah
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-troveclient
Confirmed
Medium
Suthan Venkataramanaiah

Bug Description

Recently, a Trove "list" API change was made (https://bugs.launchpad.net/trove/+bug/1339400) to return IPs or hostnames. That change allows Horizon or Trove CLI to show IPs or hostnames of instances in lists with no additional calls.

With the above change merged, exactly one of hostname or IP will always be present in the list of instances. However, a call per row is still possible in Horizon today if hostname is not present (but IP is). This is because Horizon checks for hostname first and since hostname is not present, it lazy-loads each instance which causes the slow rendering of Trove instances in Horizon.

One proposal is to allow clients of python-troveclient to disable lazy-loading of returned resources. Then Horizon would set the flag to disable lazy-loading when calling instances.list().

Changed in python-troveclient:
assignee: nobody → Suthan Venkataramanaiah (svenkataramanaia)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-troveclient (master)

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

Changed in python-troveclient:
status: New → In Progress
Mat Lowery (mat-lowery)
description: updated
Changed in python-troveclient:
importance: Undecided → Medium
milestone: none → juno-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on python-troveclient (master)

Change abandoned by Suthan Venkataramanaiah (<email address hidden>) on branch: master
Review: https://review.openstack.org/108459

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Suthan Venkataramanaiah (<email address hidden>) on branch: master
Review: https://review.openstack.org/106843
Reason: Abandoning in favor of a more generic solution

Mat Lowery (mat-lowery)
Changed in python-troveclient:
status: In Progress → Confirmed
Revision history for this message
Mat Lowery (mat-lowery) wrote :

Adjusted bug description with a more generic solution.

description: updated
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.