Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: create --all-terragrunt creates Terragrunt stacks with cycles. #1777

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

i4ki
Copy link
Contributor

@i4ki i4ki commented Jun 26, 2024

What this PR does / why we need it:

When Terragrunt modules are nested, Terramate generates stacks with after references to parent stacks, then creates a cycle because child stacks have an implicit dependency on parent stacks.

Which issue(s) this PR fixes:

Closes #1774

Special notes for your reviewer:

Does this PR introduce a user-facing change?

yes, fixes a bug.

Copy link

github-actions bot commented Jun 26, 2024

metric: time/op
CloudReadLines-4: old 1.01ms ± 5%: new 0.99ms ± 5%: delta: -2.07%
CloudReadLine-4: old 7.13ms ± 2%: new 7.11ms ± 1%: delta: 0.00%
ListFiles-4: old 52.9µs ± 1%: new 52.9µs ± 1%: delta: 0.00%
Generate-4: old 2.55s ± 2%: new 2.55s ± 2%: delta: 0.00%
GenerateRegex-4: old 1.70s ± 1%: new 1.72s ± 1%: delta: 1.42%
TokensForExpressionComplex-4: old 1.24ms ± 0%: new 1.24ms ± 0%: delta: 0.00%
TokensForExpressionPlainStringNoNewline-4: old 783ns ± 1%: new 782ns ± 1%: delta: 0.00%
TokensForExpressionStringWith100Newlines-4: old 21.6µs ± 1%: new 21.5µs ± 1%: delta: -0.42%
TokensForExpressionObjectWith100KeysWithNumberValues-4: old 1.44ms ± 0%: new 1.43ms ± 0%: delta: -0.12%
TokensForExpression-4: old 1.24ms ± 0%: new 1.24ms ± 0%: delta: 0.00%
PartialEvalComplex-4: old 577µs ± 0%: new 575µs ± 1%: delta: -0.29%
PartialEvalSmallString-4: old 3.59µs ± 0%: new 3.59µs ± 0%: delta: 0.00%
PartialEvalHugeString-4: old 1.87ms ± 0%: new 1.88ms ± 0%: delta: 0.15%
PartialEvalHugeInterpolatedString-4: old 4.93ms ± 1%: new 4.93ms ± 1%: delta: 0.00%
PartialEvalObject-4: old 24.3µs ± 1%: new 24.2µs ± 1%: delta: -0.38%
TmAllTrueLiteralList-4: old 6.26ms ± 0%: new 6.26ms ± 0%: delta: -0.13%
TmAllTrueFuncall-4: old 161µs ± 0%: new 160µs ± 0%: delta: -0.16%
TmAnyTrueLiteralList-4: old 147ms ± 0%: new 147ms ± 0%: delta: 0.00%
TmAnyTrueFuncall-4: old 161µs ± 0%: new 160µs ± 0%: delta: -0.16%
TmTernary-4: old 2.86µs ± 1%: new 2.83µs ± 1%: delta: -1.23%
TmTry-4: old 52.0µs ± 0%: new 51.9µs ± 0%: delta: -0.11%
metric: alloc/op
CloudReadLines-4: old 3.12MB ± 0%: new 3.12MB ± 0%: delta: 0.00%
CloudReadLine-4: old 3.37MB ± 0%: new 3.37MB ± 0%: delta: 0.00%
ListFiles-4: old 22.0kB ± 0%: new 22.0kB ± 0%: delta: 0.00%
Generate-4: old 2.32GB ± 0%: new 2.32GB ± 0%: delta: -0.00%
GenerateRegex-4: old 955MB ± 0%: new 955MB ± 0%: delta: 0.00%
TokensForExpressionComplex-4: old 404kB ± 0%: new 404kB ± 0%: delta: 0.00%
TokensForExpressionPlainStringNoNewline-4: old 512B ± 0%: new 512B ± 0%: delta: 0.00%
TokensForExpressionStringWith100Newlines-4: old 12.4kB ± 0%: new 12.4kB ± 0%: delta: 0.00%
TokensForExpressionObjectWith100KeysWithNumberValues-4: old 394kB ± 0%: new 394kB ± 0%: delta: 0.00%
TokensForExpression-4: old 404kB ± 0%: new 404kB ± 0%: delta: 0.00%
PartialEvalComplex-4: old 408kB ± 0%: new 408kB ± 0%: delta: 0.00%
PartialEvalSmallString-4: old 1.74kB ± 0%: new 1.74kB ± 0%: delta: 0.00%
PartialEvalHugeString-4: old 166kB ± 0%: new 166kB ± 0%: delta: 0.00%
PartialEvalHugeInterpolatedString-4: old 4.38MB ± 0%: new 4.38MB ± 0%: delta: 0.00%
PartialEvalObject-4: old 21.9kB ± 0%: new 21.9kB ± 0%: delta: 0.00%
TmAllTrueLiteralList-4: old 1.74MB ± 0%: new 1.74MB ± 0%: delta: 0.00%
TmAllTrueFuncall-4: old 45.5kB ± 0%: new 45.5kB ± 0%: delta: 0.00%
TmAnyTrueLiteralList-4: old 37.9MB ± 0%: new 37.9MB ± 0%: delta: 0.00%
TmAnyTrueFuncall-4: old 45.6kB ± 0%: new 45.6kB ± 0%: delta: 0.00%
TmTernary-4: old 1.20kB ± 0%: new 1.20kB ± 0%: delta: 0.00%
TmTry-4: old 11.2kB ± 0%: new 11.2kB ± 0%: delta: 0.00%
metric: allocs/op
CloudReadLines-4: old 5.54k ± 0%: new 5.54k ± 0%: delta: 0.00%
CloudReadLine-4: old 60.0k ± 0%: new 60.0k ± 0%: delta: 0.00%
ListFiles-4: old 321 ± 0%: new 321 ± 0%: delta: 0.00%
Generate-4: old 25.9M ± 0%: new 25.9M ± 0%: delta: 0.00%
GenerateRegex-4: old 18.6M ± 0%: new 18.6M ± 0%: delta: 0.00%
TokensForExpressionComplex-4: old 4.83k ± 0%: new 4.83k ± 0%: delta: 0.00%
TokensForExpressionPlainStringNoNewline-4: old 20.0 ± 0%: new 20.0 ± 0%: delta: 0.00%
TokensForExpressionStringWith100Newlines-4: old 227 ± 0%: new 227 ± 0%: delta: 0.00%
TokensForExpressionObjectWith100KeysWithNumberValues-4: old 3.19k ± 0%: new 3.19k ± 0%: delta: 0.00%
TokensForExpression-4: old 4.83k ± 0%: new 4.83k ± 0%: delta: 0.00%
PartialEvalComplex-4: old 3.65k ± 0%: new 3.65k ± 0%: delta: 0.00%
PartialEvalSmallString-4: old 23.0 ± 0%: new 23.0 ± 0%: delta: 0.00%
PartialEvalHugeString-4: old 35.0 ± 0%: new 35.0 ± 0%: delta: 0.00%
PartialEvalHugeInterpolatedString-4: old 23.1k ± 0%: new 23.1k ± 0%: delta: 0.00%
PartialEvalObject-4: old 153 ± 0%: new 153 ± 0%: delta: 0.00%
TmAllTrueLiteralList-4: old 13.6k ± 0%: new 13.6k ± 0%: delta: -0.00%
TmAllTrueFuncall-4: old 460 ± 0%: new 460 ± 0%: delta: 0.00%
TmAnyTrueLiteralList-4: old 252k ± 0%: new 252k ± 0%: delta: 0.00%
TmAnyTrueFuncall-4: old 462 ± 0%: new 462 ± 0%: delta: 0.00%
TmTernary-4: old 28.0 ± 0%: new 28.0 ± 0%: delta: 0.00%
TmTry-4: old 147 ± 0%: new 147 ± 0%: delta: 0.00%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] terramate create --all-terragrunt error with nested stacks
1 participant