Re: [MIB-DOCTORS] [IETFMIBS] MIB Doctor for draft-ietf-isms-dtls-tm

["Bert Wijnen \(IETF\)" <[e-mail]>]

Thanks Wes, the changes do address my comments.

Bert
  ----- Original Message ----- 
  From: Wes Hardaker 
  To: Bert Wijnen (IETF) 
  Cc: Juergen Schoenwaelder ; Romascanu, Dan (Dan) ; MIB Doctors (E-mail) ; Sean Turner ; [e-mail] 
  Sent: Wednesday, April 14, 2010 11:02 PM
  Subject: Re: [IETFMIBS] [MIB-DOCTORS] MIB Doctor for draft-ietf-isms-dtls-tm


  >>>>> On Tue, 13 Apr 2010 22:35:44 +0200, "Bert Wijnen (IETF)" <[e-mail]> said:

  BW> I agree that the MIB module is structurally sound.

  Bert,

  I've addressed them all in the document as discussed in my responses
  below.  Thanks again for the review and please let me know if you want
  to discuss any of the items further!

  My responses are prefixed below with "WH:" if you want to search for
  them.  The status of each item is as follows:

    DONE:     Suggestion was acted on
    WONTDO:   Nothing was done; refer to the WH: response for reasons why
    DISCUSS:  There is an outstanding question back to you
    ANSWERED: A question was answered but had no action resulting from it

  Comments and responses:


  1 --- SMICng cghecking 
  ~~~~~~~~~~~~~~~~~~~~~~~

          + Bert: successfull parsing. Great!

          + WH: Thanks for checking; I've only been checking smilint

  2 DONE ----- more or less serious (maybe?): 
  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

            - tlstmParamsCount OBJECT-TYPE
                SYNTAX      Unsigned32
                MAX-ACCESS  read-only
                STATUS      current
                DESCRIPTION
                    "A count of the number of entries in the tlstmParamsTable"
                ::= { tlstmCertificateMapping 4 }

          + Bert: Is this an object for which the value can go up and
            down when connections come and go? If so, is it then not
            better to use a SYNTAX of Gaug32 ??

          + WH: That's fine by me.  I was using the Unsigned32 from an
            example somewhere else (that I don't remember) and looking
            through a number of MIBs that do similar things I see usages
            of both.  I'll switch them to Gaug32.

  3 DONE same for object tlstmAddrCount and maybe for some other 
  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
          Counts as well?

  4 DONE I see (page 31):                     Co-editors: 
  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

          + Bert: while there is only 1 editor!?

          + WH: Good point

  5 DONE I personally thought that on page 32: 
  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                 
                    This version of this MIB module is part of RFC XXXX;
                    see the RFC itself for full legal notices."

                 -- NOTE to RFC editor: replace XXXX with actual RFC number
                 --                     for this document and remove this note

                   REVISION     "201003060000Z"
                   DESCRIPTION  "The initial version, published in RFC XXXX."
                -- NOTE to RFC editor: replace XXXX with actual RFC number
                --                     for this document and remove this note
                ::= { snmpModules xxxx }

            + Bert: Probably should all just be in the REVISION DESCRIPTION
              clause and not in the MODULE DESCRIPTION clause. But I can
              live with it.  However, if a future revision does ever
              come outm then the DESCRIPTION clause will have to be
              updated also.

            + WH: Fair point; I've changed it to:

                     REVISION     "MIBDATE0000Z"
                     DESCRIPTION  "This version of this MIB module is part of
                                   RFC XXXX; see the RFC itself for full legal
                                   notices."

              -- NOTE to RFC editor: replace XXXX with actual RFC number 
              --                     for this document and change the date to the
              --                     current date and remove this note
  -- 
  Wes Hardaker                                     
  My Pictures:  http://capturedonearth.com/
  My Thoughts:  http://pontifications.hardakers.net/