-
-
Notifications
You must be signed in to change notification settings - Fork 147
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
Expose Telemetry Metrics #415
Comments
@maennchen I'm happy to explore opening a PR for this. Is this something you have started? if not, Is there any chance you could give me a shove in the right direction as far as a high level plan? You know this project better than anyone so a little bit of help from you could save me a LOT of time. |
@maennchen if you have time, a quick reply could help me get started on this. |
@mrmarcsmith Sorry, i don't really have time right now. I'm on vacation until next week. It'd be awesome to get a PR with this feature. I have thought about this at some point. I thought that at least the following events should be recorded:
The first three should probably be added to the JobBroadcaster module and the execution in the Executor module. |
That should be enough to get me started! Thank you for replying while on vacation. |
Replaced by #453 |
@mrmarcsmith Released as |
Thank you @maennchen! |
Expose Metrics via Telemetry
The text was updated successfully, but these errors were encountered: