commit 6b11eb794f5881c9c8ed284b2edfa4c31ed62b2b
Author: Mark Goddard <email address hidden>
Date: Wed Jan 31 11:11:32 2018 +0000
Add resource_class to fields in ironic node cache
Per the discussion in [1], the ironic nodes added to the node cache in
the ironic virt driver may be missing the required field resource_class,
as this field is not in _NODE_FIELDS. In practice, this is typically
not an issue (possibly never), as the normal code path uses a
detailed list to sync all ironic nodes, which contain all fields
(including resource_class). However, some code paths use a single
node query with the fields limited to _NODE_FIELDS, so could result in a
node in the cache without a resource_class.
Reviewed: https:/ /review. openstack. org/546085 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=6b11eb794f5 881c9c8ed284b2e dfa4c31ed62b2b
Committed: https:/
Submitter: Zuul
Branch: stable/queens
commit 6b11eb794f5881c 9c8ed284b2edfa4 c31ed62b2b
Author: Mark Goddard <email address hidden>
Date: Wed Jan 31 11:11:32 2018 +0000
Add resource_class to fields in ironic node cache
Per the discussion in [1], the ironic nodes added to the node cache in
the ironic virt driver may be missing the required field resource_class,
as this field is not in _NODE_FIELDS. In practice, this is typically
not an issue (possibly never), as the normal code path uses a
detailed list to sync all ironic nodes, which contain all fields
(including resource_class). However, some code paths use a single
node query with the fields limited to _NODE_FIELDS, so could result in a
node in the cache without a resource_class.
This change adds resource_class to _NODE_FIELDS.
[1] /review. openstack. org/#/c/ 532288/ 9/nova/ virt/ironic/ driver. py@79
https:/
Change-Id: Id84b4a47d05532 d341a9b6ca2de7e 9e66e1930da 30c31ae94070282 566a6cc568)
Closes-Bug: #1746209
(cherry picked from commit 5895566a428be4c