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

Merge "Fix DEFAULT_SCHEMA to validate host_aggreates"

Browse Source
This commit is contained in:
Jenkins
2017-09-05 08:45:35 +00:00
committed by Gerrit Code Review
parent fa7afc89ab 4c3c84dee9
commit 4640d88adf
2 changed files with 44 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

50
watcher/decision_engine/scope/default.py
View File

@@ -36,12 +36,10 @@ class DefaultScope(base.BaseScope):
"host_aggregates": {
"type": "array",
"items": {
"type": "object",
"properties": {
"anyOf": [
{"type": ["string", "number"]}
]
},
"anyOf": [
{"$ref": "#/host_aggregates/id"},
{"$ref": "#/host_aggregates/name"},
]
}
},
"availability_zones": {
@@ -69,7 +67,8 @@ class DefaultScope(base.BaseScope):
"uuid": {
"type": "string"
}
}
},
"additionalProperties": False
}
},
"compute_nodes": {
@@ -80,18 +79,17 @@ class DefaultScope(base.BaseScope):
"name": {
"type": "string"
}
}
},
"additionalProperties": False
}
},
"host_aggregates": {
"type": "array",
"items": {
"type": "object",
"properties": {
"anyOf": [
{"type": ["string", "number"]}
]
},
"anyOf": [
{"$ref": "#/host_aggregates/id"},
{"$ref": "#/host_aggregates/name"},
]
}
},
"instance_metadata": {
@@ -106,7 +104,29 @@ class DefaultScope(base.BaseScope):
}
},
"additionalProperties": False
}
},
"host_aggregates": {
"id": {
"properties": {
"id": {
"oneOf": [
{"type": "integer"},
{"enum": ["*"]}
]
}
},
"additionalProperties": False
},
"name": {
"properties": {
"name": {
"type": "string"
}
},
"additionalProperties": False
}
},
"additionalProperties": False
}
def __init__(self, scope, config, osc=None):

10
watcher/tests/api/v1/test_audit_templates.py
View File

@@ -511,7 +511,7 @@ class TestPost(FunctionalTestWithSetup):
response.json['created_at']).replace(tzinfo=None)
self.assertEqual(test_time, return_created_at)
def test_create_audit_template_vlidation_with_aggregates(self):
def test_create_audit_template_validation_with_aggregates(self):
scope = [{'host_aggregates': [{'id': '*'}]},
{'availability_zones': [{'name': 'AZ1'},
{'name': 'AZ2'}]},
@@ -532,6 +532,14 @@ class TestPost(FunctionalTestWithSetup):
"be included and excluded together"):
self.post_json('/audit_templates', audit_template_dict)
scope = [{'host_aggregates': [{'id1': '*'}]}]
audit_template_dict = post_get_test_audit_template(
goal=self.fake_goal1.uuid,
strategy=self.fake_strategy1.uuid, scope=scope)
response = self.post_json('/audit_templates',
audit_template_dict, expect_errors=True)
self.assertEqual(500, response.status_int)
def test_create_audit_template_does_autogenerate_id(self):
audit_template_dict = post_get_test_audit_template(
goal=self.fake_goal1.uuid, strategy=None)