Modify output banners to have constant length and use dashes instead

of "-" to be less visually heavy (unless cowsay is installed of course)
pull/2592/merge
Michael DeHaan 12 years ago
parent e9d11173cd
commit bc0be456da

@ -39,12 +39,12 @@ def colorize(lead, num, color):
def hostcolor(host, stats): def hostcolor(host, stats):
if ANSIBLE_COLOR: if ANSIBLE_COLOR:
if stats['failures'] != 0 or stats['unreachable'] != 0: if stats['failures'] != 0 or stats['unreachable'] != 0:
return "%-41s" % stringc(host, 'red') return "%-37s" % stringc(host, 'red')
elif stats['changed'] != 0: elif stats['changed'] != 0:
return "%-41s" % stringc(host, 'yellow') return "%-37s" % stringc(host, 'yellow')
else: else:
return "%-41s" % stringc(host, 'green') return "%-37s" % stringc(host, 'green')
return "%-30s" % host return "%-26s" % host
def main(args): def main(args):
@ -184,7 +184,7 @@ def main(args):
playbook_cb.on_stats(pb.stats) playbook_cb.on_stats(pb.stats)
for h in hosts: for h in hosts:
t = pb.stats.summarize(h) t = pb.stats.summarize(h)
print "%-30s : %s %s %s %s " % ( print "%s : %s %s %s %s" % (
hostcolor(h, t), hostcolor(h, t),
colorize('ok', t['ok'], 'green'), colorize('ok', t['ok'], 'green'),
colorize('changed', t['changed'], 'yellow'), colorize('changed', t['changed'], 'yellow'),

@ -143,7 +143,11 @@ def banner(msg):
(out, err) = cmd.communicate() (out, err) = cmd.communicate()
return "%s\n" % out return "%s\n" % out
else: else:
return "\n%s ********************* " % msg width = 78 - len(msg)
if width < 3:
width = 3
filler = "-" * width
return "\n%s %s " % (msg, filler)
def command_generic_msg(hostname, result, oneline, caption): def command_generic_msg(hostname, result, oneline, caption):
''' output the result of a command run ''' ''' output the result of a command run '''

Loading…
Cancel
Save