Re: [alto] Yangdoctors early review of draft-ietf-alto-oam-yang-06

2023-04-12 Thread Andy Bierman
On Tue, Apr 11, 2023 at 7:52 AM Jensen Zhang wrote: > Hi Andy, > > Great thanks for your thorough yangdoctors review. We have gone over all > your comments. Please see our responses inline. > > You can also check details of our proposed changes in the [GitHub > issues][1]. > > [1]: >

Re: [alto] Yangdoctors early review of draft-ietf-alto-oam-yang-06

2023-04-11 Thread Jensen Zhang
Hi Andy, Great thanks for your thorough yangdoctors review. We have gone over all your comments. Please see our responses inline. You can also check details of our proposed changes in the [GitHub issues][1]. [1]:

[alto] Yangdoctors early review of draft-ietf-alto-oam-yang-06

2023-04-03 Thread Andy Bierman via Datatracker
Reviewer: Andy Bierman Review result: Almost Ready # Review: draft-ietf-alto-oam-yang-06.txt ## module ietf-alto - General: - very well written - large module but quite understandable for implementors ### List Key Issues - there are 3 list keys that use unconstrained 'string' as the