Extra loop in populate_network_choices when launching instance

Bug #1359026 reported by Liyingjun
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Wishlist
Liyingjun

Bug Description

Liyingjun (liyingjun)
Changed in horizon:
assignee: nobody → Liyingjun (liyingjun)
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/115489

Changed in horizon:
status: New → In Progress
Revision history for this message
Gary W. Smith (gary-w-smith) wrote :

While it is true that the same network list is iterated twice, this change would only have a benefit if the list of networks were very large.

Changed in horizon:
importance: Undecided → Wishlist
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/115489
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=d02c9f4acdffef047d49d0c3568831ccf2373e1a
Submitter: Jenkins
Branch: master

commit d02c9f4acdffef047d49d0c3568831ccf2373e1a
Author: liyingjun <email address hidden>
Date: Thu Aug 7 08:31:53 2014 +0800

    Remove extra loop in populate_network_choices

    There is an extra loop in populate_network_choices in launch instance
    workflow, this could be merged into one loop.

    Change-Id: I03a988e95f23367ebc41fc13b7f08dfd2d9da9b7
    Closes-bug: 1359026

Changed in horizon:
status: In Progress → Fix Committed
Akihiro Motoki (amotoki)
Changed in horizon:
milestone: none → juno-3
Thierry Carrez (ttx)
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: juno-3 → 2014.2
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.