docs(package): Fix incorrect logs_input.storage.directory reference in Docker Compose multi-host guide.#2076
Open
junhaoliao wants to merge 3 commits intoy-scope:mainfrom
Open
docs(package): Fix incorrect logs_input.storage.directory reference in Docker Compose multi-host guide.#2076junhaoliao wants to merge 3 commits intoy-scope:mainfrom
logs_input.storage.directory reference in Docker Compose multi-host guide.#2076junhaoliao wants to merge 3 commits intoy-scope:mainfrom
Conversation
… in Docker Compose multi-host guide.
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughDocumentation updated to reflect a configuration key rename from Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
The multi-host Docker Compose deployment guide incorrectly refers to
logs_input.storage.directorywhen instructing users to configure filesystem storage paths. The
logs_inputconfig key does nothave a
storagesub-key - itsdirectoryfield sits directly underlogs_input.The correct config structure (from
clp-config.template.json.yaml):This PR corrects
logs_input.storage.directory→logs_input.directoryin the relevant line ofthe Docker Compose deployment guide.
Checklist
breaking change.
Validation performed
1. Docs build succeeds
Task: Verify the docs site builds without errors after the one-line change.
Command:
Output:
2. Rendered page contains the corrected key
Task: Confirm the rendered HTML page shows
logs_input.directoryand no longer showslogs_input.storage.directory.Command:
Output:
Explanation: The page now correctly references
logs_input.directory.Summary by CodeRabbit