-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
docs: clarify that users should not run tofu/terraform init when usin… #4891
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
base: main
Are you sure you want to change the base?
Conversation
📝 WalkthroughWalkthroughInserted a warning in the Quick Start docs advising users not to run Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested reviewers
Pre-merge checks and finishing touches❌ Failed checks (3 warnings)
✅ Passed checks (2 passed)
✨ Finishing touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🧪 Early access (Sonnet 4.5): enabledWe are currently testing the Sonnet 4.5 model, which is expected to improve code review quality. However, this model may lead to increased noise levels in the review comments. Please disable the early access features if the noise level causes any inconvenience. Note:
Comment |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
docs-starlight/src/content/docs/01-getting-started/01-quick-start.mdx (1)
81-81: Spell out the recovery step for existing.terraformfoldersThe new warning is helpful, but readers who already ran
tofu init/terraform initneed an explicit fix—otherwise they’ll keep hitting the brokenterragrunt outputflow described in #4874. Please add a short sentence telling them to delete the existing.terraformdirectory before re-running Terragrunt.- ⚠️ Important: Do not run tofu init or terraform init yourself when using Terragrunt. Terragrunt maintains its own .terraform folder, and if a .terraform folder exists in your working directory, it can interfere with Terragrunt’s behavior. + ⚠️ Important: Do not run `tofu init` or `terraform init` yourself when using Terragrunt. Terragrunt maintains its own `.terraform` folder, and if a `.terraform` folder exists in your working directory, it can interfere with Terragrunt’s behavior. If you already ran one of those commands, delete the local `.terraform` directory before re-running Terragrunt.
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs-starlight/src/content/docs/01-getting-started/01-quick-start.mdx(1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
docs-starlight/**/*.md*
⚙️ CodeRabbit configuration file
Review the documentation for clarity, grammar, and spelling. Make sure that the documentation is easy to understand and follow. There is currently a migration underway from the Jekyll based documentation in
docsto the Starlight + Astro based documentation indocs-starlight. Make sure that thedocs-starlightdocumentation is accurate and up-to-date with thedocsdocumentation, and that any difference between them results in an improvement in thedocs-starlightdocumentation.
Files:
docs-starlight/src/content/docs/01-getting-started/01-quick-start.mdx
|
|
||
| You can now run `terragrunt` commands within the `foo` directory, as if you were using `tofu` or `terraform`. | ||
|
|
||
| ⚠️ Important: Do not run `tofu init` or `terraform init` yourself when using Terragrunt. Terragrunt maintains its own `.terraform` folder, and if a `.terraform` folder exists in your working directory, it can interfere with Terragrunt’s behavior. If you already ran one of those commands, delete the local `.terraform` directory before re-running Terragrunt. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't true? Can you confirm that it is?
Description
This section of the documentation update focuses on providing a prominent warning in the Quick Start guide advising users not to run
tofu initorterraform initwhen using Terragrunt.Summary by CodeRabbit