Do not overwrite the iPXE boot script for every deployment

Bug #1489904 reported by Lucas Alvares Gomes
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Invalid
Low
Lucas Alvares Gomes

Bug Description

The iPXE boot script (default boot.ipxe [1]) gets overwritten for every node deployment, this script is unique for all deployments so we don't need to rewrite it all the time.

Also sometimes one may want to changing the content of the file to test a deployment (i.e something like [2]), but since Ironic always overwrite the file testing it gets more painful than it should.

[1] https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/boot.ipxe
[2] https://bugs.launchpad.net/ironic/+bug/1486950

Changed in ironic:
importance: Undecided → Low
assignee: nobody → Lucas Alvares Gomes (lucasagomes)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (master)

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

Changed in ironic:
status: New → In Progress
Changed in ironic:
assignee: Lucas Alvares Gomes (lucasagomes) → SHIGEMATSU Mitsuhiro (pshige)
Changed in ironic:
assignee: SHIGEMATSU Mitsuhiro (pshige) → Lucas Alvares Gomes (lucasagomes)
Revision history for this message
Julia Kreger (juliaashleykreger) wrote :

This fix is necessary as bifrost carries support for broken ipxe implementations by enabling them to chain-load to a current version and then proceed. Without it, bifrost looses that support.

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

Reviewed: https://review.openstack.org/218290
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=d72493deb21db90961c80757fd81d30fb39a43c8
Submitter: Jenkins
Branch: master

commit d72493deb21db90961c80757fd81d30fb39a43c8
Author: Lucas Alvares Gomes <email address hidden>
Date: Fri Aug 28 15:19:30 2015 +0100

    Do not overwrite the iPXE boot script on every deployment

    This patch just add a check to verify if the iPXE boot script have already
    been laid down in the HTTP root directory and do not overwrite it if
    it's already there. The script is the same for all deployments so we
    don't need to write it over and over. This also makes it easier to test
    some custom parameter in the script without having to deal with Ironic
    undoing the changes prior to deploy a node.

    Closes-Bug: #1489904
    Change-Id: I0d56767978a6d1290442e50d7cb1308d78e3a352

Changed in ironic:
status: In Progress → Fix Committed
Revision history for this message
Lucas Alvares Gomes (lucasagomes) wrote :

This bug is being reverted at https://review.openstack.org/219749

The change brings more risks than benefits

Changed in ironic:
status: Fix Committed → Invalid
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.