Early Review of draft-ietf-rtgwg-yang-vrrp-02
review-ietf-rtgwg-yang-vrrp-02-rtgdir-early-rogge-2017-04-24-00
Request | Review of | draft-ietf-rtgwg-yang-vrrp |
---|---|---|
Requested revision | No specific revision (document currently at 11) | |
Type | Early Review | |
Team | Routing Area Directorate (rtgdir) | |
Deadline | 2017-01-12 | |
Requested | 2016-12-09 | |
Requested by | Jonathan Hardwick | |
Authors | Xufeng Liu , Athanasios Kyparlis , Ravi Parikh , Acee Lindem , Mingui Zhang | |
I-D last updated | 2017-04-24 | |
Completed reviews |
Rtgdir Early review of -02
by Henning Rogge
(diff)
Yangdoctors Early review of -01 by Radek Krejčí (diff) Genart Last Call review of -07 by Linda Dunbar (diff) Secdir Last Call review of -07 by Rich Salz (diff) Opsdir Last Call review of -08 by Zitao Wang (diff) Genart Telechat review of -08 by Linda Dunbar (diff) |
|
Assignment | Reviewer | Henning Rogge |
State | Completed | |
Request | Early review on draft-ietf-rtgwg-yang-vrrp by Routing Area Directorate Assigned | |
Reviewed revision | 02 (document currently at 11) | |
Result | Has nits | |
Completed | 2017-04-24 |
review-ietf-rtgwg-yang-vrrp-02-rtgdir-early-rogge-2017-04-24-00
Hi, Jonathan Hardwick asked me to do an early review of the draft-ietf-rtgwg-yang-vrrp document (currently revision 02) for the routing directorate. The draft itself is pretty straight forward and compact, especially when you consider that a lot of text has to be repeated two or four times (IPv4/IPv6, config vs. read-only state). But I had quite a bit of trouble mapping the phrases from the new draft-ietf-rtgwg-yang-vrrp-02 document to the existing VRRP documents (e.g. RFC5798). This might come from my unfamilarity with VRRP. The draft YANG model allows to read (if:interfaces-state) and configure (if:interfaces) virtual IP addresses, but this does not seem to be a common phrase from the RFCs. Is it the same as "address of the virtual router" often mentioned in RFC5798? In addition to this, I found (I think) a typo or inconsistency in Appendix A: the ascii art says "eth0" but tree says "eth1". Henning Rogge