Comment 2 for bug 1529066

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

Reviewed: https://review.openstack.org/261562
Committed: https://git.openstack.org/cgit/openstack/fuel-agent/commit/?id=95fe335f2451703e68318787196a1ea4648f94e3
Submitter: Jenkins
Branch: master

commit 95fe335f2451703e68318787196a1ea4648f94e3
Author: Alexander Gordeev <email address hidden>
Date: Fri Dec 25 19:45:13 2015 +0300

    Place bootloader onto /boot partition-less disks

    Sometimes, there's no separate /boot fs image. Therefore,
    a bootloader should have been installed onto disk where
    rootfs image lands.

    It seems that Nailgun driver didn't support such cases and then
    an expection regarding missing /boot partition was thrown.

    That was totally wrong approach. Nailgun driver should work fine
    without /boot partition if there's no /boot image.

    In addition, this change removes ugly hacks from Ironic driver.
    Apparently Ironic driver explicitly sets self._boot_done and
    self._boot_partition_done flags to True, hence enables to bypass
    that exception. Since now, Nailgun can handle such a case, so
    there're no any needs to keep that hack in Ironic driver.

    Change-Id: I2d97fec7810643865abf0414f2cb80da1c591398
    Closes-Bug: #1529066