Skip to content

Commit db2201a

Browse files
committed
fix so that more than just list is caught
1 parent fd50de7 commit db2201a

1 file changed

Lines changed: 23 additions & 22 deletions

File tree

scripts/pepper

Lines changed: 23 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -51,33 +51,34 @@ class Pepper(object):
5151
if HAS_SALT:
5252
logger.info('Use Salt outputters')
5353
for ret in json.loads(result)['return']:
54-
if 'data' in ret:
55-
salt.output.display_output(
56-
ret['data'],
57-
self.cli.options.output or ret.get('outputter', 'nested'),
58-
opts=self.opts
59-
)
60-
elif isinstance(ret, list):
54+
if isinstance(ret, dict):
55+
if 'data' in ret:
56+
salt.output.display_output(
57+
ret['data'],
58+
self.cli.options.output or ret.get('outputter', 'nested'),
59+
opts=self.opts
60+
)
61+
else:
62+
for minionid, minionret in ret.items():
63+
if isinstance(minionret, dict) and 'ret' in minionret:
64+
# version >= 2017.7
65+
salt.output.display_output(
66+
{minionid: minionret['ret']},
67+
self.cli.options.output or minionret.get('out', self.output),
68+
opts=self.opts
69+
)
70+
else:
71+
salt.output.display_output(
72+
{minionid: minionret},
73+
self.cli.options.output or self.output,
74+
opts=self.opts
75+
)
76+
else:
6177
salt.output.display_output(
6278
{self.cli.options.client: ret},
6379
'nested',
6480
opts=self.opts,
6581
)
66-
else:
67-
for minionid, minionret in ret.items():
68-
if isinstance(minionret, dict) and 'ret' in minionret:
69-
# version >= 2017.7
70-
salt.output.display_output(
71-
{minionid: minionret['ret']},
72-
self.cli.options.output or minionret.get('out', self.output),
73-
opts=self.opts
74-
)
75-
else:
76-
salt.output.display_output(
77-
{minionid: minionret},
78-
self.cli.options.output or self.output,
79-
opts=self.opts
80-
)
8182
else:
8283
print(result)
8384
if exit_code is not None:

0 commit comments

Comments
 (0)