-
Notifications
You must be signed in to change notification settings - Fork 7
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
Engine: Implement Awk mapping #343
Comments
Instead of void pushMetric(String name, AssocArray attributes, Double value) { /* push metric to telemetry manager */ } This way, there is no need to parse the output of the AWK script and handle syntax errors. |
Also, please note that we can't provide multiple sources to AWK (because they are not input files, but just |
I'm thinking: if we implement this Instead we could call the The YAML then becomes: memory:
keys: [id] # key to identify the monitor instance
simple:
sources:
source1:
# run command
compute:
- type: awk
script: |
/^Mem:/ {
attrs["id"] = $3
attrs["system.memory.state"] = "free"
pushMetric("system.memory.usage", attrs, $4);
}
mapping:
type: none # yes, none! This mapping section becomes totally optional!
# Manifests the mapping of the metrics for the documentation (Optional)
metrics:
- system.memory.usage{system.memory.state="free"}
- system.memory.limit |
I'm currently considering this proposal: MetricsHub manages attributes both at the monitor level and at the metric level itself. We know that before metrics are pushed, monitor attributes and metric attributes are merged, with everything applied to the metric. If we consider the current MetricsHub model, perhaps we should update the extension to define |
Required connector model
The monitor identified should be managed using the
keys
field and the attributes defined in the metric outputs (Output of the awk script).The text was updated successfully, but these errors were encountered: