-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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: Fixing some stacks docs #3887
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
WalkthroughThis pull request updates Terragrunt documentation by introducing a new CLI command Changes
Sequence Diagram(s)sequenceDiagram
participant U as User
participant C as Terragrunt CLI
participant S as Terragrunt Stack Manager
U->>C: Execute "terragrunt stack output [unit] --format [option]"
C->>S: Request aggregated or unit-specific outputs
S-->>C: Return outputs in requested format
C->>U: Display outputs
Possibly related PRs
Suggested Reviewers
Poem
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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 (2)
docs/_docs/04_reference/06-experiments.md (1)
115-116
: Consider adding descriptions for the new criteria.The new criteria "Add support for stack 'values'" and "Add support for recursive stacks" would benefit from brief descriptions explaining what these features entail and their intended use cases. This would help users understand what to expect from these features once they're stable.
Apply this diff to add descriptions:
- - [ ] Add support for stack "values". - - [ ] Add support for recursive stacks. + - [ ] Add support for stack "values" - Allow defining and sharing common values across multiple units in a stack. + - [ ] Add support for recursive stacks - Enable nesting of stacks within stacks for hierarchical organization of infrastructure.docs/_docs/04_reference/02-cli-options.md (1)
371-429
: Consider adding error handling information.The documentation for the
stack output
command is comprehensive, but it would be helpful to include information about how the command handles errors, such as:
- What happens when a unit's output cannot be retrieved
- How to handle missing outputs
- Error messages users might encounter
Add a new section about error handling:
| `raw` | Outputs key-value pairs in a compact, JSON-like format. When accessing lists or complex structures, data must be retrieved using an index-based approach. | `--format=raw` | +### Error Handling + +The `stack output` command handles errors in the following ways: +- If a unit's output cannot be retrieved, the command will display an error message and continue with the remaining units +- When accessing a non-existent output, the command will return an error +- For missing or undefined outputs, the command will indicate which outputs are unavailable
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
docs/_docs/04_reference/02-cli-options.md
(1 hunks)docs/_docs/04_reference/06-experiments.md
(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: unessential
🔇 Additional comments (3)
docs/_docs/04_reference/06-experiments.md (1)
117-119
: LGTM! Clear and comprehensive stabilization criteria.The new criteria for integration testing, parallelism compatibility, and error handling are well-defined and cover important aspects of stack functionality.
docs/_docs/04_reference/02-cli-options.md (2)
430-465
: LGTM! Clear and helpful JSON format example.The JSON format example is well-structured and provides a realistic representation of the output, making it easy for users to understand what to expect.
489-496
: LGTM! Concise raw format example.The raw format example effectively demonstrates the simplicity of accessing simple values.
Description
Fixes up some stacks docs.
TODOs
Read the Gruntwork contribution guidelines.
Release Notes (draft)
Updated stacks docs.
Summary by CodeRabbit
New Features
Documentation