With change I6827137f35c0cb4f9fc4c6f753d9a035326ed01b in
Ocata, we changed the ComputeManager to manage a single
ResourceTracker and that single ResourceTracker will
manage multiple compute nodes. The only time a single
nova-compute service hosts multiple compute nodes is for
ironic where there is a compute node per instance. The
problem is the ResourceTracker.stats variable, unlike the
ResourceTracker.compute_nodes variable, is not node-specific
so it's possible for node stats to leak across nodes based
on how the stats are used (and copied).
This change adds a functional recreate test to show the issue
before it's fixed. The fixture setup had to be tweaked a
bit to avoid modifying class variables by reference between
test cases.
NOTE(mriedem): Conflicts are due to not having change
I433700e833f97c0fec946dafc2cdda9d49e1100b or change
I075785abcd4f4a8e180959daeadf215b9cd175c8 in Queens.
Change-Id: Icc5f615baa1042347ec1699eb84ba0670445b995
Related-Bug: #1784705
(cherry picked from commit fc05626d43571733da0803df0fd9a7c69766b8fd)
Reviewed: https:/ /review. openstack. org/587921 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=cc8167a1914 a70ea7b33685628 5df2571a754ca0
Committed: https:/
Submitter: Zuul
Branch: stable/queens
commit cc8167a1914a70e a7b336856285df2 571a754ca0
Author: Matt Riedemann <email address hidden>
Date: Tue Jul 31 16:00:28 2018 -0400
Add recreate test for RT.stats bug 1784705
With change I6827137f35c0cb 4f9fc4c6f753d9a 035326ed01b in .stats variable, unlike the cker.compute_ nodes variable, is not node-specific
Ocata, we changed the ComputeManager to manage a single
ResourceTracker and that single ResourceTracker will
manage multiple compute nodes. The only time a single
nova-compute service hosts multiple compute nodes is for
ironic where there is a compute node per instance. The
problem is the ResourceTracker
ResourceTra
so it's possible for node stats to leak across nodes based
on how the stats are used (and copied).
This change adds a functional recreate test to show the issue
before it's fixed. The fixture setup had to be tweaked a
bit to avoid modifying class variables by reference between
test cases.
Conflicts:
nova/ tests/functiona l/compute/ test_resource_ tracker. py
NOTE(mriedem): Conflicts are due to not having change f97c0fec946dafc 2cdda9d49e1100b or change 4f4a8e180959dae adf215b9cd175c8 in Queens.
I433700e833
I075785abcd
Change-Id: Icc5f615baa1042 347ec1699eb84ba 0670445b995 3da0803df0fd9a7 c69766b8fd)
Related-Bug: #1784705
(cherry picked from commit fc05626d4357173