yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #70706
[Bug 1746209] [NEW] Ironic virt driver node cache may be missing required fields
Public bug reported:
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 this should be changed to include
the required resource_class.
There are a number of other minor related issues picked up in that
discussion, which don't really deserve their own bugs:
* Filter the node list in _refresh_cache using _NODE_FIELDS.
* Improve unit tests to use representative filtered node objects.
* Remove _parse_node_instance_info and associated tests.
[1]
https://review.openstack.org/#/c/532288/9/nova/virt/ironic/driver.py@79
** Affects: nova
Importance: Undecided
Status: New
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1746209
Title:
Ironic virt driver node cache may be missing required fields
Status in OpenStack Compute (nova):
New
Bug description:
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 this
should be changed to include the required resource_class.
There are a number of other minor related issues picked up in that
discussion, which don't really deserve their own bugs:
* Filter the node list in _refresh_cache using _NODE_FIELDS.
* Improve unit tests to use representative filtered node objects.
* Remove _parse_node_instance_info and associated tests.
[1]
https://review.openstack.org/#/c/532288/9/nova/virt/ironic/driver.py@79
To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1746209/+subscriptions
Follow ups