-
Notifications
You must be signed in to change notification settings - Fork 294
RFC: Metric schema metadata #871
Comments
@jcooklin This looks great. The only comment I have on it is about the source. I agree that the source should go, but shouldn't there be a field that says where the metric came from? I feel like where the metric came from is part of the identity of a metric. An example from my use cases: A CPU measurement from a file server can mean something entirely different for us then a CPU from a server running jobs. Thoughts? |
Source field in the metric has become quite the discussion topic as it has different meanings when a metric is collected on the host snapd is running on vs a metric collected through a proxy plugin. Identifying source through tags can provide a lot more meaningful metadata about where that metric was collected from that can not be easily done through a single source string field on the metric. |
The source term is overloaded and I think it means different things to different people. The data that is being collected should be uniquely identified by the namespace. In the example above you can imagine a namespace path like /path/goes/here/{hostname}/cpu. The dynamic parts of the namespace (in this example hostname) are always identified so that a plugin author can use that information. If a plugin author wanted to add meta-data (whatever that data is) they could do so via tags as @geauxvirtual suggests. It would be up to the plugin author to know what meta-data is relevant to each metric being collected. |
👍 Agreed, using tags is the great way to add a lot of resources into a plugin. It's up to the plugin authors. |
Yeah, I participated in that last discussion as well, I just think that where the metric actually came from (not the machine collecting them) is critical to identifying the metric. It sounds like it is just better to leave that up to the plugin authors though. We might want to call that out in the documentation though when we get to that point. |
This spec propose the following changes related to the metric type.
Key terms and concepts
This field does not contribute to the identity of the metric.
Use Cases
Use Cases
Proposal
plugin_running_on
The text was updated successfully, but these errors were encountered: