Thank you. Those changes would address my concerns.
Yours,
Joel
Padma Pillay-Esnault (ppe) wrote:
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
