[RFE] Add resources CRUD notifications to Ironic

Bug #1606520 reported by Yuriy Zveryanskyy on 2016-07-26
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Wishlist
Yuriy Zveryanskyy

Bug Description

For efficient resource indexation Ironic needs CRUD (create, update or delete) notifications support. The main use case is Searchlight https://wiki.openstack.org/wiki/Searchlight

Spec: https://review.openstack.org/#/c/347242

tags: added: rfe

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

Changed in ironic:
assignee: nobody → Yuriy Zveryanskyy (yzveryanskyy)
status: New → In Progress
description: updated

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

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

Dmitry Tantsur (divius) on 2016-08-22
Changed in ironic:
importance: Undecided → Wishlist

Reviewed: https://review.openstack.org/347403
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=e497a32ee09253cde2942a55f6c7c89dad7be831
Submitter: Jenkins
Branch: master

commit e497a32ee09253cde2942a55f6c7c89dad7be831
Author: Yuriy Zveryanskyy <email address hidden>
Date: Tue Jul 26 16:43:03 2016 +0300

    Replace DB API call to object's method in iLO drivers

    Ironic code should use versioned objects for CRUD operations if
    possible. This patch modifies iLO driver inspect module and
    changes ports creation from DB API call to Port.create()
    method.

    Related-Bug: #1606520
    Change-Id: I7b3ec087663ce3a7cf30c0708830f8b1be616ee8

Change abandoned by Yuriy Zveryanskyy (<email address hidden>) on branch: master
Review: https://review.openstack.org/357799
Reason: merged with previous patch

tags: added: needs-spec

spec is currently under review, so removed needs-spec tag - please feel free to add it back if spec should merge before removing that tag

tags: removed: needs-spec

Reviewed: https://review.openstack.org/347242
Committed: https://git.openstack.org/cgit/openstack/ironic-specs/commit/?id=7ae6184c314c6f90e590abd5ef5c3608e13fc728
Submitter: Jenkins
Branch: master

commit 7ae6184c314c6f90e590abd5ef5c3608e13fc728
Author: Yuriy Zveryanskyy <email address hidden>
Date: Tue Jul 26 13:23:49 2016 +0300

    Add CRUD and node states notifications to ironic

    This spec proposes to add resources create, update, delete
    notifications, node provision state, maintenance mode and
    console notifications.

    Partial-Bug: #1606520
    Co-Authored-By: Vladyslav Drok <email address hidden>
    Change-Id: I94aa182add471efbbe703f7b785040222f1f5c67

Reviewed: https://review.openstack.org/397792
Committed: https://git.openstack.org/cgit/openstack/ironic-specs/commit/?id=27d9e8fd88defa257d62868c746b76598bd86788
Submitter: Jenkins
Branch: master

commit 27d9e8fd88defa257d62868c746b76598bd86788
Author: Yuriy Zveryanskyy <email address hidden>
Date: Tue Nov 15 17:08:42 2016 +0200

    Update CRUD and node states notification spec

    Add new "console_restore" action for node console notifications.
    This action will be used when console for node is already enabled
    in the DB but restart via driver is required due to dead or
    restarted conductor process.

    Change-Id: I80eddf508f61646f84d0e74fe3578b8d26cde151
    Partial-Bug: #1606520

Ruby Loo (rloo) wrote :

The spec (https://review.openstack.org/347242) was approved and updated! Changed tag to 'rfe-approved'.

tags: added: rfe-approved
removed: rfe

Reviewed: https://review.openstack.org/348437
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=50a2f50ab86842fe7f56db08ae841b46263ccca8
Submitter: Jenkins
Branch: master

commit 50a2f50ab86842fe7f56db08ae841b46263ccca8
Author: Yuriy Zveryanskyy <email address hidden>
Date: Thu Jul 28 18:25:59 2016 +0300

    Add node provision state change notification

    This patch adds node provision state changes notifications,
    event types are:
    "baremetal.node.provision_set.{start, end, success, error}"
    Developer documentation updated.

    Change-Id: I6a5a2da6541596b7d1b7c1bf2795cfc6f86191fb
    Partial-Bug: #1606520

Change abandoned by Yuriy Zveryanskyy (<email address hidden>) on branch: master
Review: https://review.openstack.org/379469
Reason: not needed currently

Reviewed: https://review.openstack.org/356541
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=499ef55dd1c484b257f7b892978ee0a291b09f56
Submitter: Jenkins
Branch: master

commit 499ef55dd1c484b257f7b892978ee0a291b09f56
Author: Yuriy Zveryanskyy <email address hidden>
Date: Fri Nov 4 11:25:19 2016 +0200

    Add ironic resources CRUD notifications

    This patch adds notifications for create, update or delete ironic
    resources (node, port and chassis). Event types general form are:
    baremetal.<resource>.{create, update, delete}.{start,end,error}.
    Developer documentation updated.

    Partial-Bug: #1606520
    Change-Id: I95c64d9aa806ff2d7e7dae54ced169c98282c67d

Reviewed: https://review.openstack.org/396239
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=876db5a61314740dc949c2641131f230f9be6c98
Submitter: Jenkins
Branch: master

commit 876db5a61314740dc949c2641131f230f9be6c98
Author: Yuriy Zveryanskyy <email address hidden>
Date: Thu Nov 10 16:29:27 2016 +0200

    Add node maintenance notifications

    This patch adds node maintenance notifications, event types are:
    "baremetal.node.maintenance_set.{start, end, error}".
    Developer documentation updated.

    Change-Id: I9105821ed0a4db614ea3e1c73ad563b82b1c6082
    Partial-Bug: #1606520

Reviewed: https://review.openstack.org/397812
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=294f974fe70982624e626ed221715ae13194ef0d
Submitter: Jenkins
Branch: master

commit 294f974fe70982624e626ed221715ae13194ef0d
Author: Yuriy Zveryanskyy <email address hidden>
Date: Tue Nov 15 18:17:55 2016 +0200

    Add node console notifications

    This patch adds node console notifications, event types are:
    "baremetal.node.console_{set, restore}.{start, end, error}".
    Developer documentation updated.

    Change-Id: I3b3ac74607fd6e218fdf0ea3ff30964e527db399
    Partial-Bug: #1606520

Changed in ironic:
status: In Progress → Fix Committed
Changed in ironic:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers