Tags: alxrem/terragrunt
Tags
Fix empty outputs (gruntwork-io#1568) If stack run finished without errors, `summarizePlanAllErrors()` receives empty buffer and outputs empty line. This change ensures that only non-empty outputs are getting logged. Related: gruntwork-io#1541
automatically chomp newlines when using run_cmd (gruntwork-io#1553)
Change default logging level to Info (gruntwork-io#1541) * Change default logging level to Info Recent PR (gruntwork-io#1510) changed logging, and also introduced default logging level. While it seemed like a good idea, it also introduced a lot of confusion, because in logrus `Println()` is an alias to `Info()` loglevel - so lots of messages were lost. This change restores previous behavior in logging output. Related: gruntwork-io#1529 Related: gruntwork-io#1530 Related: gruntwork-io#1531 Related: gruntwork-io#1532 * Use `Infof()` instead of `Printf()` `Printf()` is an alias, so this does not change any functionality - just makes logging more explicit. * Update log levels according to PR reviews * Some minor fixes after PR review * Use same default logLevel across different modules This also updates integration tests to catch this error. * Log both prompt and a assume message for better visibility
Faster dependency outputs (gruntwork-io#1533) * Refactor out TerraformSource struct and functions so we can use it in config package without circular dependency * [skip ci] wip * Add regression testing for dependency optimization * Update cli/tfsource/types.go Co-authored-by: Yevgeniy Brikman <brikis98@users.noreply.github.com> * Add comment indicating what would happen if the ref changes Co-authored-by: Yevgeniy Brikman <brikis98@users.noreply.github.com>
Write prompts to ErrWriter instead of log (gruntwork-io#1526) * Write prompts to ErrWriter instead of log Also added small test to integration suit. Closes: gruntwork-io#1524 * Provide more details about the change
Introduce run-all to replace xxx-all (gruntwork-io#1520) * Introduce run-all to replace xxx-all * Fix bug in terraformcliargs manipulation with run-all update * Fix regression bug with xxx-all where OriginalTerraformCommand was not set correctly * Consolidate deprecated commands handler * Track a list of commands that should never be run with run-all * Make sure to pass -input=false to all commands that require them * Update cli/cli_app.go Co-authored-by: Yevgeniy Brikman <brikis98@users.noreply.github.com> * Add state command to promptable run-all * Fix bugs from logging changes * Fix more issues from logging Co-authored-by: Yevgeniy Brikman <brikis98@users.noreply.github.com>
Option for less verbose output (gruntwork-io#1510) * Option for less verbose output This moves majority of Terragrunt's output to DEBUG level, and sets default output mode to ERROR. The idea is that in default run user should see much less output. Closes: gruntwork-io#432 * Updated after review Apart from small fixes, this: - adds deprecation check for command line arguments, and gives a warning if `--terragrunt-debug` is used; - imports `logrus` under its own name to avoid confusion; Related: gruntwork-io#432 * Updates after review | 2 Main change - global LogEntry object introduced in `util/logger.go`. It helps us log properly when we didn't yet parse command line arguments * Fix tests for `cli/cli_app.go` P.S. Tests rock! * Rework global logger Global logger should be a fallback option, rather a way to go. So instead of using the same everywhere, we use global if no other logger is available (for example, if we have not yet parsed arguments). This change also addresses other PR review comments, like: - wrong documentation; - using `logrus.WarnLevel.String()` instead of hardcoded strings; - updated `isDeprecatedOption()`, with a more clear usage; * Fix typo in docs section * Apply `go fmt` properly * Apply `go fmt` properly | 2 * Fix `TestParseTerragruntOptionsFromArgs` test * Terragrunt should output all log to stderr * Set stderr as default output * Update TestApplySkipTrue() Add `--terragrunt-log-level info`, so that we could test it. * Fix tests This fixes tests that rely on double output of `run_shell`, as well as brings back `--terragrunt-debug`, but in slightly different form: to only generate `terragrunt-debug.tfvars.json` * Remove `--terragrunt-debug` from excluded and fix `\n` Terragrunt integration tests depend on newlines, and to ensure that logrus can properly print newlines with `\n` `DisableQuote: true` is needed [1]. In particular, this fixes `TestIncludeDirsDependencyConsistencyRegression` and `TestIncludeDirsStrict`. [1] - sirupsen/logrus#608 * Bring `--terragrunt-debug` tests back
Fix formatting bug in aws-provider-patch (gruntwork-io#1519) * [skip ci] Add regression test for aws-provider-patch bug * Run regression test that should fail * Bump hclv2 version * Bump terraform to 0.14.4 in container * Fix build binary step to be compatible with unsupported os_arch combo
PreviousNext