[RFE] Support tags for nodes in Ironic
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| Ironic |
In Progress
|
Wishlist
|
Ruby Loo |
Bug Description
Ironic will support "tags" field stored in metadata for every node. The same tags will be stored in extra metadata for Nova flavor, so that every flavor can be bound to some group of nodes. To achieve this:
* Ironic API should allow listing nodes by tag
* Scheduler should allow deploying on nodes with tag, fetched from flavor (XXX does it belong in Ironic Nova driver or in Nova scheduler itself?)
Vladyslav Drok (vdrok) wrote : | #1 |
Changed in ironic: | |
status: | New → Confirmed |
importance: | Undecided → Wishlist |
tags: | added: rfe |
Changed in ironic: | |
assignee: | nobody → Zhenguo Niu (niu-zglinux) |
status: | Confirmed → In Progress |
Ruby Loo (rloo) wrote : | #2 |
Specification is available at http://
Copying this from old BP (https:/
Gerrit topic: https:/
Addressed by: https:/
Add nodes tagging support
Gerrit topic: https:/
Addressed by: https:/
Add db migration script for creating tags table
Addressed by: https:/
Add db migration for tags table
Addressed by: https:/
Add db api layer for CRUD operations on node tags
Addressed by: https:/
Add Tag, TagList objects and a new tags field to Node
Addressed by: https:/
Added node tags in ironic-api
Addressed by: https:/
Joined 'tags' column while getting node
Fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #4 |
Fix proposed to branch: master
Review: https:/
Change abandoned by Zhenguo Niu (<email address hidden>) on branch: master
Review: https:/
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit c1a8606095d4021
Author: Zhenguo Niu <email address hidden>
Date: Wed Oct 21 18:04:35 2015 +0800
Add db api layer for CRUD operations on node tags
- set/unset node tags
- get node tags
- add/delete single tag
- check whether node tag exists
- Delete all tags attached to the node when it's destroyed
This will not support creating node with tags, just ignore
the specified tags.
Change-Id: Ibe83a726d904fd
Partial-bug: #1526266
tags: |
added: rfe-approved removed: rfe |
OpenStack Infra (hudson-openstack) wrote : | #7 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 19ef56cf00a7887
Author: Zhenguo Niu <email address hidden>
Date: Fri Jan 29 21:28:50 2016 +0800
Joined 'tags' column while getting node
Partial-bug: #1526266
Change-Id: Iab795248d7b5b1
Changed in ironic: | |
assignee: | Zhenguo Niu (niu-zglinux) → Vladyslav Drok (vdrok) |
Changed in ironic: | |
status: | In Progress → Opinion |
Ruby Loo (rloo) wrote : | #8 |
@Tao, what's the reason for setting the status to 'Opinion'? There are code patches for this.
Changed in ironic: | |
status: | Opinion → In Progress |
Fix proposed to branch: master
Review: https:/
Changed in ironic: | |
assignee: | Vladyslav Drok (vdrok) → Ruby Loo (rloo) |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: master
commit afc432af4d2342a
Author: Ruby Loo <email address hidden>
Date: Mon Jan 15 10:11:50 2018 -0500
node_
To make tags consistent with traits, change db.api.
to raise a NodeNotFound exception if the node doesn't exist.
This is a follow up to bbff319f0226af1
Change-Id: Id0ddf448d97b5a
Partial-
Change abandoned by Zhenguo Niu (<email address hidden>) on branch: master
Review: https:/
Change abandoned by Zhenguo Niu (<email address hidden>) on branch: master
Review: https:/
Change abandoned by Zhenguo Niu (<email address hidden>) on branch: master
Review: https:/
Spec is merged - https:/ /review. openstack. org/#/c/ 183971/, the first patch adding a migration too - https:/ /review. openstack. org/#/c/ 192576/.