Re: [MIB-DOCTORS] On this week IESG telechat. : Early MIB doctor review of draft-ietf-softwire-mesh-mib-04 and draft-ietf-softwire-dslite-mib

Benoit Claise <bclaise@cisco.com> Tue, 01 December 2015 10:30 UTC

Return-Path: <bclaise@cisco.com>
X-Original-To: mib-doctors@ietfa.amsl.com
Delivered-To: mib-doctors@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 3B75E1B2AFE for <mib-doctors@ietfa.amsl.com>; Tue, 1 Dec 2015 02:30:05 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.51
X-Spam-Level:
X-Spam-Status: No, score=-14.51 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id LRl7JOrBVNMI for <mib-doctors@ietfa.amsl.com>; Tue, 1 Dec 2015 02:30:01 -0800 (PST)
Received: from aer-iport-4.cisco.com (aer-iport-4.cisco.com [173.38.203.54]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D3D6E1B2AEA for <mib-doctors@ietf.org>; Tue, 1 Dec 2015 02:30:00 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=37774; q=dns/txt; s=iport; t=1448965800; x=1450175400; h=subject:to:references:cc:from:message-id:date: mime-version:in-reply-to; bh=+hFo6B3Gbcz0fHGPzz2TwfiTzKVwkALIHXTQ7x+6K4Y=; b=OcL2S+mHxDizoqiJh2/O2DNYHZMc+OUnim0AB6GwUHub6gFIhIfGOp5c BwcMnhH4LRaMxx49ho8Xg6w1R8HOOyKA3muIDu2G6kuZrljMR1cI0rtDa 6qwkH8qBfe7FJhug1/xDP+AIl4g1DC08qD7S8yil0+MCIISCIEMKrwwjq A=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0CvBABWdV1W/xbLJq1VCRaCWIEgb7wehA4iglaDFwKCDQEBAQEBAYELhDQBAQECAiMKSwEQCQIOAwQBAQoWAQECBAMCAgkDAgECAQ8lCQgGDQYCAQEFiBADEg2OIZ01jCINC4RMAQEBAQEBAQEBAQEBAQEBAQEBAQEBGIZUhH6CU4FeYIJkgUQFgU+RIINohSqGF4F3gVtJg3mDA4tqg2WDcmOBYDEdFoFBPTQBAQEBhW0BAQE
X-IronPort-AV: E=Sophos;i="5.20,368,1444694400"; d="scan'208,217";a="608637512"
Received: from aer-iport-nat.cisco.com (HELO aer-core-4.cisco.com) ([173.38.203.22]) by aer-iport-4.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 01 Dec 2015 10:29:58 +0000
Received: from [10.60.67.93] (ams-bclaise-89112.cisco.com [10.60.67.93]) by aer-core-4.cisco.com (8.14.5/8.14.5) with ESMTP id tB1ATthE011176; Tue, 1 Dec 2015 10:29:56 GMT
To: Dave Thaler <dthaler@microsoft.com>, "Romascanu, Dan (Dan)" <dromasca@avaya.com>
References: <ade96465bb8f4eaf8474dbabc07a76e7@BY2PR03MB269.namprd03.prod.outlook.com> <54758CB5.6030506@cisco.com> <565C6CA7.9040408@cisco.com> <9904FB1B0159DA42B0B887B7FA8119CA6BEAA057@AZ-FFEXMB04.global.avaya.com> <BY2PR03MB412480FE5FB1152629DDCD7A30F0@BY2PR03MB412.namprd03.prod.outlook.com>
From: Benoit Claise <bclaise@cisco.com>
Message-ID: <565D76A2.3090908@cisco.com>
Date: Tue, 01 Dec 2015 11:29:54 +0100
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0
MIME-Version: 1.0
In-Reply-To: <BY2PR03MB412480FE5FB1152629DDCD7A30F0@BY2PR03MB412.namprd03.prod.outlook.com>
Content-Type: multipart/alternative; boundary="------------030106050808080300090709"
Archived-At: <http://mailarchive.ietf.org/arch/msg/mib-doctors/dIEmclcO1MOIJGPfACFYJXk7OXs>
Cc: "MIB Doctors (E-mail)" <mib-doctors@ietf.org>, Spencer Dawkins <spencerdawkins.ietf@gmail.com>
Subject: Re: [MIB-DOCTORS] On this week IESG telechat. : Early MIB doctor review of draft-ietf-softwire-mesh-mib-04 and draft-ietf-softwire-dslite-mib
X-BeenThere: mib-doctors@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: MIB Doctors list <mib-doctors.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mib-doctors>, <mailto:mib-doctors-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mib-doctors/>
List-Post: <mailto:mib-doctors@ietf.org>
List-Help: <mailto:mib-doctors-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mib-doctors>, <mailto:mib-doctors-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 01 Dec 2015 10:30:05 -0000

Thanks Dave.

Dan, do you agree?
Shall we ask draft-ietf-softwire-mesh-mib-04 and 
draft-ietf-softwire-dslite-mib to be moved under mib-2?

Regards, Benoit
>
> Good point, and indeed in my MIB doctor review of -04, I see that I 
> had annotated that proposed assignment with a
>
> “TODO: check this”
>
> I believe it was intended to be the case (per the original ifmib WG) 
> that “transmission xxx” means “xxx” is an ifType value.
>
> Keith McCloghrie would be able to say for sure, but that’s my 
> recollection.
>
> For tunnels (of which this is one), RFC 4087 uses “transmission 131” 
> because the ifType is 131.
>
> It then defines tunnel subtypes as a separate numbering space, and 
> does not specify where in the OID
>
> hierarchy tunnel subtype MIBs should go.
>
> From http://www.iana.org/assignments/ianaiftype-mib/ianaiftype-mib the 
> current set of subtype values is:    SYNTAX     INTEGER {
>
>             other(1),        -- none of the following
>
> direct(2),       -- no intermediate header
>
> gre(3),          -- GRE encapsulation
>
> minimal(4),      -- Minimal encapsulation
>
> l2tp(5),         -- L2TP encapsulation
>
> pptp(6),         -- PPTP encapsulation
>
> l2f(7),          -- L2F encapsulation
>
> udp(8),          -- UDP encapsulation
>
> atmp(9),         -- ATMP encapsulation
>
> msdp(10),        -- MSDP encapsulation
>
> sixToFour(11),   -- 6to4 encapsulation
>
> sixOverFour(12), -- 6over4 encapsulation
>
> isatap(13),      -- ISATAP encapsulation
>
> teredo(14),      -- Teredo encapsulation
>
> ipHttps(15)      -- IPHTTPS
>
> }
>
> Checking some other docs, since each subtype need not have its own 
> MIB, only a few do:
>
> _Protocol Subtype                               MIB 
> OID                               MIB Reference_
>
> L2TP l2tp(5) ::= { transmission 95 }     [RFC3371]
>
> MSDP msdp(10) ::= { experimental 92 }   [RFC4624]
>
> Note that IANA ifType for 95 is
>
> radsl(95),          -- Rate-Adapt. Digital Subscriber Loop
>
> which is NOT L2TP, so the L2TP assignment is an anomaly.
>
> So I would say it should probably not go under transmission, since it 
> is not asking for an ifType value.
>
> What RFC 4181 actually says is:
>
>    - The value assigned to the MODULE-IDENTITY descriptor MUST be unique
>
>      and (for IETF standards-track MIB modules) SHOULD reside under the
>
>      mgmt subtree [RFC2578 <http://tools.ietf.org/html/rfc2578>].  
> Most often it will be an IANA-assigned
>
>      value directly under mib-2 [RFC2578 
> <http://tools.ietf.org/html/rfc2578>], although for media-specific
>
>      MIB modules that extend the IF-MIB [RFC2863 
> <http://tools.ietf.org/html/rfc2863>] it is customary to use
>
>      an IANA-assigned value under transmission [RFC2578 
> <http://tools.ietf.org/html/rfc2578>].  In the past,
>
>      some IETF working groups have made their own assignments from
>
>      subtrees delegated to them by IANA, but that practice has proven
>
>      problematic and is NOT RECOMMENDED.
>
> The key phrase is “that extend the IF-MIB”.   This MIB does not, it 
> extends the IP Tunnel MIB.
>
> (The IP Tunnel MIB extends the IF-MIB and so the IF Tunnel MIB does 
> use transmission.)
>
> So it would not be illegal to use transmission, since it is about 
> transmission and does indirectly
>
> (via IP Tunnel MIB) extend IF-MIB, but not directly and does not need 
> an ifType value.
>
> Dave
>
> *From:*Romascanu, Dan (Dan) [mailto:dromasca@avaya.com]
> *Sent:* Monday, November 30, 2015 7:46 AM
> *To:* Benoit Claise <bclaise@cisco.com>; Dave Thaler 
> <dthaler@microsoft.com>
> *Cc:* MIB Doctors (E-mail) <mib-doctors@ietf.org>; Spencer Dawkins 
> <spencerdawkins.ietf@gmail.com>
> *Subject:* RE: [MIB-DOCTORS] On this week IESG telechat. : Early MIB 
> doctor review of draft-ietf-softwire-mesh-mib-04 and 
> draft-ietf-softwire-dslite-mib
>
> These two documents also came on my plate at the request of IANA. The 
> reason for locating these under the transmission subtree is not clear. 
> RFC 4181 requires to do this for media-specific MIB modules – is 
> softwire considered ‘media-specific’ work?
>
> Regards,
>
> Dan
>
> *From:*MIB-DOCTORS [mailto:mib-doctors-bounces@ietf.org] *On Behalf Of 
> *Benoit Claise
> *Sent:* Monday, November 30, 2015 5:35 PM
> *To:* Dave Thaler
> *Cc:* MIB Doctors (E-mail); Spencer Dawkins
> *Subject:* [MIB-DOCTORS] On this week IESG telechat. : Early MIB 
> doctor review of draft-ietf-softwire-mesh-mib-04 and 
> draft-ietf-softwire-dslite-mib
>
> Hi Dave,
>
> Those two MIB modules are on the telechat this week, and I would like 
> to get your green light.
> Please let us know.
>
> Regards, Benoit
>
>     Hi Dave,
>
>     We are now at version 7 and 6, respectively.
>     Are you happy with the MIB modules now?
>     And may I flag this one as done at
>     http://trac.tools.ietf.org/area/ops/trac/wiki/mib-doctors-review-history
>     <https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2furldefense.proofpoint.com%2fv2%2furl%3fu%3dhttp-3A__trac.tools.ietf.org_area_ops_trac_wiki_mib-2Ddoctors-2Dreview-2Dhistory%26d%3dBQMDaQ%26c%3dBFpWQw8bsuKpl1SgiZH64Q%26r%3dI4dzGxR31OcNXCJfQzvlsiLQfucBXRucPvdrphpBsFA%26m%3dEmyVUD_iyiDij_d-DdLNVFK2XWSeKZ1aF6FTt-D5224%26s%3dpT3kgKSva5ALjadCooEr-iafVt-72eDM0plEFfsnCN8%26e%3d&data=01%7c01%7cdthaler%40microsoft.com%7c5c2e1408e814474cd23e08d2f99d7591%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=f4WZJqbumHm1ZGnM48tgMSuP4tA2VLbcUBfydJoJVP4%3d>
>
>     Regards, Benoit
>
>         Benoit asked me to do an early MIB doctor review of this
>         document.  My
>
>         full comments are in the marked up copy at
>
>         http://research.microsoft.com/~dthaler/draft-ietf-softwire-mesh-mib-04.pdf
>         <https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2furldefense.proofpoint.com%2fv2%2furl%3fu%3dhttp-3A__research.microsoft.com_-257Edthaler_draft-2Dietf-2Dsoftwire-2Dmesh-2Dmib-2D04.pdf%26d%3dBQMDaQ%26c%3dBFpWQw8bsuKpl1SgiZH64Q%26r%3dI4dzGxR31OcNXCJfQzvlsiLQfucBXRucPvdrphpBsFA%26m%3dEmyVUD_iyiDij_d-DdLNVFK2XWSeKZ1aF6FTt-D5224%26s%3dHo63Okm-fo9bjIZHdCzzwvZRa87Ac4z9r3_vn7Sy8-s%26e%3d&data=01%7c01%7cdthaler%40microsoft.com%7c5c2e1408e814474cd23e08d2f99d7591%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=6ttADpfOT8BqneSYvSroaDY4faoSF7SiQjpdX6aB2mE%3d>
>
>
>         (there’s also a .docx version if you replace the .pdf
>         extension with .docx)
>
>         The doc is in a much better state than
>         draft-ietf-softwire-dslite-mib is, and
>
>         so this was fairly easy to review (and my thanks to the
>         authors for that).
>
>         Nevertheless, I did find a few issues, a short summary of
>         which is:
>
>         1)It does not yet meet all the requirements for MIBs in RFC
>         4181, mostly around
>
>         missing boilerplate
>
>         2)The security considerations section needs (per RFC 4181) a
>         brief discussion of
>
>         what objects, if any, might be considered sensitive.  For
>         example, one might
>
>         argue the swmEncapsTable might reveal things about the
>         internal network
>
>         topology.
>
>         3)the smilint MIB checker reports several errors.  I’ve noted
>         in my comments
>
>         how to fix them.  (I used
>         http://www.ibr.cs.tu-bs.de/bin/smitools.cgi
>         <https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2furldefense.proofpoint.com%2fv2%2furl%3fu%3dhttp-3A__www.ibr.cs.tu-2Dbs.de_bin_smitools.cgi%26d%3dBQMDaQ%26c%3dBFpWQw8bsuKpl1SgiZH64Q%26r%3dI4dzGxR31OcNXCJfQzvlsiLQfucBXRucPvdrphpBsFA%26m%3dEmyVUD_iyiDij_d-DdLNVFK2XWSeKZ1aF6FTt-D5224%26s%3dvSO3FAc4-xOytETyJWMkVWpuGL92s1VT1LJllaK0tf0%26e%3d&data=01%7c01%7cdthaler%40microsoft.com%7c5c2e1408e814474cd23e08d2f99d7591%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=qixmtt%2bKMxCYecCaCiJDHq4ol2aivAxnInq%2fELTxuUk%3d>)
>
>         -Dave
>