[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Editorial comments on draft-ietf-pwe3-arch-06.txt Section 8 (was: MIB Doctor (architectural) review needed)



On Thu, 4 Dec 2003, Wijnen, Bert (Bert) wrote:
> Document: draft-ietf-pwe3-arch-06.txt
> 
> Section 8 has a "architectural overview" of how PWE3 related
> MIBs fit in their architecture. We may want to check this out.
> 
> It is on IESG agenda today (Dec 4th)... so probably too late
> to feed any comments into the telechat today. I will take a defer
> so we as MIB doctors can do EARLY review instead of LATE review.

Sigh.  My general reaction to a first read of sections 8.1 and 8.2
was that the proposed MIB architecture seemed reasonable, but since
the devil is in often in the details, and I decided to look at the
reference documents to see how the proposed architecture is actually
realized in the draft PWE3 MIB modules, in particular in the ones
cited in the example shown in Figure 16: the PW-CEM-MIB, PW-TC-MIB,
PW-MIB, and PW-MPLS-MIB.  Alas, this exercise did not come off
entirely trouble-free:  the first problem that I ran into is that
the PW-TC-MIB and PW-MIB drafts have expired (which I got around
by retrieving old versions from the archive at
http://www.watersprings.org/pub/id/index-all.html);  the second
was that the PW-CEM-MIB no longer exists, it is now called the
PW-CEP-STD-MIB;  the third problem was that the reference
[PWMPLSMIB] points to the wrong document.

I'll have some substantive comments on the MIB architecture in
a subsequent message (mainly concerns that the relationship to
the IF-MIB is not discussed), but for now I'd like to get some
editorial comments on record.  Document authors have been cc:'d.

//cmh

Editorial comments on Section 8 of <draft-ietf-pwe3-arch-06.txt>:

Don't use of obsolete term "CEM" in Fig. 15 (Sec. 8.2.1, p.38):
---------------------------------------------------------------

OLD:

                  +-----------+ +-----------+     +-----------+
        Service   |    CEM    | | Ethernet  |     |    ATM    |
         Layer    |Service MIB| |Service MIB| ... |Service MIB|
                  +-----------+ +-----------+     +-----------+

NEW:
                  +-----------+ +-----------+     +-----------+
        Service   |   SONET   | | Ethernet  |     |    ATM    |
         Layer    |Service MIB| |Service MIB| ... |Service MIB|
                  +-----------+ +-----------+     +-----------+

Fix incorrect MIB module names in Fig. 16 (sec. 8.2.1, p.39):
-------------------------------------------------------------

OLD:
                     +-----------------+
                     |    SONET MIB    | RFC2558
                     +-----------------+
                              |
                     +-----------------+
           Service   |SONET Service MIB| pw-cem-mib
            Layer    +-----------------+
          - - - - - - - - - - | - - - - - - - - - - - - - - -
                     +-----------------+
          Generic PW | Generic PW MIBS | pw-tc-mib
            Layer    +-----------------+ pw-mib
          - - - - - - - - - - | - - - - - - - - - - - - - - -
                     +-----------------+
            PSN VC   |   MPLS VC MIBS  | pw-mpls-mib
            Layer    +-----------------+
          - - - - - - - - - - | - - - - - - - - - - - - - - -
                     +-----------------+
             PSN     |    MPLS MIBs    |   mpls-te-mib
            Layer    +-----------------+   mpls-lsr-mib

NEW:
                     +-----------------+
                     |    SONET MIB    | SONET-MIB
                     +-----------------+
                              |
                     +-----------------+
           Service   |SONET Service MIB| PW-CEP-STD-MIB
            Layer    +-----------------+
          - - - - - - - - - - | - - - - - - - - - - - - - - -
                     +-----------------+
          Generic PW | Generic PW MIBS | PW-TC-MIB
            Layer    +-----------------+ PW-MIB
          - - - - - - - - - - | - - - - - - - - - - - - - - -
                     +-----------------+
            PSN VC   |   MPLS VC MIBS  | PW-MPLS-MIB
            Layer    +-----------------+
          - - - - - - - - - - | - - - - - - - - - - - - - - -
                     +-----------------+
             PSN     |    MPLS MIBs    | MPLS-TE-STD-MIB
            Layer    +-----------------+ MPLS-LSR-STD-MIB

Fix incorrect citation/module name (sec. 8.2.2, p. 39)
------------------------------------------------------

OLD:
   adaptation MIB, in this case, it would produce the PW-CEM-MIB
______________________________________________________^^^^^^^^^^
   [PWMPLSMIB] that would be used to adapt SONET services to the
____^^^^^^^^^
   underlying PSN that carries the PWE3 service.

NEW:
   adaptation MIB, in this case, it would produce the PW-CEP-STD-MIB
   [PWCEPMIB] that would be used to adapt SONET services to the
   underlying PSN that carries the PWE3 service.

Eliminate incorrect citation (sec. 8.2.3, p. 40):
-------------------------------------------------

OLD:
   This MIB is used to interconnect the Service Layer MIBs to the PSN VC
   Layer MIBs. The latter will be described in the next section.  This
   layer also provides the PW-TC-MIB [PWTCMIB].  This MIB contains
   common SMI textual conventions [RFC1902] that MAY be used by any PW
___________________________________^^^^^^^
   MIB.

NEW:
   This MIB is used to interconnect the Service Layer MIBs to the PSN VC
   Layer MIBs. The latter will be described in the next section.  This
   layer also provides the PW-TC-MIB [PWTCMIB].  This MIB contains
   common SMI textual conventions that MAY be used by any PW MIB.

Fix incorrect references
------------------------

OLD:
   [PWMPLSMIB] Danenberg et al, "SONET/SDH Circuit Emulation Service
               Over MPLS (CEM) Management Information Base Using
               SMIv2", <draft-ietf-pwe3-cep-mib-01.txt>, work in
               progress, October 2002.

NEW:
   [PWMPLSMIB] Zelig et al, "Pseudo Wire (PW) over MPLS PSN
               Management Information Base",
               <draft-ietf-pwe3-pw-mpls-mib-03.txt>, work in
               progress, October 2003

   [PWCEPMIB]  Danenberg et al, "SONET/SDH Circuit Emulation Service
               Over Packet (CEP) Management Information Base Using
               SMIv2", <draft-ietf-pwe3-cep-mib-03.txt>, work in
               progress, October 2003.

Delete unneeded obsolete reference
----------------------------------

REMOVE:
   [RFC1902]   RFC-1902: Structure of Management Information for
               Version 2 of the Simple Network Management Protocol
               (SNMPv2), Case et al, January 1996.