a-c-m
/
community.general
mirror of https://github.com/ansible-collections/community.general.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

nso_config work around ordering issues (#36774) 

Include dependencies when sorting entries to avoid issues with certain
versions of NSO.
pull/4420/head
Claes Nästén 2018-02-27 16:22:57 +01:00 committed by John R Barker
parent 52627f3fa0
commit e75989ec88
2 changed files with 162 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

73
lib/ansible/module_utils/network/nso/nso.py
View File

@ -273,13 +273,17 @@ class JsonRpc(object):
class ValueBuilder(object): class ValueBuilder(object):
class Value(object): PATH_RE = re.compile('{[^}]*}')
__slots__ = ['path', 'state', 'value']
def __init__(self, path, state, value): class Value(object):
__slots__ = ['path', 'tag_path', 'state', 'value', 'deps']
def __init__(self, path, state, value, deps):
self.path = path self.path = path
self.tag_path = ValueBuilder.PATH_RE.sub('', path)
self.state = state self.state = state
self.value = value self.value = value
self.deps = deps
def __lt__(self, rhs): def __lt__(self, rhs):
l_len = len(self.path.split('/')) l_len = len(self.path.split('/'))
@ -298,7 +302,6 @@ class ValueBuilder(object):
self._module_prefix_map_cache = None self._module_prefix_map_cache = None
self._values = [] self._values = []
self._values_dirty = False self._values_dirty = False
self._path_re = re.compile('{[^}]*}')
def build(self, parent, maybe_qname, value, schema=None): def build(self, parent, maybe_qname, value, schema=None):
qname, name = self._get_prefix_name(maybe_qname) qname, name = self._get_prefix_name(maybe_qname)
@ -313,17 +316,18 @@ class ValueBuilder(object):
if self._is_leaf_list(schema) and is_version(self._client, [(4, 5)]): if self._is_leaf_list(schema) and is_version(self._client, [(4, 5)]):
self._build_leaf_list(path, schema, value) self._build_leaf_list(path, schema, value)
elif self._is_leaf(schema): elif self._is_leaf(schema):
deps = schema.get('deps', [])
if self._is_empty_leaf(schema): if self._is_empty_leaf(schema):
exists = self._client.exists(path) exists = self._client.exists(path)
if exists and value != [None]: if exists and value != [None]:
self._add_value(path, State.ABSENT, None) self._add_value(path, State.ABSENT, None, deps)
elif not exists and value == [None]: elif not exists and value == [None]:
self._add_value(path, State.PRESENT, None) self._add_value(path, State.PRESENT, None, deps)
else: else:
if maybe_qname is None: if maybe_qname is None:
value_type = self._get_type(path) value_type = self._get_type(path)
else: else:
value_type = self._get_child_type(parent, maybe_qname) value_type = self._get_child_type(parent, qname)
if 'identityref' in value_type: if 'identityref' in value_type:
if isinstance(value, list): if isinstance(value, list):
@ -331,7 +335,7 @@ class ValueBuilder(object):
in [self._get_prefix_name(v) for v in value]] in [self._get_prefix_name(v) for v in value]]
else: else:
value, t_value = self._get_prefix_name(value) value, t_value = self._get_prefix_name(value)
self._add_value(path, State.SET, value) self._add_value(path, State.SET, value, deps)
elif isinstance(value, dict): elif isinstance(value, dict):
self._build_dict(path, schema, value) self._build_dict(path, schema, value)
elif isinstance(value, list): elif isinstance(value, list):
@ -344,11 +348,44 @@ class ValueBuilder(object):
@property @property
def values(self): def values(self):
if self._values_dirty: if self._values_dirty:
self._values.sort() self._values = self._sort_values(sorted(self._values))
self._values_dirty = False self._values_dirty = False
return self._values return self._values
def _sort_values(self, values):
class N(object):
def __init__(self, v):
self.tmp_mark = False
self.mark = False
self.v = v
sorted_values = []
nodes = [N(v) for v in values]
def visit(n):
if n.tmp_mark:
return False
if not n.mark:
n.tmp_mark = True
for m in nodes:
if m.v.tag_path in n.v.deps:
if not visit(m):
return False
n.tmp_mark = False
n.mark = True
sorted_values.insert(0, n.v)
return True
while any((not n.mark for n in nodes)):
n = next((node for node in nodes if not node.mark))
visit(n)
return sorted_values[::-1]
def _build_dict(self, path, schema, value): def _build_dict(self, path, schema, value):
keys = schema.get('key', []) keys = schema.get('key', [])
for dict_key, dict_value in value.items(): for dict_key, dict_value in value.items():
@ -360,18 +397,20 @@ class ValueBuilder(object):
self.build(path, dict_key, dict_value, child_schema) self.build(path, dict_key, dict_value, child_schema)
def _build_leaf_list(self, path, schema, value): def _build_leaf_list(self, path, schema, value):
deps = schema.get('deps', [])
entry_type = self._get_type(path, schema) entry_type = self._get_type(path, schema)
# remove leaf list if treated as a list and then re-create the # remove leaf list if treated as a list and then re-create the
# expected list entries. # expected list entries.
self._add_value(path, State.ABSENT, None) self._add_value(path, State.ABSENT, None, deps)
for entry in value: for entry in value:
if 'identityref' in entry_type: if 'identityref' in entry_type:
entry, t_entry = self._get_prefix_name(entry) entry, t_entry = self._get_prefix_name(entry)
entry_path = '{0}{{{1}}}'.format(path, entry) entry_path = '{0}{{{1}}}'.format(path, entry)
self._add_value(entry_path, State.PRESENT, None) self._add_value(entry_path, State.PRESENT, None, deps)
def _build_list(self, path, schema, value): def _build_list(self, path, schema, value):
deps = schema.get('deps', [])
for entry in value: for entry in value:
entry_key = self._build_key(path, entry, schema['key']) entry_key = self._build_key(path, entry, schema['key'])
entry_path = '{0}{{{1}}}'.format(path, entry_key) entry_path = '{0}{{{1}}}'.format(path, entry_key)
@ -380,12 +419,12 @@ class ValueBuilder(object):
if entry_state == 'absent': if entry_state == 'absent':
if entry_exists: if entry_exists:
self._add_value(entry_path, State.ABSENT, None) self._add_value(entry_path, State.ABSENT, None, deps)
else: else:
if not entry_exists: if not entry_exists:
self._add_value(entry_path, State.PRESENT, None) self._add_value(entry_path, State.PRESENT, None, deps)
if entry_state in State.SYNC_STATES: if entry_state in State.SYNC_STATES:
self._add_value(entry_path, entry_state, None) self._add_value(entry_path, entry_state, None, deps)
self.build(entry_path, None, entry) self.build(entry_path, None, entry)
@ -439,8 +478,8 @@ class ValueBuilder(object):
'no child in {0} with name {1}. children {2}'.format( 'no child in {0} with name {1}. children {2}'.format(
path, qname, ','.join((c.get('qname', c.get('name', None)) for c in schema['children'])))) path, qname, ','.join((c.get('qname', c.get('name', None)) for c in schema['children']))))
def _add_value(self, path, state, value): def _add_value(self, path, state, value, deps):
self._values.append(ValueBuilder.Value(path, state, value)) self._values.append(ValueBuilder.Value(path, state, value, deps))
self._values_dirty = True self._values_dirty = True
def _get_prefix_name(self, qname): def _get_prefix_name(self, qname):
@ -496,7 +535,7 @@ class ValueBuilder(object):
return None return None
def _ensure_schema_cached(self, path): def _ensure_schema_cached(self, path):
path = self._path_re.sub('', path) path = ValueBuilder.PATH_RE.sub('', path)
if path not in self._schema_cache: if path not in self._schema_cache:
schema = self._client.get_schema(path=path, levels=1) schema = self._client.get_schema(path=path, levels=1)
self._schema_cache[path] = schema self._schema_cache[path] = schema

