Cannot deploy to powered on node

Bug #1306202 reported by Jim Rollenhagen
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ironic
Invalid
Medium
Josh Gachnang

Bug Description

Nova may attempt to deploy to a powered on node, but Ironic will block this.

ironic-conductor returns the error "NodeInWrongPowerState: Can not change instance association while node <uuid> is in power state power on."

This may be desirable in some environments, but not in others. For example, Ironic-python-agent expects nodes to always be powered on, with the agent ramdisk running.

This validation should be done in the deploy driver, not in `update_node` in the conductor.

Tags: driver
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/86744

Changed in ironic:
assignee: nobody → Josh Gachnang (joshnang)
status: New → In Progress
Dmitry Tantsur (divius)
Changed in ironic:
importance: Undecided → Medium
aeva black (tenbrae)
Changed in ironic:
milestone: none → juno-2
tags: added: driver
Changed in ironic:
assignee: Josh Gachnang (joshnang) → Jim Rollenhagen (jim-rollenhagen)
Changed in ironic:
assignee: Jim Rollenhagen (jim-rollenhagen) → Josh Gachnang (joshnang)
Changed in ironic:
assignee: Josh Gachnang (joshnang) → Jim Rollenhagen (jim-rollenhagen)
Revision history for this message
Jim Rollenhagen (jim-rollenhagen) wrote :

Implementation for this is covered by this spec: https://review.openstack.org/#/c/102405/

... and this code review, kind of. Needs to be updated: https://review.openstack.org/#/c/86744/

I'll take this code review over and continue to aim for J2, but with spec review and all it may need to be bumped to J3.

Changed in ironic:
assignee: Jim Rollenhagen (jim-rollenhagen) → Josh Gachnang (joshnang)
Revision history for this message
aeva black (tenbrae) wrote :

bumping to J3 as this has grown into a spec and isn't approved yet

Changed in ironic:
milestone: juno-2 → juno-3
Revision history for this message
Josh Gachnang (joshnang) wrote :

Switching to Invalid since it is now part of the long running agent spec.

https://review.openstack.org/#/c/102405/

Changed in ironic:
status: In Progress → Invalid
Ruby Loo (rloo)
Changed in ironic:
milestone: juno-3 → none
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on ironic (master)

Change abandoned by Josh Gachnang (<email address hidden>) on branch: master
Review: https://review.openstack.org/86744

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.