feat: -detailed-exitcode with run-all commands

[levkohimins]

Description

Fixes #1470.

For the case where a run-all is done with -detailed-exitcode flag, Terragrunt uses the following logic to return the appropriate exit code:

• If any unit throws a 1, throw a 1.
• If any unit throws a 2, but nothing throws a 1, throw a 2.
• If nothing throws a non-zero, throw a 0.

TODOs

Read the Gruntwork contribution guidelines.

• Update the docs.
• Run the relevant tests successfully, including pre-commit checks.
• Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
• Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Added / Removed / Updated [X].

Migration Guide

[yhakbar]

LGTM.

I think we need to have some documentation for indicating how this works. In a follow-up PR, could we add some notes to the run-all docs indicating that when using the -detailed-exitcode, Terragrunt will use information aggregated from all the runs to determine the final exit code, and how?

[denis256]

Unused constant 'DetailedExitCodeNoChanges'

[denis256]

Unused constant 'DetailedExitCodeChangesPresent'

grep -irn "DetailedExitCodeChangesPresent" .
./shell/detailed_exitcode.go:10:        DetailedExitCodeChangesPresent
./test/integration_test.go:130:func TestDetailedExitCodeChangesPresentAll(t *testing.T) {
./test/integration_test.go:148:func TestDetailedExitCodeChangesPresentOne(t *testing.T) {

[levkohimins]

Removed unused.

[denis256]

hm, afaik, erorrs are checked with require., other checks are through assert.

[levkohimins]

Fixed

[denis256]

Failed on pre-commit on fmt for terraform files

Terraform fmt............................................................Failed
- hook id: terraform-fmt
- exit code: 3

[levkohimins]

LGTM.

I think we need to have some documentation for indicating how this works. In a follow-up PR, could we add some notes to the run-all docs indicating that when using the -detailed-exitcode, Terragrunt will use information aggregated from all the runs to determine the final exit code, and how?

Sure, thanks. Could you please indicate where exactly in the documents this should be mentioned? Here?

[levkohimins]

Failed on pre-commit on fmt for terraform files

Terraform fmt............................................................Failed
- hook id: terraform-fmt
- exit code: 3

Fixed

[yhakbar]

Sure, thanks. Could you please indicate where exactly in the documents this should be mentioned? Here?

Ya. I'll write it up, though. Don't worry about it.

I'm seeing that this can also be worded better:

[WARNING] Using run-all with plan is currently broken for certain use cases...