1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798991001011021031041051061071081091101111121131141151161171181191201211221231241251261271281291301311321331341351361371381391401411421431441451461471481491501511521531541551561571581591601611621631641651661671681691701711721731741751761771781791801811821831841851861871881891901911921931941951961971981992002012022032042052062072082092102112122132142152162172182192202212222232242252262272282292302312322332342352362372382392402412422432442452462472482492502512522532542552562572582592602612622632642652662672682692702712722732742752762772782792802812822832842852862872882892902912922932942952962972982993003013023033043053063073083093103113123133143153163173183193203213223233243253263273283293303313323333343353363373383393403413423433443453463473483493503513523533543553563573583593603613623633643653663673683693703713723733743753763773783793803813823833843853863873883893903913923933943953963973983994004014024034044054064074084094104114124134144154164174184194204214224234244254264274284294304314324334344354364374384394404414424434444454464474484494504514524534544554564574584594604614624634644654664674684694704714724734744754764774784794804814824834844854864874884894904914924934944954964974984995005015025035045055065075085095105115125135145155165175185195205215225235245255265275285295305315325335345355365375385395405415425435445455465475485495505515525535545555565575585595605615625635645655665675685695705715725735745755765775785795805815825835845855865875885895905915925935945955965975985996006016026036046056066076086096106116126136146156166176186196206216226236246256266276286296306316326336346356366376386396406416426436446456466476486496506516526536546556566576586596606616626636646656666676686696706716726736746756766776786796806816826836846856866876886896906916926936946956966976986997007017027037047057067077087097107117127137147157167177187197207217227237247257267277287297307317327337347357367377387397407417427437447457467477487497507517527537547557567577587597607617627637647657667677687697707717727737747757767777787797807817827837847857867877887897907917927937947957967977987998008018028038048058068078088098108118128138148158168178188198208218228238248258268278288298308318328338348358368378388398408418428438448458468478488498508518528538548558568578588598608618628638648658668678688698708718728738748758768778788798808818828838848858868878888898908918928938948958968978988999009019029039049059069079089099109119129139149159169179189199209219229239249259269279289299309319329339349359369379389399409419429439449459469479489499509519529539549559569579589599609619629639649659669679689699709719729739749759769779789799809819829839849859869879889899909919929939949959969979989991000100110021003100410051006100710081009101010111012101310141015101610171018101910201021102210231024102510261027102810291030103110321033103410351036103710381039104010411042104310441045104610471048104910501051105210531054105510561057105810591060106110621063106410651066106710681069107010711072107310741075107610771078107910801081108210831084108510861087108810891090109110921093109410951096109710981099110011011102110311041105110611071108110911101111111211131114111511161117111811191120112111221123112411251126112711281129113011311132113311341135113611371138113911401141114211431144114511461147114811491150115111521153115411551156115711581159116011611162116311641165116611671168116911701171117211731174117511761177117811791180118111821183118411851186118711881189119011911192119311941195119611971198119912001201120212031204120512061207120812091210121112121213121412151216121712181219122012211222122312241225122612271228122912301231123212331234123512361237123812391240124112421243124412451246124712481249125012511252125312541255125612571258125912601261126212631264126512661267126812691270127112721273127412751276127712781279128012811282128312841285128612871288128912901291129212931294129512961297129812991300130113021303130413051306130713081309131013111312131313141315131613171318131913201321132213231324132513261327132813291330133113321333133413351336133713381339134013411342134313441345134613471348134913501351135213531354135513561357135813591360136113621363136413651366136713681369137013711372137313741375137613771378 |
- """
- :codeauthor: Pedro Algarvio (pedro@algarvio.me)
- :codeauthor: Alexandru Bleotu (alexandru.bleotu@morganstanley.com)
- tests.unit.pillar_test
- ~~~~~~~~~~~~~~~~~~~~~~
- """
- # Import python libs
- import logging
- import os
- import shutil
- import tempfile
- import textwrap
- # Import salt libs
- import salt.config
- import salt.exceptions
- import salt.fileclient
- import salt.utils.stringutils
- from salt.utils.files import fopen
- from tests.support.helpers import with_tempdir
- from tests.support.mock import MagicMock, patch
- # Import Salt Testing libs
- from tests.support.runtests import RUNTIME_VARS
- from tests.support.unit import TestCase
- log = logging.getLogger(__name__)
- class MockFileclient:
- def __init__(self, cache_file=None, get_state=None, list_states=None):
- if cache_file is not None:
- self.cache_file = lambda *x, **y: cache_file
- if get_state is not None:
- self.get_state = lambda sls, env: get_state[sls]
- if list_states is not None:
- self.list_states = lambda *x, **y: list_states
- # pylint: disable=unused-argument,no-method-argument,method-hidden
- def cache_file(*args, **kwargs):
- raise NotImplementedError()
- def get_state(*args, **kwargs):
- raise NotImplementedError()
- def list_states(*args, **kwargs):
- raise NotImplementedError()
- # pylint: enable=unused-argument,no-method-argument,method-hidden
- class PillarTestCase(TestCase):
- def tearDown(self):
- for attrname in (
- "generic_file",
- "generic_minion_file",
- "ssh_file",
- "ssh_minion_file",
- "top_file",
- ):
- try:
- delattr(self, attrname)
- except AttributeError:
- continue
- def test_pillarenv_from_saltenv(self):
- with patch("salt.pillar.compile_template") as compile_template:
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "json",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "",
- "pillar_roots": {"dev": [], "base": []},
- "file_roots": {"dev": [], "base": []},
- "extension_modules": "",
- "pillarenv_from_saltenv": True,
- }
- grains = {
- "os": "Ubuntu",
- }
- pillar = salt.pillar.Pillar(opts, grains, "mocked-minion", "dev")
- self.assertEqual(pillar.opts["saltenv"], "dev")
- self.assertEqual(pillar.opts["pillarenv"], "dev")
- def test_ext_pillar_no_extra_minion_data_val_dict(self):
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "json",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "",
- "pillar_roots": {"dev": [], "base": []},
- "file_roots": {"dev": [], "base": []},
- "extension_modules": "",
- "pillarenv_from_saltenv": True,
- }
- mock_ext_pillar_func = MagicMock()
- with patch(
- "salt.loader.pillars",
- MagicMock(return_value={"fake_ext_pillar": mock_ext_pillar_func}),
- ):
- pillar = salt.pillar.Pillar(opts, {}, "mocked-minion", "dev")
- # ext pillar function doesn't have the extra_minion_data arg
- with patch(
- "salt.utils.args.get_function_argspec",
- MagicMock(return_value=MagicMock(args=[])),
- ):
- pillar._external_pillar_data(
- "fake_pillar", {"arg": "foo"}, "fake_ext_pillar"
- )
- mock_ext_pillar_func.assert_called_once_with(
- "mocked-minion", "fake_pillar", arg="foo"
- )
- # ext pillar function has the extra_minion_data arg
- mock_ext_pillar_func.reset_mock()
- with patch(
- "salt.utils.args.get_function_argspec",
- MagicMock(return_value=MagicMock(args=["extra_minion_data"])),
- ):
- pillar._external_pillar_data(
- "fake_pillar", {"arg": "foo"}, "fake_ext_pillar"
- )
- mock_ext_pillar_func.assert_called_once_with(
- "mocked-minion", "fake_pillar", arg="foo"
- )
- def test_ext_pillar_no_extra_minion_data_val_list(self):
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "json",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "",
- "pillar_roots": {"dev": [], "base": []},
- "file_roots": {"dev": [], "base": []},
- "extension_modules": "",
- "pillarenv_from_saltenv": True,
- }
- mock_ext_pillar_func = MagicMock()
- with patch(
- "salt.loader.pillars",
- MagicMock(return_value={"fake_ext_pillar": mock_ext_pillar_func}),
- ):
- pillar = salt.pillar.Pillar(opts, {}, "mocked-minion", "dev")
- # ext pillar function doesn't have the extra_minion_data arg
- with patch(
- "salt.utils.args.get_function_argspec",
- MagicMock(return_value=MagicMock(args=[])),
- ):
- pillar._external_pillar_data("fake_pillar", ["foo"], "fake_ext_pillar")
- mock_ext_pillar_func.assert_called_once_with(
- "mocked-minion", "fake_pillar", "foo"
- )
- # ext pillar function has the extra_minion_data arg
- mock_ext_pillar_func.reset_mock()
- with patch(
- "salt.utils.args.get_function_argspec",
- MagicMock(return_value=MagicMock(args=["extra_minion_data"])),
- ):
- pillar._external_pillar_data("fake_pillar", ["foo"], "fake_ext_pillar")
- mock_ext_pillar_func.assert_called_once_with(
- "mocked-minion", "fake_pillar", "foo"
- )
- def test_ext_pillar_no_extra_minion_data_val_elem(self):
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "json",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "",
- "pillar_roots": {"dev": [], "base": []},
- "file_roots": {"dev": [], "base": []},
- "extension_modules": "",
- "pillarenv_from_saltenv": True,
- }
- mock_ext_pillar_func = MagicMock()
- with patch(
- "salt.loader.pillars",
- MagicMock(return_value={"fake_ext_pillar": mock_ext_pillar_func}),
- ):
- pillar = salt.pillar.Pillar(opts, {}, "mocked-minion", "dev")
- # ext pillar function doesn't have the extra_minion_data arg
- with patch(
- "salt.utils.args.get_function_argspec",
- MagicMock(return_value=MagicMock(args=[])),
- ):
- pillar._external_pillar_data("fake_pillar", "fake_val", "fake_ext_pillar")
- mock_ext_pillar_func.assert_called_once_with(
- "mocked-minion", "fake_pillar", "fake_val"
- )
- # ext pillar function has the extra_minion_data arg
- mock_ext_pillar_func.reset_mock()
- with patch(
- "salt.utils.args.get_function_argspec",
- MagicMock(return_value=MagicMock(args=["extra_minion_data"])),
- ):
- pillar._external_pillar_data("fake_pillar", "fake_val", "fake_ext_pillar")
- mock_ext_pillar_func.assert_called_once_with(
- "mocked-minion", "fake_pillar", "fake_val"
- )
- def test_ext_pillar_with_extra_minion_data_val_dict(self):
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "json",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "",
- "pillar_roots": {"dev": [], "base": []},
- "file_roots": {"dev": [], "base": []},
- "extension_modules": "",
- "pillarenv_from_saltenv": True,
- }
- mock_ext_pillar_func = MagicMock()
- with patch(
- "salt.loader.pillars",
- MagicMock(return_value={"fake_ext_pillar": mock_ext_pillar_func}),
- ):
- pillar = salt.pillar.Pillar(
- opts, {}, "mocked-minion", "dev", extra_minion_data={"fake_key": "foo"}
- )
- # ext pillar function doesn't have the extra_minion_data arg
- with patch(
- "salt.utils.args.get_function_argspec",
- MagicMock(return_value=MagicMock(args=[])),
- ):
- pillar._external_pillar_data(
- "fake_pillar", {"arg": "foo"}, "fake_ext_pillar"
- )
- mock_ext_pillar_func.assert_called_once_with(
- "mocked-minion", "fake_pillar", arg="foo"
- )
- # ext pillar function has the extra_minion_data arg
- mock_ext_pillar_func.reset_mock()
- with patch(
- "salt.utils.args.get_function_argspec",
- MagicMock(return_value=MagicMock(args=["extra_minion_data"])),
- ):
- pillar._external_pillar_data(
- "fake_pillar", {"arg": "foo"}, "fake_ext_pillar"
- )
- mock_ext_pillar_func.assert_called_once_with(
- "mocked-minion",
- "fake_pillar",
- arg="foo",
- extra_minion_data={"fake_key": "foo"},
- )
- def test_ext_pillar_with_extra_minion_data_val_list(self):
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "json",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "",
- "pillar_roots": {"dev": [], "base": []},
- "file_roots": {"dev": [], "base": []},
- "extension_modules": "",
- "pillarenv_from_saltenv": True,
- }
- mock_ext_pillar_func = MagicMock()
- with patch(
- "salt.loader.pillars",
- MagicMock(return_value={"fake_ext_pillar": mock_ext_pillar_func}),
- ):
- pillar = salt.pillar.Pillar(
- opts, {}, "mocked-minion", "dev", extra_minion_data={"fake_key": "foo"}
- )
- # ext pillar function doesn't have the extra_minion_data arg
- with patch(
- "salt.utils.args.get_function_argspec",
- MagicMock(return_value=MagicMock(args=[])),
- ):
- pillar._external_pillar_data("fake_pillar", ["bar"], "fake_ext_pillar")
- mock_ext_pillar_func.assert_called_once_with(
- "mocked-minion", "fake_pillar", "bar"
- )
- # ext pillar function has the extra_minion_data arg
- mock_ext_pillar_func.reset_mock()
- with patch(
- "salt.utils.args.get_function_argspec",
- MagicMock(return_value=MagicMock(args=["extra_minion_data"])),
- ):
- pillar._external_pillar_data("fake_pillar", ["bar"], "fake_ext_pillar")
- mock_ext_pillar_func.assert_called_once_with(
- "mocked-minion", "fake_pillar", "bar", extra_minion_data={"fake_key": "foo"}
- )
- def test_ext_pillar_with_extra_minion_data_val_elem(self):
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "json",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "",
- "pillar_roots": {"dev": [], "base": []},
- "file_roots": {"dev": [], "base": []},
- "extension_modules": "",
- "pillarenv_from_saltenv": True,
- }
- mock_ext_pillar_func = MagicMock()
- with patch(
- "salt.loader.pillars",
- MagicMock(return_value={"fake_ext_pillar": mock_ext_pillar_func}),
- ):
- pillar = salt.pillar.Pillar(
- opts, {}, "mocked-minion", "dev", extra_minion_data={"fake_key": "foo"}
- )
- # ext pillar function doesn't have the extra_minion_data arg
- with patch(
- "salt.utils.args.get_function_argspec",
- MagicMock(return_value=MagicMock(args=[])),
- ):
- pillar._external_pillar_data("fake_pillar", "bar", "fake_ext_pillar")
- mock_ext_pillar_func.assert_called_once_with(
- "mocked-minion", "fake_pillar", "bar"
- )
- # ext pillar function has the extra_minion_data arg
- mock_ext_pillar_func.reset_mock()
- with patch(
- "salt.utils.args.get_function_argspec",
- MagicMock(return_value=MagicMock(args=["extra_minion_data"])),
- ):
- pillar._external_pillar_data("fake_pillar", "bar", "fake_ext_pillar")
- mock_ext_pillar_func.assert_called_once_with(
- "mocked-minion", "fake_pillar", "bar", extra_minion_data={"fake_key": "foo"}
- )
- def test_ext_pillar_first(self):
- """
- test when using ext_pillar and ext_pillar_first
- """
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "yaml",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "",
- "pillar_roots": [],
- "extension_modules": "",
- "saltenv": "base",
- "file_roots": [],
- "ext_pillar_first": True,
- }
- grains = {
- "os": "Ubuntu",
- "os_family": "Debian",
- "oscodename": "raring",
- "osfullname": "Ubuntu",
- "osrelease": "13.04",
- "kernel": "Linux",
- }
- tempdir = tempfile.mkdtemp(dir=RUNTIME_VARS.TMP)
- try:
- sls_files = self._setup_test_topfile_sls_pillar_match(tempdir,)
- fc_mock = MockFileclient(
- cache_file=sls_files["top"]["dest"],
- list_states=["top", "ssh", "ssh.minion", "generic", "generic.minion"],
- get_state=sls_files,
- )
- with patch.object(
- salt.fileclient, "get_file_client", MagicMock(return_value=fc_mock)
- ), patch(
- "salt.pillar.Pillar.ext_pillar",
- MagicMock(
- return_value=(
- {"id": "minion", "phase": "alpha", "role": "database"},
- [],
- )
- ),
- ):
- pillar = salt.pillar.Pillar(opts, grains, "mocked-minion", "base")
- self.assertEqual(pillar.compile_pillar()["generic"]["key1"], "value1")
- finally:
- shutil.rmtree(tempdir, ignore_errors=True)
- def test_dynamic_pillarenv(self):
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "json",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "",
- "pillar_roots": {
- "__env__": ["/srv/pillar/__env__"],
- "base": ["/srv/pillar/base"],
- },
- "file_roots": {"base": ["/srv/salt/base"], "dev": ["/svr/salt/dev"]},
- "extension_modules": "",
- }
- pillar = salt.pillar.Pillar(opts, {}, "mocked-minion", "base", pillarenv="dev")
- self.assertEqual(
- pillar.opts["pillar_roots"],
- {"base": ["/srv/pillar/base"], "dev": ["/srv/pillar/__env__"]},
- )
- def test_ignored_dynamic_pillarenv(self):
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "json",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "",
- "pillar_roots": {
- "__env__": ["/srv/pillar/__env__"],
- "base": ["/srv/pillar/base"],
- },
- "file_roots": {"base": ["/srv/salt/base"], "dev": ["/svr/salt/dev"]},
- "extension_modules": "",
- }
- pillar = salt.pillar.Pillar(opts, {}, "mocked-minion", "base", pillarenv="base")
- self.assertEqual(pillar.opts["pillar_roots"], {"base": ["/srv/pillar/base"]})
- @patch("salt.fileclient.Client.list_states")
- def test_malformed_pillar_sls(self, mock_list_states):
- with patch("salt.pillar.compile_template") as compile_template:
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "json",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "",
- "pillar_roots": [],
- "file_roots": [],
- "extension_modules": "",
- }
- grains = {
- "os": "Ubuntu",
- "os_family": "Debian",
- "oscodename": "raring",
- "osfullname": "Ubuntu",
- "osrelease": "13.04",
- "kernel": "Linux",
- }
- mock_list_states.return_value = ["foo", "blah"]
- pillar = salt.pillar.Pillar(opts, grains, "mocked-minion", "base")
- # Mock getting the proper template files
- pillar.client.get_state = MagicMock(
- return_value={
- "dest": "/path/to/pillar/files/foo.sls",
- "source": "salt://foo.sls",
- }
- )
- # Template compilation returned a string
- compile_template.return_value = "BAHHH"
- self.assertEqual(
- pillar.render_pillar({"base": ["foo.sls"]}),
- ({}, ["SLS 'foo.sls' does not render to a dictionary"]),
- )
- # Template compilation returned a list
- compile_template.return_value = ["BAHHH"]
- self.assertEqual(
- pillar.render_pillar({"base": ["foo.sls"]}),
- ({}, ["SLS 'foo.sls' does not render to a dictionary"]),
- )
- # Template compilation returned a dictionary, which is what's expected
- compile_template.return_value = {"foo": "bar"}
- self.assertEqual(
- pillar.render_pillar({"base": ["foo.sls"]}), ({"foo": "bar"}, [])
- )
- # Test improper includes
- compile_template.side_effect = [
- {"foo": "bar", "include": "blah"},
- {"foo2": "bar2"},
- ]
- self.assertEqual(
- pillar.render_pillar({"base": ["foo.sls"]}),
- (
- {"foo": "bar", "include": "blah"},
- ["Include Declaration in SLS 'foo.sls' is not formed as a list"],
- ),
- )
- # Test includes as a list, which is what's expected
- compile_template.side_effect = [
- {"foo": "bar", "include": ["blah"]},
- {"foo2": "bar2"},
- ]
- self.assertEqual(
- pillar.render_pillar({"base": ["foo.sls"]}),
- ({"foo": "bar", "foo2": "bar2"}, []),
- )
- # Test includes as a list overriding data
- compile_template.side_effect = [
- {"foo": "bar", "include": ["blah"]},
- {"foo": "bar2"},
- ]
- self.assertEqual(
- pillar.render_pillar({"base": ["foo.sls"]}), ({"foo": "bar"}, [])
- )
- # Test includes using empty key directive
- compile_template.side_effect = [
- {"foo": "bar", "include": [{"blah": {"key": ""}}]},
- {"foo": "bar2"},
- ]
- self.assertEqual(
- pillar.render_pillar({"base": ["foo.sls"]}), ({"foo": "bar"}, [])
- )
- # Test includes using simple non-nested key
- compile_template.side_effect = [
- {"foo": "bar", "include": [{"blah": {"key": "nested"}}]},
- {"foo": "bar2"},
- ]
- self.assertEqual(
- pillar.render_pillar({"base": ["foo.sls"]}),
- ({"foo": "bar", "nested": {"foo": "bar2"}}, []),
- )
- # Test includes using nested key
- compile_template.side_effect = [
- {"foo": "bar", "include": [{"blah": {"key": "nested:level"}}]},
- {"foo": "bar2"},
- ]
- self.assertEqual(
- pillar.render_pillar({"base": ["foo.sls"]}),
- ({"foo": "bar", "nested": {"level": {"foo": "bar2"}}}, []),
- )
- def test_includes_override_sls(self):
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "json",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "",
- "pillar_roots": {},
- "file_roots": {},
- "extension_modules": "",
- }
- grains = {
- "os": "Ubuntu",
- "os_family": "Debian",
- "oscodename": "raring",
- "osfullname": "Ubuntu",
- "osrelease": "13.04",
- "kernel": "Linux",
- }
- with patch("salt.pillar.compile_template") as compile_template, patch.object(
- salt.pillar.Pillar,
- "_Pillar__gather_avail",
- MagicMock(return_value={"base": ["blah", "foo"]}),
- ):
- # Test with option set to True
- opts["pillar_includes_override_sls"] = True
- pillar = salt.pillar.Pillar(opts, grains, "mocked-minion", "base")
- # Mock getting the proper template files
- pillar.client.get_state = MagicMock(
- return_value={
- "dest": "/path/to/pillar/files/foo.sls",
- "source": "salt://foo.sls",
- }
- )
- compile_template.side_effect = [
- {"foo": "bar", "include": ["blah"]},
- {"foo": "bar2"},
- ]
- self.assertEqual(
- pillar.render_pillar({"base": ["foo.sls"]}), ({"foo": "bar2"}, [])
- )
- # Test with option set to False
- opts["pillar_includes_override_sls"] = False
- pillar = salt.pillar.Pillar(opts, grains, "mocked-minion", "base")
- # Mock getting the proper template files
- pillar.client.get_state = MagicMock(
- return_value={
- "dest": "/path/to/pillar/files/foo.sls",
- "source": "salt://foo.sls",
- }
- )
- compile_template.side_effect = [
- {"foo": "bar", "include": ["blah"]},
- {"foo": "bar2"},
- ]
- self.assertEqual(
- pillar.render_pillar({"base": ["foo.sls"]}), ({"foo": "bar"}, [])
- )
- def test_topfile_order(self):
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "yaml",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "",
- "pillar_roots": [],
- "extension_modules": "",
- "saltenv": "base",
- "file_roots": [],
- }
- grains = {
- "os": "Ubuntu",
- "os_family": "Debian",
- "oscodename": "raring",
- "osfullname": "Ubuntu",
- "osrelease": "13.04",
- "kernel": "Linux",
- }
- def _run_test(nodegroup_order, glob_order, expected):
- tempdir = tempfile.mkdtemp(dir=RUNTIME_VARS.TMP)
- try:
- sls_files = self._setup_test_topfile_sls(
- tempdir, nodegroup_order, glob_order
- )
- fc_mock = MockFileclient(
- cache_file=sls_files["top"]["dest"],
- list_states=[
- "top",
- "ssh",
- "ssh.minion",
- "generic",
- "generic.minion",
- ],
- get_state=sls_files,
- )
- with patch.object(
- salt.fileclient, "get_file_client", MagicMock(return_value=fc_mock)
- ):
- pillar = salt.pillar.Pillar(opts, grains, "mocked-minion", "base")
- # Make sure that confirm_top.confirm_top returns True
- pillar.matchers["confirm_top.confirm_top"] = lambda *x, **y: True
- self.assertEqual(pillar.compile_pillar()["ssh"], expected)
- finally:
- shutil.rmtree(tempdir, ignore_errors=True)
- # test case where glob match happens second and therefore takes
- # precedence over nodegroup match.
- _run_test(nodegroup_order=1, glob_order=2, expected="bar")
- # test case where nodegroup match happens second and therefore takes
- # precedence over glob match.
- _run_test(nodegroup_order=2, glob_order=1, expected="foo")
- def _setup_test_topfile_sls_pillar_match(self, tempdir):
- # Write a simple topfile and two pillar state files
- top_file = tempfile.NamedTemporaryFile(dir=tempdir, delete=False)
- s = """
- base:
- 'phase:alpha':
- - match: pillar
- - generic
- """
- top_file.write(salt.utils.stringutils.to_bytes(s))
- top_file.flush()
- generic_file = tempfile.NamedTemporaryFile(dir=tempdir, delete=False)
- generic_file.write(
- b"""
- generic:
- key1: value1
- """
- )
- generic_file.flush()
- return {
- "top": {"path": "", "dest": top_file.name},
- "generic": {"path": "", "dest": generic_file.name},
- }
- def _setup_test_topfile_sls(self, tempdir, nodegroup_order, glob_order):
- # Write a simple topfile and two pillar state files
- top_file = tempfile.NamedTemporaryFile(dir=tempdir, delete=False)
- s = """
- base:
- group:
- - match: nodegroup
- - order: {nodegroup_order}
- - ssh
- - generic
- '*':
- - generic
- minion:
- - order: {glob_order}
- - ssh.minion
- - generic.minion
- """.format(
- nodegroup_order=nodegroup_order, glob_order=glob_order
- )
- top_file.write(salt.utils.stringutils.to_bytes(s))
- top_file.flush()
- ssh_file = tempfile.NamedTemporaryFile(dir=tempdir, delete=False)
- ssh_file.write(
- b"""
- ssh:
- foo
- """
- )
- ssh_file.flush()
- ssh_minion_file = tempfile.NamedTemporaryFile(dir=tempdir, delete=False)
- ssh_minion_file.write(
- b"""
- ssh:
- bar
- """
- )
- ssh_minion_file.flush()
- generic_file = tempfile.NamedTemporaryFile(dir=tempdir, delete=False)
- generic_file.write(
- b"""
- generic:
- key1:
- - value1
- - value2
- key2:
- sub_key1: []
- """
- )
- generic_file.flush()
- generic_minion_file = tempfile.NamedTemporaryFile(dir=tempdir, delete=False)
- generic_minion_file.write(
- b"""
- generic:
- key1:
- - value3
- key2:
- sub_key2: []
- """
- )
- generic_minion_file.flush()
- return {
- "top": {"path": "", "dest": top_file.name},
- "ssh": {"path": "", "dest": ssh_file.name},
- "ssh.minion": {"path": "", "dest": ssh_minion_file.name},
- "generic": {"path": "", "dest": generic_file.name},
- "generic.minion": {"path": "", "dest": generic_minion_file.name},
- }
- @with_tempdir()
- def test_include(self, tempdir):
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "yaml",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "",
- "pillar_roots": [],
- "extension_modules": "",
- "saltenv": "base",
- "file_roots": [],
- }
- grains = {
- "os": "Ubuntu",
- "os_family": "Debian",
- "oscodename": "raring",
- "osfullname": "Ubuntu",
- "osrelease": "13.04",
- "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",
- "test.sub_with_init_dot",
- "test.sub.with.slashes",
- ],
- )
- with patch.object(
- salt.fileclient, "get_file_client", MagicMock(return_value=fc_mock)
- ):
- pillar = salt.pillar.Pillar(opts, grains, "minion", "base")
- # Make sure that confirm_top.confirm_top returns True
- pillar.matchers["confirm_top.confirm_top"] = lambda *x, **y: True
- compiled_pillar = pillar.compile_pillar()
- 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)
- top_file.write(
- b"""
- base:
- '*':
- - order: 1
- - test.sub2
- minion:
- - order: 2
- - test
- """
- )
- top_file.flush()
- init_sls = tempfile.NamedTemporaryFile(dir=tempdir, delete=False)
- init_sls.write(
- b"""
- 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)
- sub1_sls.write(
- b"""
- foo1:
- bar1
- """
- )
- sub1_sls.flush()
- sub2_sls = tempfile.NamedTemporaryFile(dir=tempdir, delete=False)
- sub2_sls.write(
- b"""
- foo2:
- bar2
- """
- )
- sub2_sls.flush()
- sub_wildcard_1_sls = tempfile.NamedTemporaryFile(dir=tempdir, delete=False)
- sub_wildcard_1_sls.write(
- b"""
- foo_wildcard:
- bar_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},
- }
- @with_tempdir()
- def test_relative_include(self, tempdir):
- join = os.path.join
- with fopen(join(tempdir, "top.sls"), "w") as f:
- print(
- textwrap.dedent(
- """
- base:
- '*':
- - includer
- - simple_includer
- - includes.with.more.depth
- """
- ),
- file=f,
- )
- includer_dir = join(tempdir, "includer")
- os.makedirs(includer_dir)
- with fopen(join(includer_dir, "init.sls"), "w") as f:
- print(
- textwrap.dedent(
- """
- include:
- - .this
- - includer.that
- """
- ),
- file=f,
- )
- with fopen(join(includer_dir, "this.sls"), "w") as f:
- print(
- textwrap.dedent(
- """
- this:
- is all good
- """
- ),
- file=f,
- )
- with fopen(join(includer_dir, "that.sls"), "w") as f:
- print(
- textwrap.dedent(
- """
- that:
- is also all good
- """
- ),
- file=f,
- )
- with fopen(join(tempdir, "simple_includer.sls"), "w") as simpleincluder:
- print(
- textwrap.dedent(
- """
- include:
- - .simple
- - super_simple
- """
- ),
- file=simpleincluder,
- )
- with fopen(join(tempdir, "simple.sls"), "w") as f:
- print(
- textwrap.dedent(
- """
- simple:
- simon
- """
- ),
- file=f,
- )
- with fopen(join(tempdir, "super_simple.sls"), "w") as f:
- print(
- textwrap.dedent(
- """
- super simple:
- a caveman
- """
- ),
- file=f,
- )
- depth_dir = join(tempdir, "includes", "with", "more")
- os.makedirs(depth_dir)
- with fopen(join(depth_dir, "depth.sls"), "w") as f:
- print(
- textwrap.dedent(
- """
- include:
- - .ramble
- - includes.with.more.doors
- mordor:
- has dark depths
- """
- ),
- file=f,
- )
- with fopen(join(depth_dir, "ramble.sls"), "w") as f:
- print(
- textwrap.dedent(
- """
- found:
- my precious
- """
- ),
- file=f,
- )
- with fopen(join(depth_dir, "doors.sls"), "w") as f:
- print(
- textwrap.dedent(
- """
- mojo:
- bad risin'
- """
- ),
- file=f,
- )
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "yaml",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "top.sls",
- "pillar_roots": {"base": [tempdir]},
- "extension_modules": "",
- "saltenv": "base",
- "file_roots": [],
- "file_ignore_regex": None,
- "file_ignore_glob": None,
- }
- grains = {
- "os": "Ubuntu",
- "os_family": "Debian",
- "oscodename": "raring",
- "osfullname": "Ubuntu",
- "osrelease": "13.04",
- "kernel": "Linux",
- }
- pillar = salt.pillar.Pillar(opts, grains, "minion", "base")
- # Make sure that confirm_top.confirm_top returns True
- pillar.matchers["confirm_top.confirm_top"] = lambda *x, **y: True
- # Act
- compiled_pillar = pillar.compile_pillar()
- # Assert
- self.assertEqual(compiled_pillar["this"], "is all good")
- self.assertEqual(compiled_pillar["that"], "is also all good")
- self.assertEqual(compiled_pillar["simple"], "simon")
- self.assertEqual(compiled_pillar["super simple"], "a caveman")
- self.assertEqual(compiled_pillar["mordor"], "has dark depths")
- self.assertEqual(compiled_pillar["found"], "my precious")
- self.assertEqual(compiled_pillar["mojo"], "bad risin'")
- @with_tempdir()
- def test_missing_include(self, tempdir):
- opts = {
- "optimization_order": [0, 1, 2],
- "renderer": "yaml",
- "renderer_blacklist": [],
- "renderer_whitelist": [],
- "state_top": "top.sls",
- "pillar_roots": {"base": [tempdir]},
- "extension_modules": "",
- "saltenv": "base",
- "file_roots": [],
- "file_ignore_regex": None,
- "file_ignore_glob": None,
- }
- grains = {
- "os": "Ubuntu",
- "os_family": "Debian",
- "oscodename": "raring",
- "osfullname": "Ubuntu",
- "osrelease": "13.04",
- "kernel": "Linux",
- }
- join = os.path.join
- with fopen(join(tempdir, "top.sls"), "w") as f:
- print(
- textwrap.dedent(
- """
- base:
- '*':
- - simple_include
- """
- ),
- file=f,
- )
- include_dir = join(tempdir, "simple_include")
- os.makedirs(include_dir)
- with fopen(join(include_dir, "init.sls"), "w") as f:
- print(
- textwrap.dedent(
- """
- include:
- - simple_include.missing_include
- simple_include: is ok
- """
- ),
- file=f,
- )
- pillar = salt.pillar.Pillar(opts, grains, "minion", "base")
- # Make sure that confirm_top.confirm_top returns True
- pillar.matchers["confirm_top.confirm_top"] = lambda *x, **y: True
- # Act
- compiled_pillar = pillar.compile_pillar()
- # Assert
- self.assertEqual(compiled_pillar["simple_include"], "is ok")
- self.assertTrue("_errors" in compiled_pillar)
- self.assertTrue(
- "simple_include.missing_include" in compiled_pillar["_errors"][0]
- )
- @patch("salt.transport.client.ReqChannel.factory", MagicMock())
- class RemotePillarTestCase(TestCase):
- """
- Tests for instantiating a RemotePillar in salt.pillar
- """
- def setUp(self):
- self.grains = {}
- def tearDown(self):
- for attr in ("grains",):
- try:
- delattr(self, attr)
- except AttributeError:
- continue
- def test_get_opts_in_pillar_override_call(self):
- mock_get_extra_minion_data = MagicMock(return_value={})
- with patch(
- "salt.pillar.RemotePillarMixin.get_ext_pillar_extra_minion_data",
- mock_get_extra_minion_data,
- ):
- salt.pillar.RemotePillar({}, self.grains, "mocked-minion", "dev")
- mock_get_extra_minion_data.assert_called_once_with({"saltenv": "dev"})
- def test_multiple_keys_in_opts_added_to_pillar(self):
- opts = {
- "renderer": "json",
- "path_to_add": "fake_data",
- "path_to_add2": {"fake_data2": ["fake_data3", "fake_data4"]},
- "pass_to_ext_pillars": ["path_to_add", "path_to_add2"],
- }
- pillar = salt.pillar.RemotePillar(opts, self.grains, "mocked-minion", "dev")
- self.assertEqual(
- pillar.extra_minion_data,
- {
- "path_to_add": "fake_data",
- "path_to_add2": {"fake_data2": ["fake_data3", "fake_data4"]},
- },
- )
- def test_subkey_in_opts_added_to_pillar(self):
- opts = {
- "renderer": "json",
- "path_to_add": "fake_data",
- "path_to_add2": {
- "fake_data5": "fake_data6",
- "fake_data2": ["fake_data3", "fake_data4"],
- },
- "pass_to_ext_pillars": ["path_to_add2:fake_data5"],
- }
- pillar = salt.pillar.RemotePillar(opts, self.grains, "mocked-minion", "dev")
- self.assertEqual(
- pillar.extra_minion_data, {"path_to_add2": {"fake_data5": "fake_data6"}}
- )
- def test_non_existent_leaf_opt_in_add_to_pillar(self):
- opts = {
- "renderer": "json",
- "path_to_add": "fake_data",
- "path_to_add2": {
- "fake_data5": "fake_data6",
- "fake_data2": ["fake_data3", "fake_data4"],
- },
- "pass_to_ext_pillars": ["path_to_add2:fake_data_non_exist"],
- }
- pillar = salt.pillar.RemotePillar(opts, self.grains, "mocked-minion", "dev")
- self.assertEqual(pillar.pillar_override, {})
- def test_non_existent_intermediate_opt_in_add_to_pillar(self):
- opts = {
- "renderer": "json",
- "path_to_add": "fake_data",
- "path_to_add2": {
- "fake_data5": "fake_data6",
- "fake_data2": ["fake_data3", "fake_data4"],
- },
- "pass_to_ext_pillars": ["path_to_add_no_exist"],
- }
- pillar = salt.pillar.RemotePillar(opts, self.grains, "mocked-minion", "dev")
- self.assertEqual(pillar.pillar_override, {})
- def test_malformed_add_to_pillar(self):
- opts = {
- "renderer": "json",
- "path_to_add": "fake_data",
- "path_to_add2": {
- "fake_data5": "fake_data6",
- "fake_data2": ["fake_data3", "fake_data4"],
- },
- "pass_to_ext_pillars": MagicMock(),
- }
- with self.assertRaises(salt.exceptions.SaltClientError) as excinfo:
- salt.pillar.RemotePillar(opts, self.grains, "mocked-minion", "dev")
- self.assertEqual(
- excinfo.exception.strerror, "'pass_to_ext_pillars' config is malformed."
- )
- def test_pillar_send_extra_minion_data_from_config(self):
- opts = {
- "renderer": "json",
- "pillarenv": "fake_pillar_env",
- "path_to_add": "fake_data",
- "path_to_add2": {
- "fake_data5": "fake_data6",
- "fake_data2": ["fake_data3", "fake_data4"],
- },
- "pass_to_ext_pillars": ["path_to_add"],
- }
- mock_channel = MagicMock(
- crypted_transfer_decode_dictentry=MagicMock(return_value={})
- )
- with patch(
- "salt.transport.client.ReqChannel.factory",
- MagicMock(return_value=mock_channel),
- ):
- pillar = salt.pillar.RemotePillar(
- opts, self.grains, "mocked_minion", "fake_env"
- )
- ret = pillar.compile_pillar()
- self.assertEqual(pillar.channel, mock_channel)
- mock_channel.crypted_transfer_decode_dictentry.assert_called_once_with(
- {
- "cmd": "_pillar",
- "ver": "2",
- "id": "mocked_minion",
- "grains": {},
- "saltenv": "fake_env",
- "pillarenv": "fake_pillar_env",
- "pillar_override": {},
- "extra_minion_data": {"path_to_add": "fake_data"},
- },
- dictkey="pillar",
- )
- def test_pillar_file_client_master_remote(self):
- """
- Test condition where local file_client and use_master_when_local option
- returns a remote file client.
- """
- mocked_minion = MagicMock()
- opts = {
- "file_client": "local",
- "use_master_when_local": True,
- "pillar_cache": None,
- }
- pillar = salt.pillar.get_pillar(opts, self.grains, mocked_minion)
- self.assertEqual(type(pillar), salt.pillar.RemotePillar)
- self.assertNotEqual(type(pillar), salt.pillar.PillarCache)
- @patch("salt.transport.client.AsyncReqChannel.factory", MagicMock())
- class AsyncRemotePillarTestCase(TestCase):
- """
- Tests for instantiating a AsyncRemotePillar in salt.pillar
- """
- def setUp(self):
- self.grains = {}
- def tearDown(self):
- for attr in ("grains",):
- try:
- delattr(self, attr)
- except AttributeError:
- continue
- def test_get_opts_in_pillar_override_call(self):
- mock_get_extra_minion_data = MagicMock(return_value={})
- with patch(
- "salt.pillar.RemotePillarMixin.get_ext_pillar_extra_minion_data",
- mock_get_extra_minion_data,
- ):
- salt.pillar.RemotePillar({}, self.grains, "mocked-minion", "dev")
- mock_get_extra_minion_data.assert_called_once_with({"saltenv": "dev"})
- def test_pillar_send_extra_minion_data_from_config(self):
- opts = {
- "renderer": "json",
- "pillarenv": "fake_pillar_env",
- "path_to_add": "fake_data",
- "path_to_add2": {
- "fake_data5": "fake_data6",
- "fake_data2": ["fake_data3", "fake_data4"],
- },
- "pass_to_ext_pillars": ["path_to_add"],
- }
- mock_channel = MagicMock(
- crypted_transfer_decode_dictentry=MagicMock(return_value={})
- )
- with patch(
- "salt.transport.client.AsyncReqChannel.factory",
- MagicMock(return_value=mock_channel),
- ):
- pillar = salt.pillar.RemotePillar(
- opts, self.grains, "mocked_minion", "fake_env"
- )
- ret = pillar.compile_pillar()
- mock_channel.crypted_transfer_decode_dictentry.assert_called_once_with(
- {
- "cmd": "_pillar",
- "ver": "2",
- "id": "mocked_minion",
- "grains": {},
- "saltenv": "fake_env",
- "pillarenv": "fake_pillar_env",
- "pillar_override": {},
- "extra_minion_data": {"path_to_add": "fake_data"},
- },
- dictkey="pillar",
- )
- @patch("salt.transport.client.ReqChannel.factory", MagicMock())
- class PillarCacheTestCase(TestCase):
- """
- Tests for instantiating a PillarCache in salt.pillar
- """
- def setUp(self):
- self.grains = {}
- @classmethod
- def setUpClass(cls):
- cls.mock_master_default_opts = salt.config.DEFAULT_MASTER_OPTS.copy()
- cls.mock_master_default_opts["cachedir"] = tempfile.mkdtemp(
- dir=RUNTIME_VARS.TMP
- )
- def tearDown(self):
- for attr in ("grains",):
- try:
- delattr(self, attr)
- except AttributeError:
- continue
- def test_compile_pillar(self):
- self.mock_master_default_opts.update(
- {"pillar_cache_backend": "disk", "pillar_cache_ttl": 3600}
- )
- pillar = salt.pillar.PillarCache(
- self.mock_master_default_opts,
- self.grains,
- "mocked_minion",
- "fake_env",
- pillarenv="base",
- )
- with patch("salt.utils.cache.CacheDisk._write", MagicMock()):
- with patch(
- "salt.pillar.PillarCache.fetch_pillar",
- side_effect=[{"foo": "bar"}, {"foo": "baz"}],
- ):
- # Run once for pillarenv base
- ret = pillar.compile_pillar()
- expected_cache = {"mocked_minion": {"base": {"foo": "bar"}}}
- self.assertEqual(pillar.cache._dict, expected_cache)
- # Run a second time for pillarenv base
- ret = pillar.compile_pillar()
- expected_cache = {"mocked_minion": {"base": {"foo": "bar"}}}
- self.assertEqual(pillar.cache._dict, expected_cache)
- # Change the pillarenv
- pillar.pillarenv = "dev"
- # Run once for pillarenv dev
- ret = pillar.compile_pillar()
- expected_cache = {
- "mocked_minion": {"base": {"foo": "bar"}, "dev": {"foo": "baz"}}
- }
- self.assertEqual(pillar.cache._dict, expected_cache)
- # Run a second time for pillarenv dev
- ret = pillar.compile_pillar()
- expected_cache = {
- "mocked_minion": {"base": {"foo": "bar"}, "dev": {"foo": "baz"}}
- }
- self.assertEqual(pillar.cache._dict, expected_cache)
|