Ebben Aries

> @LimeHat @earies @rgwilton -- do you have thoughts here? Will revert back shortly - quick passthrough appears to not align well to existing support within our implementation as proposed

Just catching up on this PR now... > [JunOS](https://www.juniper.net/documentation/us/en/software/junos/interfaces-security-devices/cli-reference/topics/ref/statement/load-interval-edit-interfaces.html) is at 180 max in 10 second increments. Note: This is an invalid reference for current JUNOS/EVO platforms (very old EOL...

> > * It appears we are bringing a long standing feature that's use has primarily been on-box diagnosis (run a command to give the operator an idea of calculated...

Quick lazy audit and we have many more.... ``` Mismatched prefix: module [openconfig-acl], import [openconfig-packet-match]. Expected "oc-pkt-match", Got: "oc-match" Mismatched prefix: module [openconfig-packet-match], import [openconfig-mpls-types]. Expected "oc-mplst", Got: "oc-mpls" Mismatched...

> Just my $0.02. This is something that tooling has to be able to handle. It's not something in the style guide. We already had to handle this in most...

By no means mandatory or a big deal - it's a `SHOULD` for best practice is all and reason why strict linters likely won't flag ``` To improve readability of...

> It looks like that some checks are complaining about the deref I use in path. Is deref not support in path? Pyang doesn't give me any error on that....

> * There's no expectation that a client can set a `sample_interval` in a `TARGET_DEFINED` subscription mode. The gnmi.proto specifies that `sample_interval` is used in `SAMPLE` mode ([ref](https://github.com/openconfig/gnmi/blob/891c5a8cdcd670528157283cde220adb0f43f63e/proto/gnmi/gnmi.proto#L303)). Suggestion for...

gNMI was not designed to encompass the full suite of the YANG language and constructs that have definition, meaning and relationships for other NBI protocols such as NETCONF but rather...

Maybe we should start w/ a show of hands as to any implementation that supports `use_models` and tag this field for deprecation across the respective RPCs. To my knowledge there...