Re: I-D Action: draft-ietf-rtgwg-qos-model-00.txt

[tom petch <[e-mail]>]

uh huh; some stray thoughts on this I-D

s.9
   MITRE has approved this document for Public Release, Distribution
   Unlimited, with Public Release Case Number 19-3027.

Is this the beginnings of an IPR issue to be resolved? I don't know.


YANG Model for QoS

QoS is a very general concept for me, seeeing it more in the context of
system and application performance rather than just network so I would
like something in the title to make it clear that this is 'only'
routing;  ditto - especially - the Abstract.  Current RFC have something
in the title that makes this clear.

s.1
(NMDA) [RFC8342 [RFC8342]].
looks odd with [[ ]]

YANG has conditionals which many modellers overuse (IMHO:-)
Here I would like more.  You twice have min and max and so could require
that max> or >=  min.  Is it valid to have the one value, ie min = max,
or just a min?

contact (times seven)
     contact
       "WG Web:   <http://tools.ietf.org/wg/rtgwg/>
        WG List:  <mailto:[e-mail]>
        WG Chair: Chris Bowers
                  <mailto:[e-mail]>
        WG Chair: Jeff Tantsura
                  <mailto:[e-mail]>
should not have the WG chair therein- they are too ephemeral ! - just
web, list, editors

There is a mismatch between the editors of the I-D and those of the YANG
modules; is this intended?

References in the YANG module - I would like more! e.g.
  identity one-rate-two-color-meter-type {
       base meter-type;
       description
         "one rate two color meter type";
     }
     identity one-rate-tri-color-meter-type { ...
        reference  "RFC2697: A Single Rate Three Color Marker";
     identity two-rate-tri-color-meter-type { ...
       reference    "RFC2698: A Two Rate Three Color Marker";

while some modules lack any such references for the identity; needs to
be consistent

or, still on Reference,

     grouping protocol-cfg {
       list protocol-cfg { key "protocol-min protocol-max";
         description  "list of ranges of protocol values";

which I read as being IPv4 only - IPv6 does not have protocol!  Perhaps
a name change or explanatory note plus a reference to the IANA website.


           choice drop-algorithm {
has only one case; if intended, an explanatory note would be useful


     augment "/if:interfaces/if:interface" {
is unqualified so this will be added to all interfaces in the box; I am
unclear if this is a good idea.


IANA !

Security !

s.A.1
            "RFC 6020: YANG - A Data Modeling Language ...

Tom Petch

----- Original Message -----
From: <[e-mail]>
To: <[e-mail]>
Cc: <[e-mail]>
Sent: Wednesday, October 09, 2019 11:26 PM
Subject: I-D Action: draft-ietf-rtgwg-qos-model-00.txt


>
> A New Internet-Draft is available from the on-line Internet-Drafts
directories.
> This draft is a work item of the Routing Area Working Group WG of the
IETF.
>
>         Title           : YANG Model for QoS
>         Authors         : Aseem Choudhary
>                           Mahesh Jethanandani
>                           Norm Strahle
>                           Ebben Aries
>                           Ing-Wher Chen
> Filename        : draft-ietf-rtgwg-qos-model-00.txt
> Pages           : 89
> Date            : 2019-10-07
>
> Abstract:
>    This document describes a YANG model for Quality of Service (QoS)
>    configuration and operational parameters.
>
>
> The IETF datatracker status page for this draft is:
> https://datatracker.ietf.org/doc/draft-ietf-rtgwg-qos-model/
>
> There are also htmlized versions available at:
> https://tools.ietf.org/html/draft-ietf-rtgwg-qos-model-00
> https://datatracker.ietf.org/doc/html/draft-ietf-rtgwg-qos-model-00
>
>
> 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