search

Optum / dce-cli

Disposable Cloud Environment CLI
Apache License 2.0
38 stars 20 forks source link

Redirect terraform output to log file instead of console #55

Closed nathanagood closed 4 years ago

nathanagood commented 4 years ago

Is your feature request related to a problem? Please describe.

The existing trace-level logging is pretty verbose and actually could be a little unnerving (thinking as a first-time user here). On one hand, it's nice to see that it's doing something but it makes one think something might be wrong.

Describe the solution you'd like

Recommend changing the writer from the console's STDOUT for normal logging to deploy.log hidden in the .dce directory. Would be nice to put errors to deploy-err.log or something like that to make things easier to find.

Describe alternatives you've considered

An alternative would be to not print it anywhere, but that doesn't seem right.

Additional context

joshmarsh commented 4 years ago

Duplicate of https://github.com/Optum/dce-cli/issues/41

nathanagood commented 4 years ago

I'm working this and it will be included in the PR for issue #52. I do like keeping PRs more focused, but I was really bumping up against both this issue and #41 while trying to manually test my changes.