-
Notifications
You must be signed in to change notification settings - Fork 0
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
ENG-38732: adding metrics config #22
Conversation
// set the frequency at which endpoint level metrics should be printed. Examples are '1s', '2m', '3h'. Default value is 30m | ||
google.protobuf.StringValue frequency = 2; | ||
// set the max number of endpoints to track | ||
google.protobuf.Int32Value max_endpoints = 3; |
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.
How will this work? The 1st maxEndpoints
number of endpoints seen in a TA will be tracked?
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.
Yes
message EndpointMetricsConfig { | ||
// set this flag to enable endpoint level metrics | ||
google.protobuf.BoolValue enabled = 1; | ||
// set the frequency at which endpoint level metrics should be printed. Examples are '1s', '2m', '3h'. Default value is 30m |
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.
Printed? or sent?
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.
right now we just print the stats in the log
@@ -16,6 +16,7 @@ message AgentConfig { | |||
// javaagent has the configs specific to javaagent | |||
Javaagent javaagent = 7; | |||
LogConfig logging = 8; | |||
MetricsConfig metrics_config = 9; |
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.
Which metrics will be covered in this metrics_config?
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.
For now, we just have sampling metrics
Adding Metrics config. As of now only sampling metrics is captured and they're just being print in the logs