Cannot launch instance with Django Launch Instance form with multiple networks for a tenant

Bug #1715066 reported by Akihiro Motoki
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
High
Ivan Kolodyazhny

Bug Description

When LAUNCH_INSTANCE_LEGACY_ENABLED is set to True and the Django launch instance form is used, a user cannot launch an instance when there is multiple networks. I selected network(s) and submitted the form, but the launch instance workflow was shown again. The selected networks are reset and all networks are shown in the Available Networks. As a result, users cannot launch instances with the Django Launch Instance form.

It seems something has changed in Django 1.11.

Previously network ID is shown with a small gray characters but now "(undefined)" is shown, so it looks like generate_html() JS function does not work expectedly.

Akihiro Motoki (amotoki)
tags: added: pike-backport-potential
Changed in horizon:
importance: Undecided → High
Akihiro Motoki (amotoki)
summary: - Cannot launch instance with Django Launch Instance form when two or more
- networks
+ Cannot launch instance with Django Launch Instance form with multiple
+ networks for a tenant
Ying Zuo (yingzuo)
Changed in horizon:
milestone: none → queens-1
Ying Zuo (yingzuo)
Changed in horizon:
status: New → Confirmed
Ivan Kolodyazhny (e0ne)
Changed in horizon:
assignee: nobody → Ivan Kolodyazhny (e0ne)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to horizon (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/506190

Revision history for this message
Ivan Kolodyazhny (e0ne) wrote :

Verified that it's not reproducible with django==1.10

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/507560

Changed in horizon:
status: Confirmed → In Progress
Changed in horizon:
assignee: Ivan Kolodyazhny (e0ne) → Akihiro Motoki (amotoki)
Akihiro Motoki (amotoki)
Changed in horizon:
assignee: Akihiro Motoki (amotoki) → Ivan Kolodyazhny (e0ne)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

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

commit c13d6da80e7a68c28aa0adfe63459be8a6dbedfd
Author: Ivan Kolodyazhny <email address hidden>
Date: Tue Sep 26 17:26:45 2017 +0300

    Add render method to ThemableCheckboxSelectMultiple

    Django 1.11+ doesn't have RenderMixin class for widgets, so we have to
    implement render method for ThemableCheckboxSelectMultiple to get it
    rendered in a correct way with all supported Django versions until we
    find a better solution for our widgets implementation.

    Change-Id: I656afb162e130f2b77853368945b74330bedf747
    Closes-Bug: #1715066

Changed in horizon:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/horizon 13.0.0.0b1

This issue was fixed in the openstack/horizon 13.0.0.0b1 development milestone.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (stable/pike)

Fix proposed to branch: stable/pike
Review: https://review.openstack.org/534355

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (stable/pike)

Reviewed: https://review.openstack.org/534355
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=2ab0ea8c6a6eb8f1d6d6e04959426487c558c4a9
Submitter: Zuul
Branch: stable/pike

commit 2ab0ea8c6a6eb8f1d6d6e04959426487c558c4a9
Author: Ivan Kolodyazhny <email address hidden>
Date: Tue Sep 26 17:26:45 2017 +0300

    Add render method to ThemableCheckboxSelectMultiple

    Django 1.11+ doesn't have RenderMixin class for widgets, so we have to
    implement render method for ThemableCheckboxSelectMultiple to get it
    rendered in a correct way with all supported Django versions until we
    find a better solution for our widgets implementation.

    Change-Id: I656afb162e130f2b77853368945b74330bedf747
    Closes-Bug: #1715066
    (cherry picked from commit c13d6da80e7a68c28aa0adfe63459be8a6dbedfd)

tags: added: in-stable-pike
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/horizon 12.0.3

This issue was fixed in the openstack/horizon 12.0.3 release.

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

Change abandoned by Ivan Kolodyazhny (<email address hidden>) on branch: master
Review: https://review.openstack.org/506190
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.

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.