Arnike/watcher
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Fix 'unable to exclude instance'"

Browse Source
This commit is contained in:
Zuul
2017-12-14 05:38:34 +00:00
committed by Gerrit Code Review
parent 72e3d5c7f9 be56441e55
commit a94e35b60e
2 changed files with 11 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
watcher/decision_engine/scope/compute.py
View File

@@ -154,12 +154,16 @@ class ComputeScope(base.BaseScope):
nodes_to_remove = set() nodes_to_remove = set()
instances_to_exclude = [] instances_to_exclude = []
instance_metadata = [] instance_metadata = []
compute_scope = []
model_hosts = list(cluster_model.get_all_compute_nodes().keys()) model_hosts = list(cluster_model.get_all_compute_nodes().keys())
if not self.scope: for scope in self.scope:
compute_scope = scope.get('compute')
if not compute_scope:
return cluster_model return cluster_model
for rule in self.scope: for rule in compute_scope:
if 'host_aggregates' in rule: if 'host_aggregates' in rule:
self._collect_aggregates(rule['host_aggregates'], self._collect_aggregates(rule['host_aggregates'],
allowed_nodes) allowed_nodes)

9
watcher/tests/decision_engine/scope/fake_scopes.py
View File

@@ -14,10 +14,11 @@
# See the License for the specific language governing permissions and # See the License for the specific language governing permissions and
# limitations under the License. # limitations under the License.
fake_scope_1 = [{'availability_zones': [{'name': 'AZ1'}]}, fake_scope_1 = [{'compute': [{'availability_zones': [{'name': 'AZ1'}]},
{'exclude': {'exclude': [
[{'instances': {'instances': [
[{'uuid': 'INSTANCE_6'}]}] {'uuid': 'INSTANCE_6'}]},
]}]
} }
] ]