[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: AD review for: draft-ietf-netconf-prot-07.txt
OK, looking forward to new revision
> -----Original Message-----
> From: Rob Enns [mailto:rpe@juniper.net]
> Sent: Wednesday, August 31, 2005 13:46
> To: Wijnen, Bert (Bert)
> Cc: Netconf (E-mail)
> Subject: RE: AD review for: draft-ietf-netconf-prot-07.txt
>
>
> Thank you for the thorough review, Bert. Unless otherwise noted, all
> edits and clarifications you raise have been made to the
> protocol draft.
>
>
> > - what is the real difference between the examples
> > in sections 6.2.3 and 6.2.4?
> > I think I understand the differences between containment
> > and seclection nodes, but the examples confuse me instead
> > of helping me, because they are exactly the same and
> > even the descritive text directly following each example
> > is exactly the same.
>
> I agree, it doesn't help much as currently written. Given that
> the filtering rules and role of the containment, selection, and
> content match nodes are laid out in detail in section 6.2.5, and
> 6.3, I think we can clean this up a little by trimming section
> 6.2.3 down and limiting it to defining what a containment
> node is. The descriptive text for the example in 6.2.3 is a
> bit misleading, because the selection node also controls what
> is included in the output, and that's not discussed until 6.2.4
> and 6.2.5. I will clean this up.
>
> > - not sure I understand (1st para sect 8) what "must be able to
> > process and ignore..." means. Maybe you can explain to me, maybe
> > it needs clarfication?
>
> The peer should ignore capabilities it does not understand.
> I think it would be clearer to rewrite this as "... and MUST
> ignore any capability received from the other peer that it
> does not require or does not understand."
>
> > - sect 8.9.5.1 example
> > Am I missing something? I do not understand "top" in that
> > example. Could be me.
>
> In the fictional data model used in the draft, "top" is the
> uppermost (or outermost) element.
>
> > - sect 10. IANA Considerations.
> > I see just: TBD
> > Well I think there are IANA considerations.
> > Like a description of the registries (namespaces) to
> > be created and to be maintained by IANA. Also rules about how
> > new entries/assignments can be made (see RFC2434).
> > Further I think that this document makes a set of registartions
> > and so they should be listed here for IANA as the initial set
> > of registrations to be administered/recorded.
>
> I will post proposed text for the IANA Considerations section
> to the WG
> list.
>
> > - I think I would change "Author's Address" into "Editor's Address"
> > on page 68. Specifically cause I get the impression that
> > Rob is editor
> > and that there are quite a set of contributing authors, which you
> > have listed separately (good).
>
> Agreed, although I haven't yet figured out how to make
> xml2rfc do this.
>
> > - Has the XML Schema been validated for correct SYNTAX?
> > Can you tell me which tool you used to do so?
>
> XMLSpy (http://www.altova.com) was used to validate the schema
> and all examples against the schema.
>
> thanks,
> Rob
>
--
to unsubscribe send a message to netconf-request@ops.ietf.org with
the word 'unsubscribe' in a single line as the message text body.
archive: <http://ops.ietf.org/lists/netconf/>