RE: [Ips] I-D ACTION:draft-ietf-ips-auth-mib-06.txt

So this is hwta I get:

   C:\bwijnen\smicng\work>smicng ipsauth.inc
   E: f(ipsauth.mi2), (53,7) Leading sub-Id "mib-2" is not known in
      current module
   W: f(ipsauth.mi2), (5,5) "experimental" imported but not used
   *** 1 error and 1 warning in parsing

When I fix that, then it passes SYNTAX checking with SMIXng
It also passes smilint syntax checking then.
It also passes idnits checking.

I think I would change:

     ipsAuthDescriptors OBJECT IDENTIFIER ::= { ipsAuthObjects 1 }

     ipsAuthMethodTypes OBJECT IDENTIFIER ::= { ipsAuthDescriptors 1 }

   into something like:

     ipsAuthDescriptors OBJECT IDENTIFIER ::= { ipsAuthObjects 1 }

     ipsAuthMethodTypes OBJECT-IDENTITY
         STATUS         current
         DESCRIPTION   "Registration point for iSCSI Method Types".
         REFERENCE     "iSCSI Protocol Specification."
     ::= { ipsAuthDescriptors 1 }

   It lets you say some more about what this is.
   Not a blocking comment though.

I see in DESCRIPTION of ipsAuthInstIndex (which is a not-accessible index)
that values do not need to be preserved over reboot.
Does that also mean that ipsAuthInstDescr values do not need to be
preserved over reboot?

A similar (not need to be preserved over reboot) is present for:
ipsAuthIdentIndex, and yet there is a StorageType object in that
table. So what if the StorageType syas "permanent" ??
Or what if it says "nonVolatile"?

By the way, for a STorageType object you MUST specify which columns
(or none) of the writable columns must be writable for permanent 
objects.

You also need to describe if any (writable) objects in row can
be changed when the row is in "active" state.

This is all described in RFC2579 and in the mib-review-guidelines.

I need to check David Harrington comments first so as to not make
duplicates.

more later
Bert
> -----Original Message-----
> From: ips-bounces@ietf.org [mailto:ips-bounces@ietf.org]On Behalf Of
> Mark Bakke
> Sent: Friday, January 28, 2005 03:39
> To: ips@ietf.org
> Cc: Michael MacFaden
> Subject: Re: [Ips] I-D ACTION:draft-ietf-ips-auth-mib-06.txt
> 
> 
> This draft was issued to address Michael MacFaden's MIB doctor review
> comments.  It also addresses a few issues (StorageType and top-level
> numbering) brought up in the iSCSI MIB review that apply here as well.
> 
> Here is a brief summary of changes since -05:
> 
> - IANA Considerations section added requesting OID
> - Text added to 4.2 to clarify that index values need not be 
> preserved 
> across reboots
> - Added the use of the StorageType TC for all tables 
> containing RowStatus
>   (this addresses a comment against the iSCSI MIB which also 
> applies here)
> - Renumbered top level to match Appendix D of 
> draft-ietf-ops-mib-review-guidelines-03.txt
> - Added DESCRIPTION text to Index attributes to say they need not be 
> preserved across reboots
> - Updated DESCRIPTIONs on RowStatus attributes
> - Added REFERENCE fields for ChapUserName, SrpUserName, and 
> KerbPrincipal attributes
> - Moved CHAP, SRP, and Kerberos references to Normative section
> 
> And a few administrative changes:
> - Authors' addresses updated
> - Authors' phone numbers removed
> - Dates updated
> - Updated IPR Notice
> - Updated first page IPR Notice
> - Updated copyright statement
> - Updated references
> 
> This MIB module passed smilint 0.4.3 with no errors.
> 
> Diffs between -05 and -06 are available at:
> 
> ftp://ftpeng.cisco.com/mbakke/ips/auth-mib/auth-mib-diffs-05-06.txt
> 
> The draft, along with a .mi2 MIB-only (no internet-draft 
> text) file are
> available at ftp://ftpeng.cisco.com/mbakke/ips/auth-mib/
> 
> Mark
> 

_______________________________________________
Ips mailing list
Ips@ietf.org
https://www1.ietf.org/mailman/listinfo/ips