When changing any field in driver_info, the power_state gets set to a string representation of a python object. To test this, I have created a node with valid information, ensured it had a NULL power state (manual db update to do this...) and then did a simple update via the API:
$ ironic node-show fbe3835c-0dfc-4ec0-a693-44a7b7dd4439
+------------------------+------------------------------------------------------------------------+
| Property | Value |
+------------------------+------------------------------------------------------------------------+
| instance_uuid | None |
| target_power_state | None |
| uuid | fbe3835c-0dfc-4ec0-a693-44a7b7dd4439 |
| driver_info | {u'ssh_username': u'deva', u'ssh_virt_type': u'virsh', u'foo': u'bar', |
| | u'ssh_address': u'192.168.122.1', u'ssh_key_filename': u'/opt/stack |
| | /boot-stack/virtual-power-key', u'foo2': u'bar2'} |
| target_provision_state | None |
| created_at | |
| extra | {} |
| driver | fake_ssh |
| updated_at | |
| power_state | None |
| provision_state | None |
| reservation | |
| properties | {} |
+------------------------+------------------------------------------------------------------------+
$ ironic node-update fbe3835c-0dfc-4ec0-a693-44a7b7dd4439 remove driver_info/foo2
+------------------------+------------------------------------------------------------------------+
| Property | Value |
+------------------------+------------------------------------------------------------------------+
| instance_uuid | None |
| target_power_state | None |
| uuid | fbe3835c-0dfc-4ec0-a693-44a7b7dd4439 |
| driver_info | {u'ssh_username': u'deva', u'ssh_virt_type': u'virsh', u'foo': u'bar', |
| | u'ssh_address': u'192.168.122.1', u'ssh_key_filename': u'/opt/stack |
| | /boot-stack/virtual-power-key'} |
| target_provision_state | None |
| created_at | |
| extra | {} |
| driver | fake_ssh |
| updated_at | |
| power_state | <bound method SSHPower.get_power_state of |
| | <ironic.drivers.modules.ssh.SSHPower object at 0x8cc89ac>> |
| provision_state | None |
| reservation | |
| properties | {} |
+------------------------+------------------------------------------------------------------------+
Fix proposed to branch: master
Review: https:/