Skip to content
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

cmd/forwarder/run: register process metrics #467

Merged
merged 1 commit into from
Oct 17, 2023
Merged

Conversation

mmatczuk
Copy link
Contributor

Since we are using a custom registry we need to add Go and process stats. Unfortunately the process data is only available in Linux and Windows.

Since we are using a custom registry we need to add Go and process stats.
Unfortunately the process data is only available in Linux and Windows.
@what-the-diff
Copy link

what-the-diff bot commented Oct 17, 2023

PR Summary

  • Addition of Import Paths
    New code libraries have been included for use in our forwarder implementation. This would open up more features and functionalities that we can utilize in the software.

  • New Function registerProcMetrics
    A new operation called registerProcMetrics has been introduced. This function adds new capabilities to monitor and collect data on the software's performance.

  • Upgrade to runE function
    The existing operation called runE has undergone a revision to incorporate the newly added registerProcMetrics function. This allows the existing function to have extended functionalities drawn from the new operation; notably in inspecting performance metrics.

  • No change to Command function
    The Command function remains unchanged and continues to serve its original purpose in the software.

@mmatczuk
Copy link
Contributor Author

cc @amckenzie132

@mmatczuk mmatczuk merged commit 5fce40a into main Oct 17, 2023
@mmatczuk mmatczuk deleted the mmt/proc_metrics branch October 17, 2023 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants