-
Notifications
You must be signed in to change notification settings - Fork 3.7k
Docs: Fix documentation with correct default values and add missing options #4792
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
Merged
Merged
Changes from all commits
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
65a0ce2
update docs with correct default values and add missing options
AndreZiviani fe20fda
fix typo
AndreZiviani 9f58444
fix typo
AndreZiviani 5453a6f
add another missing option
AndreZiviani 422b018
Update docs/sources/configuration/_index.md
AndreZiviani 1a49d64
Update docs/sources/configuration/_index.md
AndreZiviani d07720d
Update docs/sources/configuration/_index.md
AndreZiviani 6d07d1d
rephrase querier_forget_delay option
AndreZiviani 9abd4ae
remove duplicated option
AndreZiviani e5cab24
rephrase max_streams_per_user option
AndreZiviani 5dc1059
fix typo
AndreZiviani bf19bbf
Update docs/sources/configuration/_index.md
AndreZiviani 439e1c2
Update docs/sources/configuration/_index.md
AndreZiviani 9cd6a8d
Merge branch 'main' into update-docs
AndreZiviani a117a8a
fix wrong values
AndreZiviani e53994c
standardize boolean naming
AndreZiviani File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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.
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.
Loki doesn't have support for querier shuffle sharding yet, I wonder if we should keep this configuration removed until it does?
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.
I don't see a problem leaving as is because its not only related to shuffle sharding but I can remove it no problem
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.
@trevorwhitney any input here?
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.
I don't have as much context as @slim-bean, are you saying this value isn't even used by loki yet because tenants don't have shards? If so I can see the point that adding this field to the docs because changing it won't actually do anything. At the same time it is a setting we have that will appear when someone hits the
/config
endpoint or prints the config to stderr, so I don't think there's too much harm in documenting it to help explain how that value got there.