Re: [mpls] I-D Action: draft-ietf-mpls-static-yang-08.txt

[Tarek Saad <[e-mail]>]

Hi Tom,

Thanks. We'll unify to "Mbps" and keep this consistent in units and description.

Regards,
Tarek

On 3/7/19, 7:48 AM, "tom petch" <[e-mail]> wrote:

    Tarek
    
    One point I keep noticing but do not think I have raised before is
    
          units mbps;
          description
            "Bandwidth in Mbps,
    
    I have known systems fail because one party was using mbps, another
    Mbps, so seeing a mix of the two, as here, raises a flag for me.  I note
    that
    draft-ietf-teas-yang-te-types
    has
           units "Mbps";
    which I prefer - if they really mean Mbps and not mbps; I have my
    doubts! - but then
     draft-ietf-l2sm-l2vpn-service-model-10
    has
            units "mbps";
            default "10";  description
              "LACP speed. By default, the lacp speed is 10Mbps.";
    
    Perhaps in the context of MPLS there is no ambiguity but it would be
    nice to have consistency at least within an I-D.
    
    Tom Petch
    
    ----- Original Message -----
    From: "Tarek Saad" <[e-mail]>
    Sent: Wednesday, March 06, 2019 2:53 PM
    
    
    > Hi Tom,
    >
    > As usual, thanks for your review on this one and for your patience.
    More
    > Inline..
    >
    > On Tue, Mar 5, 2019 at 12:36 PM tom petch <[e-mail]> wrote:
    >
    > > A long and winding road!
    > >
    > >  module ietf-mpls-static {
    > > .....
    > >    import ietf-routing-types {
    > >      prefix "rt-types";
    > >      reference "RFC8294: Common YANG Data Types";
    > >
    > > I see, in the RFC Index,
    > >
    > > 8294 Common YANG Data Types for the Routing Area.
    > > i.e. a different title
    > >
    > > [TS]: yes, unfortunately a deja vu (fixed in another module). We'll
    take
    > care of this in the new version.
    >
    > >
    > > module ietf-mpls-static-extended {
    > > ....
    > >   import ietf-routing {    prefix "rt";
    > >     reference "RFC6991: Common YANG Data Types";
    > > I suggest
    > >      reference "RFC8349: A YANG Data Model for Routing Management";
    > >
    > > [TS]: yes, will take of this.
    >
    >  reference "RFC XXXX: Extended YANG Data Model for MPLS Static LSPs";
    > > um - I see this I-D as titled
    > > A YANG Data Model for MPLS Static LSPs
    > >
    > > [TS]: OK for this one too.
    >
    > Regards,
    > Tarek
    >
    >
    > >
    > > Tom Petch
    > >
    > >
    > > ----- Original Message -----
    > > From: <[e-mail]>
    > > To: <[e-mail]>
    > > Cc: <[e-mail]>
    > > Sent: Saturday, March 02, 2019 8:09 PM
    > >
    > > > A New Internet-Draft is available from the on-line Internet-Drafts
    > > directories.
    > > > This draft is a work item of the Multiprotocol Label Switching WG
    of
    > > the IETF.
    > > >
    > > >         Title           : A YANG Data Model for MPLS Static LSPs
    > > >         Authors         : Tarek Saad
    > > >                           Rakesh Gandhi
    > > >                           Xufeng Liu
    > > >                           Vishnu Pavan Beeram
    > > >                           Igor Bryskin
    > > > Filename        : draft-ietf-mpls-static-yang-08.txt
    > > > Pages           : 18
    > > > Date            : 2019-03-02
    > > >
    > > > Abstract:
    > > >    This document contains the specification for the MPLS Static
    Label
    > > >    Switched Paths (LSPs) YANG model.  The model allows for the
    > > >    provisioning of static LSP(s) on Label Edge Router(s) LER(s)
    and
    > > >    Label Switched Router(s) LSR(s) devices along a LSP path
    without
    > > the
    > > >    dependency on any signaling protocol.  The MPLS Static LSP
    model
    > > >    augments the MPLS base YANG model with specific data to
    configure
    > > and
    > > >    manage MPLS Static LSP(s).
    > > >
    > > >
    > > > The IETF datatracker status page for this draft is:
    > > > https://datatracker.ietf.org/doc/draft-ietf-mpls-static-yang/
    > > >
    > > > There are also htmlized versions available at:
    > > > https://tools.ietf.org/html/draft-ietf-mpls-static-yang-08
    > > >
    https://datatracker.ietf.org/doc/html/draft-ietf-mpls-static-yang-08
    > > >
    > > > A diff from the previous version is available at:
    > > > https://www.ietf.org/rfcdiff?url2=draft-ietf-mpls-static-yang-08
    > > >
    > > >
    > > > Please note that it may take a couple of minutes from the time of
    > > submission
    > > > until the htmlized version and diff are available at
    tools.ietf.org.
    > > >
    > > > Internet-Drafts are also available by anonymous FTP at:
    > > > ftp://ftp.ietf.org/internet-drafts/
    > > >
    > > > _______________________________________________
    > > > I-D-Announce mailing list
    > > > [e-mail]
    > > > https://www.ietf.org/mailman/listinfo/i-d-announce
    > > > Internet-Draft directories: http://www.ietf.org/shadow.html
    > > > or ftp://ftp.ietf.org/ietf/1shadow-sites.txt
    > >
    > > _______________________________________________
    > > mpls mailing list
    > > [e-mail]
    > > https://www.ietf.org/mailman/listinfo/mpls
    > >
    >