Rob Shakir

Results 387 comments of Rob Shakir

> In Google's use case, we utilize JSON IETF format which explicitly encodes null/empty values. I don't think this is fully correct. Telemetry updates which must be `` use these...

i am supportive of adding new values to typed value to support these leaf types. i also believe that this cannot be (sanely) achieved through an extension. for the telemetry...

brainstorming: * `empty` seems clear: * such leaves are just a `bool` that can never be `false`, so using `bool_val` seems reasonable to me. * such leaves have their own...

Hi, I think this is valid YANG. Per Section 5.1 of 6020, then the prefix used in the import statement is only locally significant to the importing entity. We can...

Doesn't need to be handled in this PR -- but in reviewing this I had a thought. If we were doing MPLS over GRE then we'll specify something in the...

w.r.t > > https://github.com/openconfig/public/blob/master/release/models/aft/openconfig-aft-common.yang#L272 > > Description of tunnel-src-ip-address is confusing. It can be used to populate gre tunnel src address. can we update the description to make it specific...

I think that given that this is a bug, then it's OK to fix this directly. Thanks for the clear bug report, and thanks in advance for the fix!

Hi Adam, Some implementations enforce such a check only for eBGP. Does SROS support this for both cases? Examining JUNOS, it seems like there is similar behaviour there - so...

This is a typo, but yes - `LEVEL_1_2` means both level 1 and level 2.

Discussion from OC operators meeting (2025-06-03): * `sysuptime` in SNMP seems a reasonable guide __but__ actually tracks the "network management portion" of the system. this implies that if we `kill...