Cleanups for pci stats in preparation for RT using ComputeNode
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
openstack-api-site |
Fix Released
|
Low
|
Diane Fleming |
Bug Description
https:/
commit f1a0d852c8754fc
Author: Paul Murray <email address hidden>
Date: Thu Mar 5 18:48:03 2015 +0000
Cleanups for pci stats in preparation for RT using ComputeNode
While converting the resource tracker to use the ComputeNode
object it became apparent that there was a trivial bug in the
way tags were assigned in PciDevicePool.
the format for pci stats in the examples is wrong.
The bug was that a dict was assigned to the tags field and then
items were added to it as follows:
pool.tags = {}
pool.
The setter for objects is over ridden to perform type checking
and coercion for fields. In the above case the tags field is
a dict of strings, so the coercion converts all values in
the dict to be strings (if it can). In the above snippet
the values of pool_dict are not type checked or coerced because
they are added directly to the dict. The correct way to do this
is (it just so happens that pool_dict again here):
pool.tags = pool_dict
The format of the pci stats does not include an extra_info
field as was originally planned by the author. Instead tags
that would have been in extra_info are included as additional
fields. The phys_function tag was intended to have particular
meaning but has now been dropped.
The api samples and related tests for hypervisors-
and hypervisors-
samples and tests are incorrect. This patch corrects them.
The change in the samples is not an API change, only
a correction, so there is no change to the API version. However,
the api samples are included in documentation so this patch is
marked with with a document impact.
DocImpact
Change-Id: I8730f18d660777
affects: | openstack-manuals → openstack-api-site |
Changed in openstack-api-site: | |
status: | New → In Progress |
assignee: | nobody → Diane Fleming (diane-fleming) |
importance: | Undecided → Low |
milestone: | none → liberty |
Fix proposed to branch: master /review. openstack. org/189163
Review: https:/