Comment 2 for bug 1095744

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/18900
Committed: http://github.com/openstack/nova/commit/e91e6c07d9e34b79114ecac29b3669084e331f5a
Submitter: Jenkins
Branch: master

commit e91e6c07d9e34b79114ecac29b3669084e331f5a
Author: Pádraig Brady <email address hidden>
Date: Thu Jan 3 02:59:34 2013 +0000

    ensure failure to inject user files results in startup error

    This was the case in Essex but was inadvertantly changed in:
    folsom-2-95-g0d166ca.

    * nova/virt/disk/api.py: Refactor to allow specifying
    mandatory injection items, that result in an exception
    on failure to inject.
    * nova/virt/libvirt/driver.py: Specify that user 'files'
    are mandatory items and thus result in VM startup failure
    unless injected successfully.
    * nova/tests/test_virt_disk.py: A new test for the
    separate warning and error cases.

    Fixes bug: 1095744
    Change-Id: Idab5c4294c1cb52098ce44a7aae957a44fb2674f