One quick comment (not a proper review):

I note that the extra state module in Appendix A is redefining features defined in the main module.  This isn't generically necessarily the wrong thing to do, but in this case it looks like they are not required in the state module (since they don't appear to be used), and would be best removed.

Thanks,
Rob


On 07/03/2018 17:25, Susan Hares wrote:

Greetings I2RS folks:

This is a 2 Week WG LC to confirm the WG LC on draft-ietf-i2rs-yang-l2-network-topology.   During this WG LC, please indicate any issues with the updates to align with Network Management Data Store Module.  These may include:

1)Yang data model structure,

2)High level Yang language format

3)Security consideration section,

4)Any issues with the notification calls.

If you feel this updated data model is ready for publication, please send indicate that in your response.

Cheerily, Susan Hares



_______________________________________________
i2rs mailing list
i2rs@ietf.org
https://www.ietf.org/mailman/listinfo/i2rs

_______________________________________________
i2rs mailing list
i2rs@ietf.org
https://www.ietf.org/mailman/listinfo/i2rs

Reply via email to