106
test/units/module_utils/network/nso/test_nso.py
View File

@ -269,6 +269,48 @@ SCHEMA_DATA = {
} }
} }
} }
''',
'/test:deps': '''
{
"meta": {
},
"data": {
"kind":"container",
"name":"deps",
"qname":"test:deps",
"children": [
{
"kind": "leaf",
"type": {
"primitive": true,
"name": "string"
},
"name": "a",
"qname": "test:a",
"deps": ["/test:deps/c"]
},
{
"kind": "leaf",
"type": {
"primitive": true,
"name": "string"
},
"name": "b",
"qname": "test:b",
"deps": ["/test:deps/a"]
},
{
"kind": "leaf",
"type": {
"primitive": true,
"name": "string"
},
"name": "c",
"qname": "test:c"
}
]
}
}
''' '''
} }
@ -476,6 +518,70 @@ class TestValueBuilder(unittest.TestCase):
self.assertEqual(0, len(calls)) self.assertEqual(0, len(calls))
@patch('ansible.module_utils.network.nso.nso.open_url')
def test_sort_by_deps(self, open_url_mock):
calls = [
MockResponse('new_trans', {}, 200, '{"result": {"th": 1}}'),
get_schema_response('/test:deps')
]
open_url_mock.side_effect = lambda *args, **kwargs: mock_call(calls, *args, **kwargs)
parent = "/test:deps"
schema_data = json.loads(
SCHEMA_DATA['/test:deps'])
schema = schema_data['data']
values = {
'a': '1',
'b': '2',
'c': '3',
}
vb = nso.ValueBuilder(nso.JsonRpc('http://localhost:8080/jsonrpc'))
vb.build(parent, None, values, schema)
self.assertEquals(3, len(vb.values))
value = vb.values[0]
self.assertEquals('{0}/c'.format(parent), value.path)
self.assertEquals('3', value.value)
value = vb.values[1]
self.assertEquals('{0}/a'.format(parent), value.path)
self.assertEquals('1', value.value)
value = vb.values[2]
self.assertEquals('{0}/b'.format(parent), value.path)
self.assertEquals('2', value.value)
self.assertEqual(0, len(calls))
@patch('ansible.module_utils.network.nso.nso.open_url')
def test_sort_by_deps_not_included(self, open_url_mock):
calls = [
MockResponse('new_trans', {}, 200, '{"result": {"th": 1}}'),
get_schema_response('/test:deps')
]
open_url_mock.side_effect = lambda *args, **kwargs: mock_call(calls, *args, **kwargs)
parent = "/test:deps"
schema_data = json.loads(
SCHEMA_DATA['/test:deps'])
schema = schema_data['data']
values = {
'a': '1',
'b': '2'
}
vb = nso.ValueBuilder(nso.JsonRpc('http://localhost:8080/jsonrpc'))
vb.build(parent, None, values, schema)
self.assertEquals(2, len(vb.values))
value = vb.values[0]
self.assertEquals('{0}/a'.format(parent), value.path)
self.assertEquals('1', value.value)
value = vb.values[1]
self.assertEquals('{0}/b'.format(parent), value.path)
self.assertEquals('2', value.value)
self.assertEqual(0, len(calls))
class TestVerifyVersion(unittest.TestCase): class TestVerifyVersion(unittest.TestCase):
def test_valid_versions(self): def test_valid_versions(self):