-
Notifications
You must be signed in to change notification settings - Fork 81
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
Register php-fpm metrics as extension plugin #451
Conversation
✅ Deploy Preview for agent-public-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
e7d6eb0
to
6e5d099
Compare
Codecov ReportPatch coverage:
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. Additional details and impacted files@@ Coverage Diff @@
## main #451 +/- ##
==========================================
- Coverage 67.19% 66.79% -0.40%
==========================================
Files 113 114 +1
Lines 12849 12951 +102
==========================================
+ Hits 8634 8651 +17
- Misses 3647 3729 +82
- Partials 568 571 +3
☔ View full report in Codecov by Sentry. |
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.
Looks good. Just one thing to note. Could you also update the extensions plugin https://github.com/nginx/agent/blob/main/src/plugins/extensions.go as well to support enabling this extension via an agent config update from the grpc AgentConfig message? You can do it in this PR or open a separate PR afterwards if you want.
Thanks. I will take care of it in separate PR. |
Proposed changes
Register php-fpm metrics as extension plugin.
Test
Agent logs with extension functionality.
Agent logs with extension functionality not enabled.
Checklist
Before creating a PR, run through this checklist and mark each as complete.
CONTRIBUTING
documentmake install-tools
and have attached any dependency changes to this pull requestREADME.md
)