Comment 0 for bug 1627022

Revision history for this message
Lucas Alvares Gomes (lucasagomes) wrote : Incosistency deploy whole disk images in UEFI and BIOS mode

When deploying whole disk images in UEFI mode there's a check that prevents the deployment unless the boot_option capability is explicitly set to "local" (for local boot)[0]. This is not checked for non-UEFI deployment and I don't see the difference, both could be deployed w/o any problems because ironic already assumes local boot for whole disk images [1].

That said, the iLO driver seems to add yet another check [2], I don't understand the specifics of iLO so I would suggest to move this check to inside the iLO driver and have it performed only for that driver.

[0] https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L220-L229
[1] https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent.py#L338-L341
[2] https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ilo/deploy.py#L338-L346