saltstack/salt
1
0
Fork 0
mirror of https://github.com/saltstack/salt.git synced 2025-04-17 10:10:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Allow leading dots and / as pillar separators

Browse source
This commit is contained in:
Wayne Werner 2019-03-11 15:11:55 -05:00
parent 29c676d24b
commit ee3115fa94
No known key found for this signature in database
GPG key ID: C36D3A8D5BEF0935
2 changed files with 31 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
salt/pillar/__init__.py
View file

@ -776,7 +776,7 @@ class Pillar(object):
try:
matched_pstates.extend(fnmatch.filter(
self.avail[saltenv],
sub_sls.replace('/', '.'),
sub_sls.lstrip('.').replace('/', '.'),
))
except KeyError:
errors.extend(

33
tests/unit/test_pillar.py
View file

@ -659,14 +659,21 @@ generic:
'oscodename': 'raring',
'osfullname': 'Ubuntu',
'osrelease': '13.04',
'kernel': 'Linux'
'kernel': 'Linux',
}
sls_files = self._setup_test_include_sls(tempdir)
fc_mock = MockFileclient(
cache_file=sls_files['top']['dest'],
get_state=sls_files,
list_states=['top', 'test.init', 'test.sub1',
'test.sub2', 'test.sub_wildcard_1'],
list_states=[
'top',
'test.init',
'test.sub1',
'test.sub2',
'test.sub_wildcard_1',
'test.sub_with_init_dot',
'test.sub.with.slashes',
],
)
with patch.object(salt.fileclient, 'get_file_client',
MagicMock(return_value=fc_mock)):
@ -677,6 +684,8 @@ generic:
self.assertEqual(compiled_pillar['foo_wildcard'], 'bar_wildcard')
self.assertEqual(compiled_pillar['foo1'], 'bar1')
self.assertEqual(compiled_pillar['foo2'], 'bar2')
self.assertEqual(compiled_pillar['sub_with_slashes'], 'sub_slashes_worked')
self.assertEqual(compiled_pillar['sub_init_dot'], 'sub_with_init_dot_worked')
def _setup_test_include_sls(self, tempdir):
top_file = tempfile.NamedTemporaryFile(dir=tempdir, delete=False)
@ -695,6 +704,8 @@ base:
include:
- test.sub1
- test.sub_wildcard*
- .test.sub_with_init_dot
- test/sub/with/slashes
''')
init_sls.flush()
sub1_sls = tempfile.NamedTemporaryFile(dir=tempdir, delete=False)
@ -717,12 +728,28 @@ foo_wildcard:
''')
sub_wildcard_1_sls.flush()
sub_with_init_dot_sls = tempfile.NamedTemporaryFile(dir=tempdir, delete=False)
sub_with_init_dot_sls.write(b'''
sub_init_dot:
sub_with_init_dot_worked
''')
sub_with_init_dot_sls.flush()
sub_with_slashes_sls = tempfile.NamedTemporaryFile(dir=tempdir, delete=False)
sub_with_slashes_sls.write(b'''
sub_with_slashes:
sub_slashes_worked
''')
sub_with_slashes_sls.flush()
return {
'top': {'path': '', 'dest': top_file.name},
'test': {'path': '', 'dest': init_sls.name},
'test.sub1': {'path': '', 'dest': sub1_sls.name},
'test.sub2': {'path': '', 'dest': sub2_sls.name},
'test.sub_wildcard_1': {'path': '', 'dest': sub_wildcard_1_sls.name},
'test.sub_with_init_dot': {'path': '', 'dest': sub_with_init_dot_sls.name},
'test.sub.with.slashes': {'path': '', 'dest': sub_with_slashes_sls.name},
}