Hello Joel Thank you for your review. Please see below for responses.
> -----Original Message----- > From: Joel M. Halpern [mailto:[email protected]] > Sent: Friday, October 16, 2009 3:33 PM > To: General Area Review Team > Cc: [email protected]; Ross Callon > Subject: Re: [Gen-art] Review: draft-ietf-l3vpn-ospfv3-pece-03 > > I have been selected as the General Area Review Team (Gen-ART) > reviewer for this draft (for background on Gen-ART, please see > http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html ). > > Please resolve these comments along with any other Last Call comments > you may receive. > > Document: draft-ietf-l3vpn-ospfv3-pece-03 > OSPFv3 as a PE-CE routing protocol > Reviewer: Joel Halpern > Review Date: 16-Oct-2009 > IETF LC End Date: 23-Oct-2009 > IESG Telechat date: N/A > > Summary: This document is almost ready for publication as a Proposed > Standard. > > Major issues: - > > Minor issues: > "The OSPFv3 Domain ID is a 6-byte number and its default value > if none is configured should be NULL." > NULL is used multiple times in the spec talking about this value. > But, the field is a fixed length field. The only conclusion I can come > to is that NULL means all-zeros. But the text does not say that. > Please say it. [PPE] Correct. The text can be made more explicit with the following change "The OSPFv3 Domain ID is a 6-byte number and its default value is 0." > > The OSPFv3 Router ID fields is described as "OPTIONAL." It is > optional in the sense that it may be set to 0, not in the sense that it > may be omitted. I am not at all sure "OPTIONAL" is the right word. [PPE] Correct for the interpretation. The text can be changed from "The OSPFv3 Router ID is a 32 bit number as in OSPFv2. This field is OPTIONAL and may be set to 0." To "The OSPFv3 Router ID is a 32 bit number as in OSPFv2. Setting this field is OPTIONAL and its default value is 0. Thanks Padma > > Nits/editorial comments: _______________________________________________ Gen-art mailing list [email protected] https://www.ietf.org/mailman/listinfo/gen-art
