search

intuit / simple_deploy

Maintenance Mode - Simple Deploy is an opinionated CLI tool for managing AWS Cloud Formation Stacks.
MIT License
64 stars 22 forks source link

Add command output more details as info level messages from execute command #150

Closed weavenet closed 11 years ago

weavenet commented 11 years ago

Currently, the output is overly terse:

$ simple_deploy execute -e default -n weaver-dev -c uptime 2013-01-11 07:57:25 -0800 INFO : Setting command: 'uptime'.

$

keviny22 commented 11 years ago

How verbose should the output be?

Is this enough information or do we also want to output the specific instance name the command is being issued against as it iterates through the instances?

2013-02-07 16:26:25 -0800 INFO : Will send command: 'uptime' as kyoung to all instances. 2013-02-07 16:26:25 -0800 INFO : Will send command: 'uptime' via sudo to all instances.

weavenet commented 11 years ago

I think we need one more statement when the command is complete. When engineers ran the command it was not clear that the command was actually ran.