-
Notifications
You must be signed in to change notification settings - Fork 486
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
docs: add a tenant example to prometheus.remote_write.md #5737
Conversation
This commit adds an example for pushing the metrics to a multi-tenant Mimir instance with non-default tenant. It could be useful for people who have less experience with multi-tenant Mimir, but would like to have a clear example how to use recommended way for ingestion there.
docs/sources/flow/reference/components/prometheus.remote_write.md
Outdated
Show resolved
Hide resolved
@grafana/grafana-agent-maintainers Can one of you take a quick look over the example to double check the technical accuracy |
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.
From a technical perspective looks good, just one comment around having a more generic url and comment 😃
docs/sources/flow/reference/components/prometheus.remote_write.md
Outdated
Show resolved
Hide resolved
Add a missing article. Co-authored-by: Clayton Cornell <[email protected]>
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.
Just a minor nit but LGTM, thank you!
docs/sources/flow/reference/components/prometheus.remote_write.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Mischa Thompson <[email protected]>
@thecoldwine Thanks for your input here. Getting this PR merged into Main :-) |
* Add a tenant example to prometheus.remote_write.md This commit adds an example for pushing the metrics to a multi-tenant Mimir instance with non-default tenant. It could be useful for people who have less experience with multi-tenant Mimir, but would like to have a clear example how to use recommended way for ingestion there. * Update docs/sources/flow/reference/components/prometheus.remote_write.md Add a missing article. Co-authored-by: Clayton Cornell <[email protected]> * formatting + more generic Mimir url in the snippet * A minor nit Co-authored-by: Mischa Thompson <[email protected]> --------- Co-authored-by: Clayton Cornell <[email protected]> Co-authored-by: Mischa Thompson <[email protected]> (cherry picked from commit 1caf600)
* Add a tenant example to prometheus.remote_write.md This commit adds an example for pushing the metrics to a multi-tenant Mimir instance with non-default tenant. It could be useful for people who have less experience with multi-tenant Mimir, but would like to have a clear example how to use recommended way for ingestion there. * Update docs/sources/flow/reference/components/prometheus.remote_write.md Add a missing article. Co-authored-by: Clayton Cornell <[email protected]> * formatting + more generic Mimir url in the snippet * A minor nit Co-authored-by: Mischa Thompson <[email protected]> --------- Co-authored-by: Clayton Cornell <[email protected]> Co-authored-by: Mischa Thompson <[email protected]> (cherry picked from commit 1caf600)
* Add a tenant example to prometheus.remote_write.md This commit adds an example for pushing the metrics to a multi-tenant Mimir instance with non-default tenant. It could be useful for people who have less experience with multi-tenant Mimir, but would like to have a clear example how to use recommended way for ingestion there. * Update docs/sources/flow/reference/components/prometheus.remote_write.md Add a missing article. Co-authored-by: Clayton Cornell <[email protected]> * formatting + more generic Mimir url in the snippet * A minor nit Co-authored-by: Mischa Thompson <[email protected]> --------- Co-authored-by: Clayton Cornell <[email protected]> Co-authored-by: Mischa Thompson <[email protected]> (cherry picked from commit 1caf600)
* Add a tenant example to prometheus.remote_write.md This commit adds an example for pushing the metrics to a multi-tenant Mimir instance with non-default tenant. It could be useful for people who have less experience with multi-tenant Mimir, but would like to have a clear example how to use recommended way for ingestion there. * Update docs/sources/flow/reference/components/prometheus.remote_write.md Add a missing article. Co-authored-by: Clayton Cornell <[email protected]> * formatting + more generic Mimir url in the snippet * A minor nit Co-authored-by: Mischa Thompson <[email protected]> --------- Co-authored-by: Clayton Cornell <[email protected]> Co-authored-by: Mischa Thompson <[email protected]> (cherry picked from commit 1caf600) Co-authored-by: Mikhail Aksenov <[email protected]>
* Add a tenant example to prometheus.remote_write.md This commit adds an example for pushing the metrics to a multi-tenant Mimir instance with non-default tenant. It could be useful for people who have less experience with multi-tenant Mimir, but would like to have a clear example how to use recommended way for ingestion there. * Update docs/sources/flow/reference/components/prometheus.remote_write.md Add a missing article. Co-authored-by: Clayton Cornell <[email protected]> * formatting + more generic Mimir url in the snippet * A minor nit Co-authored-by: Mischa Thompson <[email protected]> --------- Co-authored-by: Clayton Cornell <[email protected]> Co-authored-by: Mischa Thompson <[email protected]> (cherry picked from commit 1caf600) Co-authored-by: Mikhail Aksenov <[email protected]>
* Add a tenant example to prometheus.remote_write.md This commit adds an example for pushing the metrics to a multi-tenant Mimir instance with non-default tenant. It could be useful for people who have less experience with multi-tenant Mimir, but would like to have a clear example how to use recommended way for ingestion there. * Update docs/sources/flow/reference/components/prometheus.remote_write.md Add a missing article. Co-authored-by: Clayton Cornell <[email protected]> * formatting + more generic Mimir url in the snippet * A minor nit Co-authored-by: Mischa Thompson <[email protected]> --------- Co-authored-by: Clayton Cornell <[email protected]> Co-authored-by: Mischa Thompson <[email protected]> (cherry picked from commit 1caf600) Co-authored-by: Mikhail Aksenov <[email protected]>
PR Description
This PR adds an example for pushing the metrics to a multi-tenant Mimir instance with non-default tenant.
It could be useful for people who have less experience with multi-tenant Mimir, but would like to have a clear example how to use recommended way for ingestion there.
Notes to the Reviewer
There are no functional changes, just one particular example in the
prometheus.remote_write
config.PR Checklist