-
Notifications
You must be signed in to change notification settings - Fork 587
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(docs): update grafana-agent example #1963
Conversation
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.
LGTM!
I guess we should not merge until the agent is released |
Flamegraph.com reportCreated by Flamegraph.com Github Action |
size-limit report 📦
|
Flamegraph.com reportCreated by Flamegraph.com Github Action |
Codecov Report
@@ Coverage Diff @@
## main #1963 +/- ##
=======================================
Coverage 60.50% 60.50%
=======================================
Files 177 177
Lines 4991 4991
Branches 1157 1157
=======================================
Hits 3020 3020
Misses 1959 1959
Partials 12 12 |
I think its fine to merge since it's a little hidden for now (we don't link to it in docs yet) grafana agent should be updated soon |
remove phlare references in agent flow configuration
depends on grafana/agent#4013