Subject: [PWE3] Comments on draft-ietf-pwe3-pw-tc-mib-07.txt

From: "Adrian Farrel" <[email-masked]>
Date: Wed, 12 Jul 2006 00:04:11 +0100

Hi,

The chairs just asked for comments on the MIB modules, so...

  PwGroupID ::= TEXTUAL-CONVENTION
     STATUS      current
     DESCRIPTION
          "An administrative identification mechanism for grouping a
           set of service-specific pseudo-wire services. May only
           have local significance."
     SYNTAX  Unsigned32
>> It isn't really a mechanism, is it?
>> Use of "May only" could be misinterpreted. In fact, this sentence
>> does not add much value unless you explain how I can tell the
>> scope.

  PwVlanCfg ::= TEXTUAL-CONVENTION
     STATUS      current
     DESCRIPTION
          "VLAN configuration for Ethernet PW.
           Values between 0 to 4095 indicate the actual VLAN field
           value.
           A value of 4096 indicates that the object refer to
           untagged frames, i.e. frames without 802.1Q field.
           A value of 4097 indicates that the object is not
           relevant."
     SYNTAX  Unsigned32 (0..4097)
>> s/between 0 to 4095/between 0 and 4095/

  PwOperStatusTC ::= TEXTUAL-CONVENTION
>> mismatch between description and enumeration

  PwCwStatusTC ::= TEXTUAL-CONVENTION
         notYetKnown(7) indicates that a label mapping has not yet
         received from the peer.
>> s/not yet/not yet been/

  PwCapabilities ::= TEXTUAL-CONVENTION
     STATUS      current
     DESCRIPTION
        "Indicates the optional capabilities of the control protocol.
         A value of zero indicates the basic LDP PW signaling.
         Values may be added in the future based on new capabilities
         introduced in IETF documents.
        "
    SYNTAX   BITS {
      pwStatusIndication (0)
       }
 >> Should you move this into an IAN MIB module to make it more easily 
extensible?

  PwStatus ::= TEXTUAL-CONVENTION
     STATUS      current
     DESCRIPTION
        "The status of the PW and the interfaces affecting this PW.
         If none of the bits are set, it indicate no faults are
         reported.
>> s/are set/is set/

>> Several references are not used.

Cheers,
Adrian 

_______________________________________________
pwe3 mailing list
[email-masked]
https://www1.ietf.org/mailman/listinfo/pwe3