Drivers which do not implement the rescue interface should not report it as supported
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ironic |
Fix Released
|
High
|
aeva black |
Bug Description
All the drivers which use the pxe.PXEDeploy deploy interface also mapped this class to their rescue interface. This should not have been done. Deploy and Rescue interfaces are separate classes.
Furthermore, the pxe.PXERescue class was merely a stub which,, if used, would have caused drivers to incorrectly report that that interface was supported (when it wasn't). It is an optional interface, and does not need a stub class.
This is what the current output for a configured node looks like. This is misleading, because rescue() is not supported by any PXE-based driver today.
$ ironic node-validate 9f4fce58-
+------
| Interface | Result | Reason |
+------
| console | None | not supported |
| deploy | True | |
| power | True | |
| rescue | True | |
+------
Changed in ironic: | |
milestone: | none → icehouse-rc1 |
Changed in ironic: | |
importance: | Undecided → High |
Changed in ironic: | |
status: | Fix Committed → Fix Released |
Changed in ironic: | |
milestone: | icehouse-rc1 → 2014.1 |
Fix proposed to branch: master /review. openstack. org/81336
Review: https:/