RE: My review of: draft-ietf-hubmib-efm-cu-mib-06.txt

"Wijnen, Bert \(Bert\)" <[email protected]> Thu, 22 Feb 2007 10:45:29 +0100
Newsgroups gmane.ietf.hubmib
Message-ID <[email protected]>
Thank you Edward!

WG members,
As soon as the new document shows up I plan to issue another
WG Last Call, so people can check the latest changes.
Pls be prepared!

Bert=20

> -----Original Message-----
> From: Edward Beili [mailto:[email protected]]=20
> Sent: woensdag 21 februari 2007 22:54
> To: Wijnen, Bert (Bert)
> Cc: Hub Mib; Dan Romascanu (E-mail)
> Subject: RE: [Hubmib] My review of:=20
> draft-ietf-hubmib-efm-cu-mib-06.txt
>=20
> Bert,
>=20
> - RFC3410 is moved to Informative References
>=20
> - RFCs 2863, 2864, 3635, G.991.2 and G.992.1 are moved to=20
> Normative References
>=20
> - I left ANFP as an Informative Reference, since it's purpose=20
> in the MIB is to serve an example.
>=20
> The latest version of the draft is attached together with the=20
> extracted MIB files.
> I'm sending it to the internet-drafts, so it'll be published=20
> in a day or two.
>=20
> Thanks for your thorough reviews,
> -E.
>=20
>=20
> > -----Original Message-----
> > From: Wijnen, Bert (Bert) [mailto:[email protected]]
> > Sent: Wednesday, February 21, 2007 16:51
> > To: Edward Beili
> > Cc: Dan Romascanu (E-mail); Hub Mib
> > Subject: RE: [Hubmib] My review of:=20
> > draft-ietf-hubmib-efm-cu-mib-06.txt
> >=20
> > Looks good now. One thing (I should have seen yesterday=20
> too) is that=20
> > you need to move a few informative refrences to the normative=20
> > references and vice versa
> >=20
> > - I think RFC3410 is informative (it is also an informational
> >   RFC).
> >=20
> > - RFC2863 and RFC2864 are normative, because we IMPORT from those.
> >=20
> > - Since we use them in REFERENCE clauses or we use profiles
> >   from (as listed in DESCRIPTION clauses), I think that also
> >   ANFP< but certainly 991.2 and 992.1 are normative, no?
> >=20
> > - Since we state:
> >    3.4.  Relation to Ethernet-Like and MAU MIB modules
> >=20
> >    The implementation of EtherLike-MIB [RFC3635] and MAU-MIB
> >    [I-D.ietf-hubmib-rfc3636bis] is REQUIRED for the EFMCu=20
> interfaces.
> >=20
> >   We probably also better make RFC3635 a normative ref.
> >=20
> > With that I think we would be ready.
> >=20
> > Further, I would like to react to a few of Ed's rebuttals:
> >=20
> > > > - But I do want you to fix SMICng reported error:
> > > >=20
> > > >   E: f(rfc2864.mi2), (168,26) Item "ifStackGroup2" should be
> > IMPORTed
> > > >  =20
> > > >   since you do list that as a mandatory group.
> > >=20
> > > [EB] ifStackGroup2 is already imported, I've fixed that in
> > the version
> > > I sent before.
> > >=20
> >=20
> > My appology, the error is in RFC2864, not in the EFM-CU-MIB.
> >=20
> > > > > >- Did we resolve the use of Rowstatus for the ifCapStackTable
> > > > > >  and ifInvCapStackTable? In any event, pls re-check the
> > > > > >  feedback we've got on that. I do not think that what we
> > > > > >  currently have in the MIB module is acceptable.
> > > > >=20
> > > > > [EB] Replaced with TruthValue.
> > > >=20
> > > > This is much better.
> > > > I wonder if it would now be better to rename the object from=20
> > > > ifCapStackStatus to ifCapStackCapability to better=20
> represent its=20
> > > > purpose. Same for possibly renaming ifInvCapStackStatus into=20
> > > > ifInvCapStackCapability.
> > > >=20
> > > > I am not hung up on it though.
> > >=20
> > > [EB] ifCapStack already stands for "Interface Capability Stack" -=20
> > > appending "Capability" would make it "Interface Capability Stack=20
> > > Capability". How about: ifCapStackAbility ?
> > > Or we can leave it ifCapStackStatus, to emphasize its
> > similarity with
> > > IfStackTable
> > >=20
> >=20
> > Your argument for consistency with ifCapStackStatus makes sense.
> > And as I said, I am not hung up on it.
> > So I am OK now.
> >=20
> >=20
> > > [EB] I've found only one table without the persistency definition
> > > (efmCuPme10PStatusTable) and corrected it - now all tables
> > contain the
> > > persistency behavior definition in the DESCRIPTION clause for the=20
> > > table.
> > > Basically only the Status tables are non-persistent.
> > > Would that be satisfactory?
> > >=20
> >=20
> > Yep.
> >=20
> > I think we made good progress.
> >=20
> > Pls correct the references (as stated at the top of this
> > email) and then you can submit to internet-drafts as far as I am=20
> > concenrned.
> >=20
> > Next step is then (another) W Last Call to givbe anyone a chance to=20
> > look at the latest changes.
> >=20
> > Bert
> >=20
>=20