[Gen-art] RE: Gen-ART review of draft-ietf-imss-fc-rtm-mib-03.txt
"Romascanu, Dan \(Dan\)" <dromasca@avaya.com> Thu, 27 April 2006 20:50 UTC
Received: from [127.0.0.1] (helo=stiedprmman1.va.neustar.com) by megatron.ietf.org with esmtp (Exim 4.43) id 1FZDRG-0005OP-K0; Thu, 27 Apr 2006 16:50:14 -0400
Received: from [10.91.34.44] (helo=ietf-mx.ietf.org) by megatron.ietf.org with esmtp (Exim 4.43) id 1FZDRF-0005OK-NS for gen-art@ietf.org; Thu, 27 Apr 2006 16:50:13 -0400
Received: from nj300815-ier2.net.avaya.com ([198.152.12.103]) by ietf-mx.ietf.org with esmtp (Exim 4.43) id 1FZDRF-00024V-9I for gen-art@ietf.org; Thu, 27 Apr 2006 16:50:13 -0400
Received: from IS0004AVEXU1.global.avaya.com (h135-64-105-51.avaya.com [135.64.105.51]) by nj300815-ier2.net.avaya.com (Switch-3.1.8/Switch-3.1.7) with ESMTP id k3RKnwCg005189 for <gen-art@ietf.org>; Thu, 27 Apr 2006 16:49:59 -0400
content-class: urn:content-classes:message
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
X-MimeOLE: Produced By Microsoft Exchange V6.0.6603.0
Date: Thu, 27 Apr 2006 23:50:10 +0300
Message-ID: <AAB4B3D3CF0F454F98272CBE187FDE2F0A694D79@is0004avexu1.global.avaya.com>
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
Thread-Topic: Gen-ART review of draft-ietf-imss-fc-rtm-mib-03.txt
Thread-Index: AcZqOnlAeNKH7RuVT1ef3hkUQo8VeQAAB85w
From: "Romascanu, Dan (Dan)" <dromasca@avaya.com>
To: Keith McCloghrie <kzm@cisco.com>
X-Scanner: InterScan AntiVirus for Sendmail
X-Brightmail-Tracker: AAAAAQAAAAQ=
X-Whitelist: TRUE
X-Spam-Score: 0.0 (/)
X-Scan-Signature: a92270ba83d7ead10c5001bb42ec3221
Cc: sgai@cisco.com, cds@cisco.com, General Area Review Team <gen-art@ietf.org>, sgai@ip6.com, skode@cisco.com
Subject: [Gen-art] RE: Gen-ART review of draft-ietf-imss-fc-rtm-mib-03.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www1.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www1.ietf.org/pipermail/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www1.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
Errors-To: gen-art-bounces@ietf.org
Keith, It would be better to release version 04, if you can do it in the next few days. I would like to place this document on the agenda of the 5/11 meeting. Thanks and Regards, Dan > -----Original Message----- > From: Keith McCloghrie [mailto:kzm@cisco.com] > Sent: Thursday, April 27, 2006 11:38 PM > To: Romascanu, Dan (Dan) > Cc: Keith McCloghrie; General Area Review Team; > cds@cisco.com; skode@cisco.com; sgai@cisco.com; Romascanu, > Dan (Dan); black_david@emc.com; sgai@ip6.com; spencer@mcsr-labs.org > Subject: Re: Gen-ART review of draft-ietf-imss-fc-rtm-mib-03.txt > > Dan, > > The ID-State-Tracker says that this I-D is in the "Waiting > for Writeup" > state, and thus the state diagram says that after your > writeup and "Go-Ahead", it will enter the "IESG evaluation" state. > > So, do you want me to make the changes (outlined below) now > and submit an updated draft-ietf-imss-fc-rtm-mib-04.txt, or > wait until later ?? > > Thanks, > Keith. > > > Hi, Keith, > > > > The changes you propose would would for me. > > > > Thanks especially for your proposed change to 5.3. I don't > think a lot > > of description is required, just enough to clearly identify what's > > being discussed. > > > > Spencer > > > > > > > Spencer, > > > > > > Thnaks for the comments. > > > > > >> I was selected as General Area Review Team reviewer for this > > >> specification (for background on Gen-ART, please see > > >> http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html). > > >> > > >> Summary: This document is almost ready for publication as a > > >> Proposed Standard. I do have a question on Section 5.3, > listed below. > > >> > > >> I also identified two editorial comments (not part of > the Gen-ART > > >> review for Brian). I hope this is useful. > > >> > > >> Thanks, > > >> > > >> 1. Introduction > > >> > > >> This memo defines a portion of the Management > Information Base (MIB) > > >> for use with network management protocols in the > Internet community. > > >> In particular, it describes managed objects for > information related > > >> to the Fibre Channel network's Routing Table for > routing within a > > >> Fabric. Managed objects specific to particular > routing protocols, > > >> such as FSPF, are not specified in this MIB module. > > >> > > >> Spencer (NIT): FSPF is not exanded until later in the > document (and > > >> should have the reference to [FC-SW-4] that doesn't appear until > > >> Section 4). > > >> Suggest "... such as Fabric Shortest Path First (FSPF) protocol > > >> [FC-SW-4], ..." as replacement text. > > > > > > Thanks for catching that. > > > > > >> 5.3. Fabric Index > > >> > > >> The latest standard for an interconnecting Fabric > containing multiple > > >> Fabric Switch elements is [FC-SW-4] (which replaces > the previous > > >> revision [FC-SW-3]). [FC-SW-4] specifies the > operation of both a > > >> single Fabric in a physical infrastructure, as well > as the support of > > >> multiple Virtual Fabrics operating within one (or > more) physical > > >> infrastructures. Whether operating on a physical > Fabric (i.e., > > >> without Virtual Fabrics) or within a Virtual Fabric, > the operation of > > >> FSPF within a Fabric is identical. Therefore, this > MIB defines all > > >> Fabric-related information in tables which are INDEX-ed by an > > >> arbitrary integer, named a "Fabric Index", the syntax > of which is > > >> IMPORTed from the T11-TC-MIB. When a device is > connected to a single > > >> physical Fabric, without use of any virtual Fabrics, > the value of > > >> this Fabric Index will always be 1. In an > environment of multiple > > >> virtual and/or physical Fabrics, this index provides > a means to > > >> distinguish one Fabric from another. > > >> > > >> Spencer: I can guess what a Virtual Fabric is, but I'm > guessing and > > >> the term hasn't been introduced yet, and there's no > reference for > > >> it. Not a critical problem, but since there's a nice overview in > > >> Section 3, maybe it could have a sentence or two that introduces > > >> the concept, before it appears in Section 5.3? > > > > > > Adding [FC-SW-4] as a reference is easy, and I can also > include an > > > extra paragraph to mention Virtual Fabrics at the end of > section 3. > > > I hope it will be sufficient to do so by lifting some text out of > > > FC-SW-4, even though FC-SW-4's definitions are somewhat > circular -- > > > specifically, a Virtual Fabric is defined in terms of Virtual > > > Switches, which are defined in terms of a "Core Switch" for which > > > the definition includes Virtual Fabric :-(. > > > > > >> 5.5. The t11FcRouteTable's INDEX > > >> > > >> Providing the same useful feature in the MIB in this document, > > >> results in having an unusually large number (ten) of > objects in the > > >> t11FcRouteTable's INDEX clause. However, all ten are > either integers > > >> or strings of length 0 or 3 octets. Thus, the > aggregate number of > > >> sub-identifiers to be appended to an OBJECT-TYPE's > OID when naming an > > >> instance of an object in this table, is at most 22 > sub-identifiers, > > >> i.e., less than the *minimum* number to be appended for the > > >> inetCidrRouteTable table. In other words, while ten > is an unusually > > >> large number of objects in an INDEX clause, the > resultant OIDs are > > >> not unusually large. > > >> > > >> Spencer (more than a NIT): This paragraph is really > tortured, until > > >> you get to the last sentence, which seems all that's > needed anyway > > >> (suggest "While this useful feature results in an > unusually large > > >> number (ten) of objects in the t11FcRouteTable's INDEX > clause, all > > >> ten are all ten are either integers or strings of length 0 or 3 > > >> octets, so the resulting OIDs are not unusually > large."). But the > > >> reason I flagged this as "more than a nit" was that > "length 0 or 3 > > >> octets" was confusing - the point is, "maximum length of > 3 octets", > > >> isn't it? FC people are very aware of FCAddressIdentifier > > >> structure, but no one else is, so "0 or 3 octets" is a > distraction, > > >> and the wordy paragraph just made it worse. > > > > > > Yes, that will be an improvement (minus the typo that you > caught in > > > your subsequent message). > > > > > > Thanks, > > > Keith. > > > > > > _______________________________________________ Gen-art mailing list Gen-art@ietf.org https://www1.ietf.org/mailman/listinfo/gen-art
- [Gen-art] Gen-ART review of draft-ietf-imss-fc-rt… Spencer Dawkins
- Re: [Gen-art] Gen-ART review of draft-ietf-imss-f… Spencer Dawkins
- [Gen-art] Re: Gen-ART review of draft-ietf-imss-f… Keith McCloghrie
- [Gen-art] Re: Gen-ART review of draft-ietf-imss-f… Spencer Dawkins
- [Gen-art] Re: Gen-ART review of draft-ietf-imss-f… Keith McCloghrie
- [Gen-art] RE: Gen-ART review of draft-ietf-imss-f… Romascanu, Dan (Dan)
- [Gen-art] Re: Gen-ART review of draft-ietf-imss-f… Keith McCloghrie
- [Gen-art] Re: Gen-ART review of draft-ietf-imss-f… Spencer Dawkins
- [Gen-art] Re: Gen-ART review of draft-ietf-imss-f… Keith McCloghrie
- [Gen-art] RE: Gen-ART review of draft-ietf-imss-f… Romascanu, Dan (Dan)
- [Gen-art] Gen-ART review of draft-ietf-imss-fc-rt… Spencer Dawkins
- [Gen-art] Re: Gen-ART review of draft-ietf-imss-f… Keith McCloghrie
- [Gen-art] Re: Gen-ART review of draft-ietf-imss-f… Suresh Krishnan
- [Gen-art] Re: Gen-ART review of draft-ietf-imss-f… Henrik Levkowetz
- Re: [Gen-art] (full) review of draft-ietf-imss-fc… Keith McCloghrie
- Re: [Gen-art] (full) review of draft-ietf-imss-fc… Romascanu, Dan (Dan)
- Re: [Gen-art] (full) review of draft-ietf-imss-fc… Keith McCloghrie
- Re: [Gen-art] (full) review of draft-ietf-imss-fc… Romascanu, Dan (Dan)