Hi Robert, Sorry for the late reply and thanks a lot for your comments. We will make the changes to the state module in next revision.
Best regards, Jie From: Robert Wilton [mailto:rwil...@cisco.com] Sent: Thursday, March 08, 2018 1:53 AM To: Susan Hares <sha...@ndzh.com>; i2rs@ietf.org; draft-ietf-i2rs-yang-l2-network-topol...@ietf.org Subject: Re: [i2rs] WG LC confirmation on draft-ietf-i2rs-yang-l2-network-topology (3/7 to 3/21/2018) 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<mailto:i2rs@ietf.org> https://www.ietf.org/mailman/listinfo/i2rs
_______________________________________________ i2rs mailing list i2rs@ietf.org https://www.ietf.org/mailman/listinfo/i2rs
