ietf
[Top] [All Lists]

Re: Gen-ART LC review of draft-ietf-ccamp-rfc5787bis-05

2012-10-09 11:12:34
Roni,

Thanks for your review, and sorry for the delay on the response, but we've also 
been working on incorporating other changes to the draft as well.

To answer your question on section 6.1, that was a good catch. That should have 
said "other than the first", rather than "preceding the first". This will be 
corrected.

To answer your question on section 10, the text is repeated in the subsections 
to make life easier for IANA, since they would probably have replicated it 
themselves anyway in the three registry listings.

On your last comment, we agree with you that the more familiar reader will not 
have a problem.

Thanks again,
Andy

From: Roni Even 
<ron(_dot_)even(_dot_)tlv(_at_)gmail(_dot_)com<mailto:ron(_dot_)even(_dot_)tlv(_at_)gmail(_dot_)com>>
Date: Monday, August 13, 2012 14:07
To: 
"draft-ietf-ccamp-rfc5787bis(_dot_)all(_at_)tools(_dot_)ietf(_dot_)org<mailto:draft-ietf-ccamp-rfc5787bis(_dot_)all(_at_)tools(_dot_)ietf(_dot_)org>"
 
<draft-ietf-ccamp-rfc5787bis(_dot_)all(_at_)tools(_dot_)ietf(_dot_)org<mailto:draft-ietf-ccamp-rfc5787bis(_dot_)all(_at_)tools(_dot_)ietf(_dot_)org>>
Cc: "ietf(_at_)ietf(_dot_)org<mailto:ietf(_at_)ietf(_dot_)org>" 
<ietf(_at_)ietf(_dot_)org<mailto:ietf(_at_)ietf(_dot_)org>>, 
"gen-art(_at_)ietf(_dot_)org<mailto:gen-art(_at_)ietf(_dot_)org>" 
<gen-art(_at_)ietf(_dot_)org<mailto:gen-art(_at_)ietf(_dot_)org>>
Subject: Gen-ART LC review of draft-ietf-ccamp-rfc5787bis-05
Resent-To: 
<acee(_dot_)lindem(_at_)ericsson(_dot_)com<mailto:acee(_dot_)lindem(_at_)ericsson(_dot_)com>>,
 Adrian Farrell 
<adrian(_at_)olddog(_dot_)co(_dot_)uk<mailto:adrian(_at_)olddog(_dot_)co(_dot_)uk>>,
 Andrew Malis 
<andrew(_dot_)g(_dot_)malis(_at_)verizon(_dot_)com<mailto:andrew(_dot_)g(_dot_)malis(_at_)verizon(_dot_)com>>,
 <dbrungard(_at_)att(_dot_)com<mailto:dbrungard(_at_)att(_dot_)com>>, 
<dimitri(_dot_)papadimitriou(_at_)alcatel-lucent(_dot_)com<mailto:dimitri(_dot_)papadimitriou(_at_)alcatel-lucent(_dot_)com>>,
 <lberger(_at_)labn(_dot_)net<mailto:lberger(_at_)labn(_dot_)net>>

I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, 
please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Please resolve these comments along with any other Last Call comments you may 
receive.



Document: draft-ietf-ccamp-rfc5787bis-05.

Reviewer: Roni Even

Review Date:2012–8–12

IETF LC End Date: 2012–8–17

IESG Telechat date:



Summary: This draft is almost ready for publication as a standard track RFC.





Major issues:



Minor issues:
In section 6.1 “ If specified more than once, instances preceding the first 
will be ignored and condition SHOULD be logged for possible action by the 
network operator.”  I am not sure what is meant by preceding the first.




Nits/editorial comments:



 1.  The following note appears in section 10.1, 10.2 and 10.3. “Note that the 
same values for the Inter-RA Export Upward sub-TLV and the Inter-RA Export 
Downward Sub-TLV MUST be used when they appear in the Link TLV, Node Attribute 
TLV, and Router Address TLV.” – why not have it in section 10 before section 
10.1.
 2.  I saw in appendix  B that one of the changes from RFC 5787 was to clarify 
the terminology before defining extensions, I would have found it easier to 
read if the ASON hierarchy and the relation to OSPF in section 2 were presented 
in figures. This was more an issue to me as a reader not familiar with the 
terminology and I would like to think that the more familiar reader will not 
have problem.

<Prev in Thread] Current Thread [Next in Thread>