Remove pxe_deploy_timeout option

Bug #1270981 reported by Michael Davies
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Medium
Michael Davies

Bug Description

The pxe_deploy_timeout option is not used in Ironic as it's a vestigial option left over from nova-bm. So remove this option from ironic.drivers.modules.pxe

(In the future, add new config options in a higher layer for handling timeouts of various kinds of tasks, not just deployment)

This is the Ironic part of https://bugs.launchpad.net/nova/+bug/1195073

aeva black (tenbrae)
Changed in ironic:
status: New → Triaged
importance: Undecided → Medium
Michael Davies (mrda)
Changed in ironic:
assignee: nobody → Michael Davies (mrda)
Michael Davies (mrda)
summary: - Refactor timeout config options
+ Remove pxe_deploy_timeout option
description: updated
Michael Davies (mrda)
Changed in ironic:
status: Triaged → In Progress
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/67990

Revision history for this message
aeva black (tenbrae) wrote :

Posting this comment for reference. While this option is defunct, Ironic does need to handle timeouts -- see https://blueprints.launchpad.net/ironic/+spec/generic-timeouts

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

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

commit f5bafbf581827695886b125ab35d3d25f7bf1649
Author: Michael Davies <email address hidden>
Date: Tue Jan 21 11:04:52 2014 +1030

    Remove unused config option - pxe_deploy_timeout

    This is a vestigial config option left over from nova-bm refactoring, which is not used.

    Change-Id: Iede8e2935ccfcb4706ec61fc5ee2991dc5c334e2
    Closes-Bug: 1270981

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.