[RFE] Expose API node introspection state

Bug #1665664 reported by milan k on 2017-02-17
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic Inspector
Fix Released
Wishlist
Seamanhanshan
Python client for Ironic Inspector
Wishlist
Unassigned

Bug Description

Inspector doesn't expose the node introspection state through the API.
This should be a read-only information enhancing current GET@/v1/introspection/<node_id> data structure w/ the state field:

    {
       error: <Error string>/null,
       finished: True/False,
       finished_at: <ISO8601 date>/null,
       started_at: <ISO8601 date>,
       *state: starting/waiting/error/processing/
              finished/reapplying/enrolling,*
       uuid: <node_uuid>
    }

Tags: rfe Edit Tag help
Revision history for this message
Seamanhanshan (yaojun) wrote :

it should be a issue, but we can get the introspection state via ironic API implicitly.

For example:
$ openstack baremetal node list
+--------------------------------------+--------+---------------+-------------+--------------------+-------------+
| UUID | Name | Instance UUID | Power State | Provisioning State | Maintenance |
+--------------------------------------+--------+---------------+-------------+--------------------+-------------+
| 399c52ce-e8cf-4202-a9c7-5bc4f95f2a24 | BM-132 | None | power on | inspecting | False |
+--------------------------------------+--------+---------------+-------------+--------------------+-------------+

$ openstack baremetal introspection status 399c52ce-e8cf-4202-a9c7-5bc4f95f2a24
+-------------+------------------------------------------------------------------------------------------------------------------+
| Field | Value |
+-------------+------------------------------------------------------------------------------------------------------------------+
| error | None |
| finished | False |
| finished_at | None |
| links | [{u'href': u'http://10.254.10.200:5050/v1/introspection/399c52ce-e8cf-4202-a9c7-5bc4f95f2a24', u'rel': u'self'}] |
| started_at | 2017-02-21T01:11:20 |
| uuid | 399c52ce-e8cf-4202-a9c7-5bc4f95f2a24 |
+-------------+------------------------------------------------------------------------------------------------------------------+

Changed in ironic-inspector:
assignee: nobody → Seamanhanshan (yaojun)
status: New → In Progress
Anton Arefiev (aarefiev) on 2017-02-21
tags: added: rfe
Changed in ironic-inspector:
importance: Undecided → Wishlist
Revision history for this message
Anton Arefiev (aarefiev) wrote :

Seems it will require some changes in client, at least docs update

Changed in python-ironic-inspector-client:
status: New → Triaged
importance: Undecided → Wishlist
Changed in ironic-inspector:
assignee: Seamanhanshan (yaojun) → milan k (vetrisko)
Changed in ironic-inspector:
assignee: milan k (vetrisko) → Seamanhanshan (yaojun)
Revision history for this message
Ruby Loo (rloo) wrote :

patch in ironic-inspector: https://review.openstack.org/#/c/436415/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic-inspector (master)

Reviewed: https://review.openstack.org/436415
Committed: https://git.openstack.org/cgit/openstack/ironic-inspector/commit/?id=fc2e029fa6318384f840bdc8bc54894396308299
Submitter: Jenkins
Branch: master

commit fc2e029fa6318384f840bdc8bc54894396308299
Author: yaojun <email address hidden>
Date: Tue Feb 21 19:04:00 2017 +0800

    Adds node state to the API response data

    This adds the node state when the GET /v1/introspection/<node uuid or
    name> API is performed.

    Change-Id: I81c6834933f789cb644a854313aacaf49a4856a7
    Closes-Bug: #1665664

Changed in ironic-inspector:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ironic-inspector 6.0.0

This issue was fixed in the openstack/ironic-inspector 6.0.0 release.

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

Other bug subscribers