Option --ignore-parent-terragrunt does not seem to work about terragrunt-atlantis-config HOT 3 CLOSED

Upgrading to v0.5.0 should fix your issue, as I added in the HCL parsing. Please let me know if that fix didn't solve your problem :)

from terragrunt-atlantis-config.

This is certainly an interesting bug! Thanks, @gmaghera for finding this

Why this bug happens

We currently follow this pseudocode:

files := find_all_terragrunt_files()
for file in files:
  parsed := parseFile(file)
  if the parsed file does not have a Terragrunt.Source value and --ignore-parent-terragrunt is on:
    skip this file
  else do all the dependency analysis stuff

The real life code is here: https://github.com/transcend-io/terragrunt-atlantis-config/blob/master/cmd/generate.go#L103-L112

The parseFile in that pseudocode is actually using Terragrunt's parser, imported directly from their repo.

The error you are showing comes directly from the Terragrunt parser

Why it's hard to fix this bug

If we don't use the parser, we cannot easily tell if a file is a parent terragrunt file or not. And then we don't know to skip that top level terragrunt.hcl file.

If we do use the Terragrunt parser, we get an error. This isn't so bad, except the error message is useless in this context. Terragrunt always parses locals first, so even if we could detect exactly the error in your output, we still wouldn't know if the file was a parent file or not.

Workaround

As this fix may take a bit of time to create, here is a workaround: You could change the name of the top level terragrunt.hcl file to anything else, such as parent.hcl, and then update your child path = find_in_parent_folders() expressions to be path = find_in_parent_folders("parent.hcl")

This works because this repo uses the FindConfigFilesInPath directly from Terragrunt to determine which files are terraform modules: https://github.com/gruntwork-io/terragrunt/blob/24d9a82dd5c45aa0a08d56bafb95b3ffe8eb5a30/config/config.go#L237 and that function only looks for files named terragrunt.hcl or terragrunt.hcl.json: https://github.com/gruntwork-io/terragrunt/blob/24d9a82dd5c45aa0a08d56bafb95b3ffe8eb5a30/config/config.go#L22

At my company, we do this workaround, but not because we knew of the bug, but just because we have a few differently named parent config files we switch between based on infra environment.

Actual fix

I think the only good fix here is to determine if a terragrunt file is a parent, while not parsing it with Terragrunt's parser, which means we'll need to parse the HCL on our own. I haven't done this before, but it doesn't look too awful.

from terragrunt-atlantis-config.

Thank you David for the thoughtful analysis and the suggested workaround. They both make sense.

from terragrunt-atlantis-config.