chef / chef-apply

The ad-hoc execution tool for the Chef ecosystem.
https://www.chef.sh/
Apache License 2.0
14 stars 14 forks source link

Changed log level requirement for target host output #29

Closed karthanistyr closed 6 years ago

karthanistyr commented 6 years ago

Signed-off-by: Antoine Mazeas antoine@karthanis.net

Description

In the converge step, chef-apply only outputs the received log from the target host with Log.debug. This is changed to Log.info.

As discussed in the related issue (#28), the target host's output is perhaps more important than chef-apply's debug output, so this change reflects that. This also limits the noise around the target host's output.

Issues Resolved

This aims at resolving #28.

Check List

robbkidd commented 6 years ago

I'm going to opt to ignore the Expeditor error about our CI GitHub user having admin rights to the originating fork for the PR. :shipit:

robbkidd commented 6 years ago

@karthanistyr Thanks again for hunting down this behavior and contributing the improvement!