[RFE] API for dynamic lookup attributes

Bug #1525231 reported by Sam Betts
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic Inspector
Fix Released
Wishlist
milan k

Bug Description

Currently we look up nodes using only BMC and MAC addresses. We should allow users to add more to this list. Initial proposal was to create a new plugin interface, but it may be an overkill for now.

The following JSON structure will be used for attributes:

 {
   "field": "JSON path to a field in data",
   "node_field": "path to field in ironic database",
   "op": "operation"
 }

Here "op" is the same as in rules API.
New API collection /v1/lookup/attributes will be created to store lookup attributes.

Tags: rfe
Sam Betts (sambetts)
Changed in ironic-inspector:
importance: Undecided → Wishlist
status: New → Confirmed
status: Confirmed → Triaged
status: Triaged → Won't Fix
status: Won't Fix → Confirmed
Changed in ironic-inspector:
assignee: nobody → milan k (vetrisko)
status: Confirmed → In Progress
Changed in ironic-inspector:
assignee: milan k (vetrisko) → Anton Arefiev (aarefiev)
Changed in ironic-inspector:
assignee: Anton Arefiev (aarefiev) → milan k (vetrisko)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic-inspector (master)

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

commit 0ce5cdb7c8344e981a919aab136c4c2eba79d4a2
Author: dparalen <email address hidden>
Date: Mon Jan 16 18:55:08 2017 +0100

    Find a node by multiple attributes

    This patch modifies current attributes matching from a single
    name-value->node hit to a best-match score.

    Also using an UUID as the attributes table primary key to allow exposing
    attributes in API later (bug 1525231).

    Change-Id: I205e31652b21b9e030b9530149e533b29c52a387
    Closes-Bug: 1651719
    Partial-Bug: 1525231

Revision history for this message
Iury Gregory Melo Ferreira (iurygregory) wrote :

Fix Released - the patches have been merged (a while ago)

Changed in ironic-inspector:
status: In Progress → 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.