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

Thanks Tom for providing your continued feedback and valuable comments. We have published revisions -24 and -25 of  ID: draft-ietf-teas-yang-te that addresses your comments. 
Please see inline below (look for [TS]).

On 4/28/20, 6:40 AM, "Teas on behalf of tom petch" <teas-bounces@ietf.org on behalf of ietfa@btconnect.com> wrote:

    From: Teas <teas-bounces@ietf.org> on behalf of tom petch <ietfa@btconnect.com>
    Sent: 27 April 2020 17:29
    Subject: Re: [Teas] TEAS WG Document Status Reports  yang-te

    <tp>
    this I-D is BIGG so I expect that I will be noticing things for months to come so here are some more

    Editors/Authors are listed in four places, top, bottom and both YANG modules and the lists differ

    IANA Considerations has no Registrant Contact for the uri
[TS]: fixed in rev -24.

    tree diagram lacks reference to RFC8340
[TS]: Addressed in rev -25.

    downstream-info
    when origin-type != egress
    Applicable to ingress LSP only
    is a mismatch - when will allow transit
[TS]: Fixed the description in rev -24.

    enum link-down
    "Link-up flooding trigger"
    Mismatch
[TS]: fixed the description in rev -24.

    advertized-level-areas
    " List of areas .. is advertised in"
    mismatch, I think the list name is wrong
    (stupid autocorrect:-)
[TS]: fixed typo in rev -24.

    1. draft-ietf-teas-yang-te-23
    Current Status:

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

      *   Authors addressed all comments in version -23 (already uploaded).
      *   Team meeting regularly to address open issues – tracked on https://github.com/tsaad-dev/te/issues
      *   Will publish new version -24 before asking for WGLC
    Open Issues:
      *   None.
    Next Steps:.

      *   Authors will followup with YANG Dr. to make sure no further comments
      *   Will request a Working Group Last Call after completing the above

    <tp>
    I think that there is quite a bit wrong with this I-D from an administrative viewpoint some of which may be more debatable than others.

    It is VERRY  BIGGG which makes it hard to digest which for me is not helped by having 52 pages of uninterrupted tree diagram at the start (or is that intended to discourage anyone from going further?).  I see other authors taking two variations.  One is to put the entire module tree diagram at the end as an appendix.  The other is to include bite size chunks with sections of text.  You sort of do that s.3 but for me there is too little tree diagram in those sections.  Look for example at draft-ietf-opsawg-l3sm-l3nm which is only 122 pages but which I think gets a better balance in its section 6 of interspersing text with tree diagram and making it possible to follow.

[TS]: thanks for the feedback. We have revised the draft in rev -24 and included new sections that describe the main parts of model and their respective bit-size chunk sub-tree. We still included the full tree in a separate section for completeness.

    Less debatably,
    - the prefix in the YANG does not match prefix in IANA Considerations.
[TS]: fixed in rev -24.

    - the Security Considerations are out of date..
[TS]: Addressed in revision -25.

    - the device module would be better In a separate section
    - s.4 lists [4872] twice
[TS]: fixed in rev -24.

Regards,
Tarek

    Tom Petch

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

      *   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
    Teas@ietf.org
    https://www.ietf.org/mailman/listinfo/teas

    _______________________________________________
    Teas mailing list
    Teas@ietf.org
    https://www.ietf.org/mailman/listinfo/teas