Nodes should enter 'clean wait' state during a cleaning reboot

Bug #1611135 reported by Jay Faulkner
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Medium
Anup

Bug Description

A node that's rebooted due to a clean step having reboot_requested: True will not be transitioned to clean wait during the reboot process.

Expected flow:
CLEANING [ramdisk booted, runs clean step with reboot_requested] -> CLEAN WAIT [while performing the cleaning reboot]

Current behavior:
Node remains in CLEANING regardless of if a ramdisk is booting or not.

The state transitions should likely happen here: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L305

Changed in ironic:
assignee: nobody → Rebecca Finn (rebeccax-finn)
Sam Betts (sambetts)
summary: - Nodes do not enter 'clean wait' state during a cleaning reboot
+ [RFE] Nodes should enter 'clean wait' state during a cleaning reboot
tags: added: rfe
Changed in ironic:
status: New → Confirmed
importance: Undecided → Wishlist
summary: - [RFE] Nodes should enter 'clean wait' state during a cleaning reboot
+ Nodes should enter 'clean wait' state during a cleaning reboot
Changed in ironic:
importance: Wishlist → Medium
status: Confirmed → Triaged
tags: removed: rfe
Revision history for this message
Aline Bousquet (alineb) wrote :

Hi Rebecca,
Are you still working on this bug?
Thanks

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/392584

Changed in ironic:
assignee: Rebecca Finn (rebeccax-finn) → Aline Bousquet (alineb)
status: Triaged → In Progress
Revision history for this message
Anup (anup-d-navare) wrote :

Hi Aline,

Are you still working on this bug?
Thanks

Revision history for this message
Aline Bousquet (alineb) wrote :

Hi Anup,
I’m not currently working on this: feel free to work on it.
Thanks

Changed in ironic:
assignee: Aline Bousquet (alineb) → nobody
Dmitry Tantsur (divius)
Changed in ironic:
status: In Progress → Triaged
Anup (anup-d-navare)
Changed in ironic:
assignee: nobody → Anup (anup-d-navare)
Revision history for this message
Jay Faulkner (jason-oldos) wrote :

Looking at https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L487-L502 -- if any bug exists, it's that on cleaning_reboot the node will never re-enter CLEANING while the node is being cleaning_rebooted. I don't think that's a big deal, and I'm closing this bug as fixed.

I had, at some point, personally verified this bug existed; it appears to have been fixed during the implementation of Manual Cleaning. Marking as resolved.

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