Skip to main content

Telechat Review of draft-ietf-ccamp-rfc5787bis-
review-ietf-ccamp-rfc5787bis-genart-telechat-even-2012-10-26-00

Request Review of draft-ietf-ccamp-rfc5787bis
Requested revision No specific revision (document currently at 06)
Type Telechat Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2012-08-24
Requested 2012-08-17
Authors Andrew G. Malis , Acee Lindem , Papadimitriou Dimitri
I-D last updated 2012-10-26
Completed reviews Genart Last Call review of -?? by Roni Even
Genart Telechat review of -?? by Roni Even
Secdir Last Call review of -?? by Carl Wallace
Assignment Reviewer Roni Even
State Completed
Request Telechat review on draft-ietf-ccamp-rfc5787bis by General Area Review Team (Gen-ART) Assigned
Result Ready
Completed 2012-10-26
review-ietf-ccamp-rfc5787bis-genart-telechat-even-2012-10-26-00
Andy,

Thanks,

I am OK

Roni



From:

 Malis, Andrew G (Andy) [mailto:andrew.g.malis at verizon.com]

Sent:

 09 October, 2012 2:55 AM

To:

 Roni Even; draft-ietf-ccamp-rfc5787bis.all at tools.ietf.org

Cc:

 ietf at ietf.org; gen-art at ietf.org; Malis, Andrew G (Andy)

Subject:

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



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.even.tlv at gmail.com

>

Date:

Monday, August 13, 2012 14:07

To:

"

draft-ietf-ccamp-rfc5787bis.all at tools.ietf.org

" <

draft-ietf-ccamp-rfc5787bis.all at tools.ietf.org

>

Cc:

"

ietf at ietf.org

" <

ietf at ietf.org

>, "

gen-art at ietf.org

" <

gen-art at ietf.org

>

Subject:

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

Resent-To:

<

acee.lindem at ericsson.com

>, Adrian Farrell <

adrian at olddog.co.uk

>, Andrew Malis <

andrew.g.malis at verizon.com

>, <

dbrungard at att.com

>, <

dimitri.papadimitriou at alcatel-lucent.com

>, <

lberger at labn.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:



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.

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.