failed to boot instance from image without cinder volume

[sean mooney]

using master horizon it is currently not possible
to boot and instance form a glance image without
allocating a cinder volume.

this prevent horizon from booting an instance if cinder is not deployed.

to reproduce on the launch instance screen go to the source tab
and select image (default) in the select boot source drop down.

below this item a volume size element and a ratio (yes nod) booted displayed asking if the volume should be deleted on instace delete

http://picpaste.com/horizon-IsnSMR2S.PNG

the volume is required to be requested and the minium size is 1 GB.

if cinder is not deployed this results in a failure to boot with the message
"abouted:block device mapping is invalid"

http://picpaste.com/horizon-error-om6qqHST.PNG

opening the developer console the only message that appears is a warning
JQMIGRATE: jQuery.fn.attr('selected') may use property instead of attribute from this section of code.

jQuery.migrateReset=function(){warnedAbout={};jQuery.migrateWarnings.length=0;};function migrateWarn(msg){var console=window.console;if(!warnedAbout[msg]){warnedAbout[msg]=true;jQuery.migrateWarnings.push(msg);if(console&&console.warn&&!jQuery.migrateMute){console.warn("JQMIGRATE: "+msg);if(jQuery.migrateTrace&&console.trace){console.trace();}}}}

expected behavior
allow selection of boot instace from image(create new volumen) in drop down or have checkbox to control
creation of cinder volume.

[Matt Borland (palecrow)]

Note, that console log message is standard.

[Matt Borland (palecrow)]

I'm researching the nature of the bug and trying to reproduce.

[Matt Borland (palecrow)]

Note, the bug filer did some research wonders if https://bugs.launchpad.net/horizon/+bug/1568866 may have introduced the bug as he believes he had successfully used the Angular Launch Instance wizard a few weeks or so before just fine in this context.

[sean mooney (sean-k-mooney)]

this bug may be related to a change in the default value of vol_create in
https://github.com/openstack/horizon/commit/3e65e337ee5c707a6f27a7f406d76dab751cb8b2

[sean mooney (sean-k-mooney)]

testing 8bf69bb0f96079302962ba42e1901fb3eb0f58d9 one commit before https://github.com/openstack/horizon/commit/3e65e337ee5c707a6f27a7f406d76dab751cb8b2 works
broken http://picpaste.com/horizon-IsnSMR2S.PNG working http://picpaste.com/horizon-working-ElB9HfZN.PNG

[OpenStack Infra (hudson-openstack)]

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

[Travis Tripp (travis-tripp)]

FYI, I am seeing this as well and getting the following error log:

Build of instance 71851033-94b2-499b-878e-3224e3d4ee12 aborted: Block Device Mapping is Invalid.
Code
500
Details
File "/opt/stack/nova/nova/compute/manager.py", line 1751, in _do_build_and_run_instance filter_properties) File "/opt/stack/nova/nova/compute/manager.py", line 1908, in _build_and_run_instance 'create.error', fault=e) File "/usr/local/lib/python2.7/dist-packages/oslo_utils/excutils.py", line 221, in __exit__ self.force_reraise() File "/usr/local/lib/python2.7/dist-packages/oslo_utils/excutils.py", line 197, in force_reraise six.reraise(self.type_, self.value, self.tb) File "/opt/stack/nova/nova/compute/manager.py", line 1873, in _build_and_run_instance block_device_mapping) as resources: File "/usr/lib/python2.7/contextlib.py", line 17, in __enter__ return self.gen.next() File "/opt/stack/nova/nova/compute/manager.py", line 2031, in _build_resources reason=e.format_message())
Created
June 14, 2016, 11:03 p.m.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Rob Cresswell (<email address hidden>) on branch: master
Review: https://review.openstack.org/321223
Reason: https://review.openstack.org/#/c/270182/ seems to be a more complete solution to this issue.

[OpenStack Infra (hudson-openstack)]

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

commit 723e547e848f580bf5f16899cd2bffa23c77c763
Author: Sean Mooney <email address hidden>
Date: Wed May 25 21:18:47 2016 +0100

    Fix Launch Instance without Cinder

    A previous patch altered the defaults in the angular Launch Instance,
    but didn't account for those running without Cinder. This patch fixes
    those assumptions.

    Co-Authored-By: Rob Cresswell <email address hidden>
    Change-Id: I8dc54fa40fa7eb8ae95a13741405f2a80337b33d
    Closes-Bug: 1585761

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/376039

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/376039
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=58698541faaaee23f333ef45939fe10fe5687501
Submitter: Jenkins
Branch: stable/newton

commit 58698541faaaee23f333ef45939fe10fe5687501
Author: Sean Mooney <email address hidden>
Date: Wed May 25 21:18:47 2016 +0100

    Fix Launch Instance without Cinder

    A previous patch altered the defaults in the angular Launch Instance,
    but didn't account for those running without Cinder. This patch fixes
    those assumptions.

    Co-Authored-By: Rob Cresswell <email address hidden>
    Change-Id: I8dc54fa40fa7eb8ae95a13741405f2a80337b33d
    Closes-Bug: 1585761
    (cherry picked from commit 723e547e848f580bf5f16899cd2bffa23c77c763)

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/horizon 10.0.0.0rc2 release candidate.

[OpenStack Infra (hudson-openstack)]

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