search

openconfig / gnmi

gRPC Network Management Interface
Apache License 2.0
459 stars 196 forks source link

in layer2.proto file simple string could be fine instead of types.Path . It is to clear lldp interface adjacency and not for config or streaming anything out side. #133

Open jatinvd opened 1 year ago

jatinvd commented 1 year ago

from https://github.com/openconfig/gnoi/blob/main/layer2/layer2.proto

message ClearLLDPInterfaceRequest { types.Path interface = 1; } ===> this message is only meant to clear LLDP adjacency on the provided interface.

Now why to make it complicated as this is not used for any other purpose like streaming ?

can we have it as simple string?

example: message ClearLLDPInterfaceRequest { string interface = 1; }

This makes more sense i think.