-
Notifications
You must be signed in to change notification settings - Fork 477
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
Update ingester.md #4827
Update ingester.md #4827
Conversation
minor changes for clarity
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.
Needs doc review
the changes seem ok and correct, but I'm not sure if we want to break the consistency with the other components - the title is Also if we do decide to go with the new format, it will be great if we don't break existing links. |
docs/sources/mimir/operators-guide/architecture/components/ingester.md
Outdated
Show resolved
Hide resolved
docs/sources/mimir/operators-guide/architecture/components/ingester.md
Outdated
Show resolved
Hide resolved
docs/sources/mimir/operators-guide/architecture/components/ingester.md
Outdated
Show resolved
Hide resolved
docs/sources/mimir/operators-guide/architecture/components/ingester.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Ursula Kallio <ursula.kallio@grafana.com>
Thanks for contributing to doc! I committed the Ursula's edits to move forward with this PR. @osg-grafana can you take a final look, please? |
minor changes for clarity
What this PR does
Which issue(s) this PR fixes or relates to
Fixes #
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]