Telechat Review of draft-ietf-netmod-acl-model-19
review-ietf-netmod-acl-model-19-opsdir-telechat-clarke-2018-09-25-00
| Request | Review of | draft-ietf-netmod-acl-model |
|---|---|---|
| Requested revision | No specific revision (document currently at 21) | |
| Type | Telechat Review | |
| Team | Ops Directorate (opsdir) | |
| Deadline | 2018-09-25 | |
| Requested | 2018-09-10 | |
| Authors | Mahesh Jethanandani , Sonal Agarwal , Lisa Huang , Dana Blair | |
| I-D last updated | 2018-09-25 | |
| Completed reviews |
Yangdoctors Early review of -07
by Mahesh Jethanandani
(diff)
Rtgdir Telechat review of -19 by Manav Bhatia (diff) Secdir Last Call review of -19 by Steve Hanna (diff) Tsvart Last Call review of -19 by Allison Mankin (diff) Genart Telechat review of -19 by Meral Shirazipour (diff) Opsdir Telechat review of -19 by Joe Clarke (diff) |
|
| Assignment | Reviewer | Joe Clarke |
| State | Completed | |
| Request | Telechat review on draft-ietf-netmod-acl-model by Ops Directorate Assigned | |
| Reviewed revision | 19 (document currently at 21) | |
| Result | Has nits | |
| Completed | 2018-09-25 |
review-ietf-netmod-acl-model-19-opsdir-telechat-clarke-2018-09-25-00
I have been assigned to review this document on behalf of the ops directorate. This document defines a YANG model for configuring access control lists (ACLs) as well as other modules for protocol data types and ether types. Related to those, it requests IANA actions to register the modules. This document is ready, and I only found a few nits in it. Section 1: s/criteria allows/criteria allow/ as criteria is plural s/criteria is/criteria are/ === Section 3: s/criteria allows/criteria allows/ === Section 7: s/together to created a/together to create a/ === Section A.3: Why is this module not yang-version 1.1 like the other two?