Skip to main content

Last Call Review of draft-ietf-fecframe-pseudo-cdp-
review-ietf-fecframe-pseudo-cdp-genart-lc-garcia-2012-09-27-00

Request Review of draft-ietf-fecframe-pseudo-cdp
Requested revision No specific revision (document currently at 05)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2012-10-01
Requested 2012-09-20
Authors Ulas Kozat , Ali C. Begen
I-D last updated 2012-09-27
Completed reviews Genart Last Call review of -?? by Miguel Angel García
Secdir Last Call review of -?? by Klaas Wierenga
Assignment Reviewer Miguel Angel García
State Completed
Request Last Call review on draft-ietf-fecframe-pseudo-cdp by General Area Review Team (Gen-ART) Assigned
Result Ready
Completed 2012-09-27
review-ietf-fecframe-pseudo-cdp-genart-lc-garcia-2012-09-27-00
I have been selected as the General Area Review Team (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 comments you may receive.

Document: draft-ietf-fecframe-pseudo-cdp-04
Reviewer: Miguel Garcia <Miguel.A.Garcia at ericsson.com>
Review Date: 2012-09-27
IETF LC End Date: 2012-10-01



Summary: The document is ready for publication as an Informational RFC, 


but has some nits that should be addressed




Major issues: none

Minor issues: none

Nits/editorial comments:

- About the SDP example in Section 3.1. The first paragraph reads:

   The SDP description below states that the source flow defined
   by the tuple {*,*,233.252.0.1,30000} is identified with FID=0 and the
   source flow defined by the tuple {*,*,233.252.0.2,30000} is
   identified with FID=1.




However, the example shows flows does not show any flow with any token 


named "FID". I don't know if the text is trying to refer to the "mid" 


attribute, in which case the values are S1 and S2, respectively. Or the 


text is trying to refer to the "id" parameter of the "fec-source-flow" 


attribute. The text should not leave this interpretation to the reader.






- Figure 7 and 8. Do the captions need to indicate that these are "for 


IPv4", for example, in Figure 7: "Example of a source packet for IPv4"






- idnits reveals that draft-ietf-fecframe-config-signaling ha sbeen 


published as RFC 6695.




/Miguel
--
Miguel A. Garcia
+34-91-339-3608
Ericsson Spain