baremetal API extension does not show important status information

[aeva black]

The baremetal API extension require administrator privileges to use it, but it does not expose the information that an administrator will need, such as the task state.

Here is what the database stores for a node with an active deployment.

ubuntu@foo:~$ mysql nova_bm -e "select * from bm_nodes where uuid='dda8bd00-0cf6-4211-8c8c-81dfafe8c1fb'\G"
*************************** 1. row ***************************
      created_at: 2013-05-15 19:58:21
      updated_at: 2013-05-24 09:45:17
      deleted_at: NULL
         deleted: 0
              id: 77
            cpus: 24
       memory_mb: 98304
        local_gb: 2048
      pm_address: xx.xx.xx.xx
         pm_user: xxxxxx
     pm_password: xxxxxxx
    service_host: foo.novalocal
prov_mac_address: NULL
   instance_uuid: 0c2690cd-c431-41f7-90f2-10ea678e137a
      task_state: active
   terminal_port: NULL
      image_path: /var/lib/nova/instances/instance-00000133/disk
 pxe_config_path: /tftpboot/0c2690cd-c431-41f7-90f2-10ea678e137a/config
      deploy_key: 8PXQS4ZPIMR6FVXH842YFFYJDTZHIPT9
         root_mb: 1536000
         swap_mb: 1
            uuid: dda8bd00-0cf6-4211-8c8c-81dfafe8c1fb
   instance_name: compute-test0.novacompute0.novacompute

Here is what the API exposes for the same node.

ubuntu@foo:~$ nova baremetal-node-show 77
+---------------+-----------------------------------------------------------------------------------------+
| Property | Value |
+---------------+-----------------------------------------------------------------------------------------+
| instance_uuid | 0c2690cd-c431-41f7-90f2-10ea678e137a |
| pm_address | xx.xx.xx.xx |
| interfaces | [{u'datapath_id': None, u'id': 77, u'port_no': None, u'address': u'aa:aa:aa:aa:aa:aa'}] |
| cpus | 24 |
| memory_mb | 98304 |
| service_host | foo.novalocal |
| local_gb | 2048 |
| id | 77 |
| pm_user | xxxxxxx |
| terminal_port | None |
+---------------+-----------------------------------------------------------------------------------------+

At a minimum, I believe the API should also expose the uuid, task_state, and updated fields. Including the pxe_config_path would help system admins troubleshoot some problems as well.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/42047

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/42047
Committed: http://github.com/openstack/nova/commit/e5a5a0f310ca0bce4902c11044d1bd5cbb329adb
Submitter: Jenkins
Branch: master

commit e5a5a0f310ca0bce4902c11044d1bd5cbb329adb
Author: Melanie Witt <email address hidden>
Date: Thu Oct 10 18:13:20 2013 +0000

    Expose additional status in baremetal API extension

    Add fields: uuid, task_state, updated_at, and pxe_config_path
    to aid system admins in troubleshooting issues.

    Fixes bug 1184449

    Change-Id: Ia4c03cb228b3efe602455bf05883ddf03b7f18da