From 9d73dccfb8c6b4efd07daa331502c63342fae107 Mon Sep 17 00:00:00 2001 From: Eric Fried Date: Fri, 16 Feb 2018 18:18:32 -0600 Subject: [PATCH] Only pull associated *sharing* providers It was discussed and decided [1] that we only want to be pulling down, caching, and passing to update_provider_tree providers associated via aggregate with the compute node's provider tree if they are sharing providers. Otherwise we'll get e.g. all the *other* compute nodes which are also associated with a sharing provider. [1] https://review.openstack.org/#/c/540111/4/specs/rocky/approved/update-provider-tree.rst@48 Change-Id: Iab366da7623e5e31b8416e89fee7d418f7bf9b30 Closes-Bug: #1750084 --- .../functional/api/openstack/placement/test_report_client.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/nova/tests/functional/api/openstack/placement/test_report_client.py b/nova/tests/functional/api/openstack/placement/test_report_client.py index 7acea0c57..8fd2835a6 100644 --- a/nova/tests/functional/api/openstack/placement/test_report_client.py +++ b/nova/tests/functional/api/openstack/placement/test_report_client.py @@ -132,8 +132,8 @@ class SchedulerReportClientTests(test.TestCase): # We should also have empty sets of aggregate and trait # associations self.assertEqual( - [], self.client._get_providers_in_aggregates(self.context, - [uuids.agg])) + [], self.client._get_sharing_providers(self.context, + [uuids.agg])) self.assertFalse( self.client._provider_tree.have_aggregates_changed( self.compute_uuid, []))