support for namespaced Path Elements

[hellt]

Hi all,
in ParseElem the code doesn't allow for a pathElem to have a : char inside of it.

I wonder if this should be allowed to make the path to look life follows: /root/ns:leaf. Some vendors use namespaced paths in their gNMI implementations, and I am not sure if this violates the gNMI spec or not.

@robshakir do you have an opinion if the gNMI paths like /root/ns:leaf are valid and should be supported?

[hellt]

Hi, bumping
any opinions on this?

[samribeiro]

Hi @hellt, I am not sure about this myself. I dont see any mention of it in the gNMI documentation:
https://github.com/openconfig/reference/blob/master/rpc/gnmi/gnmi-path-conventions.md

Perhaps opening an issue in https://github.com/openconfig/public asking for clarification is a better approach?

[hellt]

Thanks for commenting, @samribeiro

I am not sure if that has to be clarified with OC group. The : is not restricted by the very same path conventions document.

Also in the reference gnmi_cli client the team uses another parsing approach - https://github.com/openconfig/gnmi/blob/master/cmd/gnmi_cli/gnmi_cli.go#L310
https://github.com/openconfig/gnmi/blob/d2b4e6a45802a75b3571a627519cae85a197fdda/client/gnmi/client.go#L255

I will check that method.

Feel free to close this issue if you decide not to account for : in paths