default ramdisk & kernel don't make sense

Bug #1272741 reported by Robert Collins
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Medium
Ghe Rivero

Bug Description

I just spotted this in a review:
[pxe]
# Default kernel image ID used in deployment phase (string
#host=127.0.0.1 615 # value)
#deploy_kernel=<None>

# Default ramdisk image ID used in deployment phase (string
# value)
#deploy_ramdisk=<None>

These options don't make sense:
- kernel + ramdisk are arch specific, but pxe can deploy to many arches from one machine

They are also unlikely to be used by deployers:
- they require config changes + service restarts vs a single API call [per flavour, but flavors are small #'s]

Ghe Rivero (ghe.rivero)
Changed in ironic:
status: New → Triaged
importance: Undecided → Medium
assignee: nobody → Ghe Rivero (ghe.rivero)
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/70140

Changed in ironic:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

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

commit 12b524c076fd0d129cd7b855fce7fd565a2641fb
Author: Ghe Rivero <email address hidden>
Date: Thu Jan 30 11:27:54 2014 +0100

    Remove deploy kernel and ramdisk global config

    This options are not very useful since ironic can deploy to
    multiple archs and any config change will require a service restart.

    Change-Id: I7282d86d0919d5057baff92010e4c1715126709e
    Closes-Bug: #1272741

Changed in ironic:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ironic:
milestone: none → icehouse-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ironic:
milestone: icehouse-3 → 2014.1
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.