Merge pull request #39141 from UtahDave/fix_cache_lookup_ZD1187

Don't overwrite the minion_ids var that was passed
This commit is contained in:
Thomas S Hatch 2017-02-03 13:56:24 -07:00 committed by GitHub
commit 6a9704189f

View file

@ -109,7 +109,8 @@ class MasterPillarUtil(object):
log.debug('Skipping cached mine data minion_data_cache'
'and enfore_mine_cache are both disabled.')
return mine_data
minion_ids = self.cache.list('minions')
if not minion_ids:
minion_ids = self.cache.list('minions')
for minion_id in minion_ids:
if not salt.utils.verify.valid_id(self.opts, minion_id):
continue
@ -127,7 +128,8 @@ class MasterPillarUtil(object):
log.debug('Skipping cached data because minion_data_cache is not '
'enabled.')
return grains, pillars
minion_ids = self.cache.list('minions')
if not minion_ids:
minion_ids = self.cache.list('minions')
for minion_id in minion_ids:
if not salt.utils.verify.valid_id(self.opts, minion_id):
continue