Unable to get bootable volume list at launch instance

Bug #1457028 reported by Romain Chantereau
42
This bug affects 8 people
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Medium
Romain Chantereau

Bug Description

The launch instance dialog retrieve the list of the bootable volume available. In order to do this, it calls the “volume_list” function of cinderclient with corresponding filters.

Currently, the bootable flag is requested, but instead of asking a Boolean “True” value, the function is called with the logically true integer 1.

By doing this, the type is different, a boolean is expected and an integer is sent, and for DB taking care of type (like postgresql), the corresponding SQL request failed.

Revision history for this message
Romain Chantereau (romain-m) wrote :
Changed in horizon:
status: New → Fix Committed
Changed in horizon:
assignee: nobody → Romain Chantereau (romain-m)
Revision history for this message
Romain Chantereau (romain-m) wrote :

Fix Committed means patch with fix merged (thanks to aarefiev)

Changed in horizon:
status: Fix Committed → In Progress
Revision history for this message
Daniel Lundqvist (thorwald-m) wrote :

Are you still working on this Romain?

Revision history for this message
Romain Chantereau (romain-m) wrote :

Hi Daniel, I did not see your comment.

My fix for this bug is ready to be accepted no more work is required AFAIK.

Regards and HPY

David Lyle (david-lyle)
Changed in horizon:
importance: Undecided → Medium
milestone: none → mitaka-3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

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

commit d2e3c8465780ec013df253d5a34d18e709105704
Author: Romain Chantereau <email address hidden>
Date: Wed May 20 14:32:10 2015 +0200

    Trust sql middleware value conversion and stick to the spec type.

    Co-Authored-By: Sylvain Baubeau <email address hidden>
    Closes-Bug: #1457028

    Change-Id: I2c5bc22c2e024e22ad641d2b367fa3de2dd7636b

Changed in horizon:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/282837

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/horizon 9.0.0.0b3

This issue was fixed in the openstack/horizon 9.0.0.0b3 development milestone.

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

Reviewed: https://review.openstack.org/282837
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=8b894e0f411c184330acf70baec484609ebe11aa
Submitter: Jenkins
Branch: stable/liberty

commit 8b894e0f411c184330acf70baec484609ebe11aa
Author: Romain Chantereau <email address hidden>
Date: Wed May 20 14:32:10 2015 +0200

    Trust sql middleware value conversion and stick to the spec type.

    Co-Authored-By: Sylvain Baubeau <email address hidden>
    Closes-Bug: #1457028

    Change-Id: I2c5bc22c2e024e22ad641d2b367fa3de2dd7636b
    (cherry picked from commit d2e3c8465780ec013df253d5a34d18e709105704)

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

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

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.