-
Notifications
You must be signed in to change notification settings - Fork 57
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
Support for the gNMI extension field #503
Comments
Specifically, this is an extension of the existing I'll have a look if this could be done by just letting the user define a mapping of: This would then get decoded properly, I think, given I'm seeing |
@karimra I have a working prototype.
|
Yes for both, something like: targets:
router1:
address: x.x.x.x
#
# other fields
#
registered-extensions:
- id: 42
proto-file: /path/to/protofile |
gNMI extensions (https://github.com/openconfig/reference/blob/master/rpc/gnmi/gnmi-extensions.md) allow for extended features in the gRPC messages sent back.
The extension field is present at the top-level of the gNMI RPCs.
Juniper as a vendor is registering an extensionID for their telemetry: openconfig/gnmi#76
Right now, their prototext looks like this:
which contains a lot of helpful data, which should be exported as tags on the events.
The ask:
Add generic support for the extension field, allowing for arbitrary protobuf files to be used to decode extension IDs (a config map of ID -> protobuf file maybe?). Then use the field in processor to generate tags from it.
The specific reason for this ask:
This extension field contains linecard information of the linecard generating the response. This is necessary data to deduplicate certain events (logical interfaces spanning multiple linecards will generate an event per linecard, which will be clobbered, given they look like duplicates right now). If the extension data was exported as tags, it would be a unique event.
The text was updated successfully, but these errors were encountered: