Fixed xpath duplicate origin in XR client

[marcsello]

Using a custom origin in the sensor path wasn't split up properly causing the origin to be appended to the first element of the path which led to the following error in the subscribe call:

error {
  code: 3
  message: "Cisco-IOS-XR-infra-statsd-oper:Cisco-IOS-XR-infra-statsd-oper:infra-statistics/interfaces/interface/latest/generic-counters:Invalid sensor path\000"
}

With xpath and origin separated properly the SubscribeRequest succeeds.

Tested on on Cisco IOS-XRv 6.5.3 and IOS-XRv 6.6.2.

[remingtonc]

@marcsello thank you for the contribution. I am traveling until tomorrow afternoon and cannot debug properly until then, will be tested and merged soon! 👍
To clarify - your use case is specifying the XPath as well as an origin?

xpath = 'Cisco-IOS-XR-infra-statsd-oper:infra-statistics/interfaces/interface/latest/generic-counter'
origin = 'Cisco-IOS-XR-infra-statsd-oper' # Or another custom origin?
client.parse_xpath_to_gnmi_path(xpath, origin)

Or only specifying the XPath:

xpath = 'Cisco-IOS-XR-infra-statsd-oper:infra-statistics/interfaces/interface/latest/generic-counter'
client.parse_xpath_to_gnmi_path(xpath)

This simply clarifies in your experience whether the PathElems are expected to not be namespaced, and deserves some more testing where augmentations, etc. exist.

[marcsello]

My use case is specifying only the the XPath, prepended with the origin separated by a colon.

Here is the original code snippet I was using to test the results:

[builder setup]
client = builder.construct()

path = "Cisco-IOS-XR-infra-statsd-oper:infra-statistics/interfaces/interface/latest/generic-counters"
inflight = client.subscribe_xpaths(path)

for a in inflight:
    print(a)

The subscribe_xpaths calls client.parse_xpath_to_gnmi_path at some point with the same argument passed to it.

[remingtonc]

Thank you!