Rob Shakir

Results 387 comments of Rob Shakir

What does sending a null update mean? I don't think that this sounds a good idea -- neither does it sound like a good idea to kill RPCs (just as...

A schema-unaware collector that receives an update with `atomic = true` can optimise its storage by not individually caching paths below this prefix. i.e., if `/a/b/c` is received with `atomic...

The annotation that is used in OpenConfig models is [`telemetry-atomic`](https://github.com/openconfig/public/blob/master/release/models/openconfig-extensions.yang#L154-L165). `telemetry-on-change` is used to indicate that the data model expects that this value is sent as `ON_CHANGE` in `TARGET_DEFINED` mode...

I'm not entirely sure I understand. Technically, with `atomic = true`, I think a target is allowed (based on the current loose spec) to annotate this anywhere, and a collector...

Makes sense -- happy to have this clarification. Randomly using `atomic` is indeed undesirable, since the aforementioned optimisation (caching at the prefix that is atomic) can also result in having...

Hi David, Just to understand what happened here -- can you provide an example? Today, I don't think that any unmarshalling calls `Validate` which would be what checks leafref options...

Today, this is achieved in OpenConfig through applying a policy that manipulates the next-hop -- i.e., [`set-next-hop`](http://ops.openconfig.net/branches/models/master/docs/openconfig-routing-policy.html#routing-policy-policy-definitions-policy-definition-statements-statement-actions-bgp-actions-config-set-next-hop) allows you to set the next-hop to [`SELF`](http://ops.openconfig.net/branches/models/master/docs/openconfig-routing-policy.html#type-bgp-next-hop-type). The intention of this implementation...

Generally, our support for `bits` isn't great across ygot, since it's not a type that is used in OpenConfig (why not just use enum :-))? I'm happy to review PRs...