Re: [Teas] teas-yang-rsvp wasRe: TEAS WG Document Status Reports

[tom petch <[e-mail]>]

From: Tarek Saad <[e-mail]>
Sent: 27 July 2020 13:29

Hi Tom,

Thanks for your review of ID: draft-ietf-teas-yang-rsvp and for the your comments. We have posted version https://tools.ietf.org/html/draft-ietf-teas-yang-rsvp-14
Which addresses those comments. 

<tp>
Mostly:-)
You still have  a prefix mismatch between module and IANA.

I suspect you will get asked for examples at some stage 

s.3.1 The augments diagrams use 'o' to show augmentation.  Other I-D use '>' but use it in different senses.  My take is 
A----->B
means A augments B.

3.3 I think that filters need references.  Subtree is 6241 but XPATH is probably 8639 since that specifies the context thereof which is not obvious.

Tom Petch









Please see inline (look for TS) for more details.

>>>>>>>>>>

    2. draft-ietf-teas-yang-rsvp-11
    Current Status:

    <tp>
    teas-yang-rsvp-12 would benefit from some tweaks
    Abstract confused me into thinking that the Extended model was elsewhere - I think that this needs an explicit mention.
TS: revised abstract to address this.


    Introduction ditto.
TS: revised Introduction to address this.

    Introduction should mention RFC7950 and RESTCONF

    Introduction should mention NMDA support
TS: Addressed.

    1 is wrong
    rt-type I am not familiar with but rt-types is in RFC8294
    key-chain is RFC8177
TS: corrected.

    2.4 lists two RPC but tree diagram lists three, one of which is authentication but authentication is a feature of extended not base - this confuses me
TS: corrected and added the third.

    revision description
    should be 'Initial version' in both modules

    the rpc should probably be NACM default deny all
TS: added.

    IANA Considerations
    URI lack registrant contact
    prefix are not those used in the modules
TS: updated.

    Security
    alsocontrols
TS: corrected.

    RFC5495 needs adding to I-D References
TS: updated references.

Again, thanks much for the comments.

Regards,
Tarek

    Tom Petch






       *   Version -10 was reviewed by YANG Dr. and received comments

      *   Authors published version -12 which addressed all outstanding comments
      *   Authors followed up with YANG Dr. reviewer on closing comments
    Open Issues:

      *   None
    Next Steps:

      *   Proceed to WGLC


    3. draft-ietf-teas-yang-rsvp-te-07
    Current Status:

      *   Version -07 was reviewed by YANG Dr. and received comments
    Open Issues:

      *   Authors working on addressing YANG Dr. comments
    Next Steps:

      *   Folllow-up with YANG Dr. reviewer to close on comments
      *   Proceed to WGLC after addressing

    4. draft-ietf-teas-yang-te-mpls
    Current Status:

      *   Authors discussing augmentation of this module for MPLS-TP
      *   A new module and model is expected
    Open Issues:

      *   Close on MPLS-TP modeling
    Next Steps:

      *   Publish a new revision
      *   Ask for YANG Dr. review



    _______________________________________________
    Teas mailing list
    [e-mail]
    https://www.ietf.org/mailman/listinfo/teas