Return-Path: <shares@ndzh.com>
X-Original-To: forces@ietfa.amsl.com
Delivered-To: forces@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1])
 by ietfa.amsl.com (Postfix) with ESMTP id C81DD12D176;
 Wed, 25 May 2016 07:44:54 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 1.739
X-Spam-Level: *
X-Spam-Status: No, score=1.739 tagged_above=-999 required=5
 tests=[BAYES_00=-1.9, DOS_OUTLOOK_TO_MX=2.845, HTML_MESSAGE=0.001,
 RDNS_NONE=0.793] autolearn=no autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44])
 by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024)
 with ESMTP id iaFYzKJGNP7h; Wed, 25 May 2016 07:44:52 -0700 (PDT)
Received: from hickoryhill-consulting.com (unknown [50.245.122.97])
 (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits))
 (No client certificate requested)
 by ietfa.amsl.com (Postfix) with ESMTPS id 4129C12D768;
 Wed, 25 May 2016 07:44:51 -0700 (PDT)
X-Default-Received-SPF: pass (skip=loggedin (res=PASS))
 x-ip-name=174.124.182.128; 
From: "Susan Hares" <shares@ndzh.com>
To: <rtg-ads@ietf.org>
Date: Wed, 25 May 2016 10:44:45 -0400
Message-ID: <01b401d1b693$fb411810$f1c34830$@ndzh.com>
MIME-Version: 1.0
Content-Type: multipart/alternative;
 boundary="----=_NextPart_000_01B5_01D1B672.74308980"
X-Mailer: Microsoft Outlook 14.0
Thread-Index: AdG2keUYwqsP4AJKSjCZY/xFiMIDqg==
Content-Language: en-us
X-Authenticated-User: skh@ndzh.com 
Archived-At: <http://mailarchive.ietf.org/arch/msg/forces/lUb1Y4ahJ0m1Z5gi_uEwSYudWQM>
Cc: rtg-dir@ietf.org, zhang.xian@huawei.com, forces@ietf.org,
 Jonathan.Hardwick@metaswitch.com, draft-ietf-forces-interfelfb@tools.ietf.org, 
 'Jon Hudson' <jon.hudson@gmail.com>
Subject: [forces] RTG-DIR review of ForCES Inter-FE LFB
 (draft-ietf-forces-interfelfb-04.txt)
X-BeenThere: forces@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: ForCES WG mailing list <forces.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/forces>,
 <mailto:forces-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/forces/>
List-Post: <mailto:forces@ietf.org>
List-Help: <mailto:forces-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/forces>,
 <mailto:forces-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 25 May 2016 14:44:55 -0000

This is a multipart message in MIME format.

------=_NextPart_000_01B5_01D1B672.74308980
Content-Type: text/plain;
	charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Hello:=20

I have been selected as the Routing Directorate reviewer for this draft. =
The Routing Directorate seeks to review all routing or routing-related =
drafts as they pass through IETF last call and IESG review, and =
sometimes on special request. The purpose of the review is to provide =
assistance to the Routing ADs. For more information about the Routing =
Directorate, please see  =
<http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir> =
=E2=80=8Bhttp://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir

Although these comments are primarily for the use of the Routing ADs, it =
would be helpful if you could consider them along with any other IETF =
Last Call comments that you receive, and strive to resolve them through =
discussion or by updating the draft.

Document: draft-ietf-forces-interfelfb-04.txt

Reviewer: Susan Hares

Date: 5/25/2016

IETF End Date: unknown=20

Intended Status: Standards track

=20

Summary: This document is basically ready for publication, but has nits =
that should be considered before publication.=20

=20

Comments:=20

=C2=B7         Document contains good technical content for extending =
the Forces work into an Inter-FE LFB.   The document is easily read, but =
a few nits would improve its readability.

=C2=B7         My hand check of the inter-FE LFB XML model indicated all =
XML is fine.  If an automated check of the XML with the Forces LFBs, it =
would be useful to run this check. =20

=C2=B7         The improvement in the congestion consideration section =
(5.1.3) between -03 and -04 was necessary. =20

=20

Major issues: none=20

=20

Nits:=20

Page 9  figure 5. =E2=80=93 the between figure lines is not aligned.=20

This line begins with the =E2=80=9CEthernet Frame with:=E2=80=9D=20

=20

Page 12 =E2=80=93=20

Old

/(XXX: note to editor/

New /(XXX: note to RFC editor/

=20

Page 15=20

Old /original payload i.e. skips the IFE header information./

New /original payload (i.e. skips the IFE header information)/

=20

Page 21=20

=20

Old

/This memo includes one IANA requests within the registry =
https://www.iana.org/assignments/forces/

=20

New

/This memo includes one IANA request within the registry =
https://www.iana.org/assignments/forces. /

=20

p. 22=20

Old /As such, it has no impact on their security considerations./

New/ As such, it has no impact on these documents security =
considerations./

=20

Sue Hares=20

=20

=20


------=_NextPart_000_01B5_01D1B672.74308980
Content-Type: text/html;
	charset="utf-8"
Content-Transfer-Encoding: quoted-printable

<html xmlns:v=3D"urn:schemas-microsoft-com:vml" =
xmlns:o=3D"urn:schemas-microsoft-com:office:office" =
xmlns:w=3D"urn:schemas-microsoft-com:office:word" =
xmlns:m=3D"http://schemas.microsoft.com/office/2004/12/omml" =
xmlns=3D"http://www.w3.org/TR/REC-html40"><head><meta =
http-equiv=3DContent-Type content=3D"text/html; charset=3Dutf-8"><meta =
name=3DGenerator content=3D"Microsoft Word 14 (filtered =
medium)"><style><!--
/* Font Definitions */
@font-face
	{font-family:Wingdings;
	panose-1:5 0 0 0 0 0 0 0 0 0;}
@font-face
	{font-family:"Cambria Math";
	panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
	{font-family:Calibri;
	panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
	{margin:0in;
	margin-bottom:.0001pt;
	font-size:11.0pt;
	font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
	{mso-style-priority:99;
	color:blue;
	text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
	{mso-style-priority:99;
	color:purple;
	text-decoration:underline;}
p
	{mso-style-priority:99;
	mso-margin-top-alt:auto;
	margin-right:0in;
	mso-margin-bottom-alt:auto;
	margin-left:0in;
	font-size:12.0pt;
	font-family:"Times New Roman","serif";}
p.MsoListParagraph, li.MsoListParagraph, div.MsoListParagraph
	{mso-style-priority:34;
	margin-top:0in;
	margin-right:0in;
	margin-bottom:0in;
	margin-left:.5in;
	margin-bottom:.0001pt;
	font-size:11.0pt;
	font-family:"Calibri","sans-serif";}
span.EmailStyle17
	{mso-style-type:personal-compose;
	font-family:"Calibri","sans-serif";
	color:windowtext;}
span.apple-converted-space
	{mso-style-name:apple-converted-space;}
span.icon
	{mso-style-name:icon;}
.MsoChpDefault
	{mso-style-type:export-only;
	font-family:"Calibri","sans-serif";}
@page WordSection1
	{size:8.5in 11.0in;
	margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
	{page:WordSection1;}
/* List Definitions */
@list l0
	{mso-list-id:855267336;
	mso-list-type:hybrid;
	mso-list-template-ids:118115250 67698689 67698691 67698693 67698689 =
67698691 67698693 67698689 67698691 67698693;}
@list l0:level1
	{mso-level-number-format:bullet;
	mso-level-text:=EF=82=B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l0:level2
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l0:level3
	{mso-level-number-format:bullet;
	mso-level-text:=EF=82=A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l0:level4
	{mso-level-number-format:bullet;
	mso-level-text:=EF=82=B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l0:level5
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l0:level6
	{mso-level-number-format:bullet;
	mso-level-text:=EF=82=A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
@list l0:level7
	{mso-level-number-format:bullet;
	mso-level-text:=EF=82=B7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Symbol;}
@list l0:level8
	{mso-level-number-format:bullet;
	mso-level-text:o;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:"Courier New";}
@list l0:level9
	{mso-level-number-format:bullet;
	mso-level-text:=EF=82=A7;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;
	font-family:Wingdings;}
ol
	{margin-bottom:0in;}
ul
	{margin-bottom:0in;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext=3D"edit" spidmax=3D"1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext=3D"edit">
<o:idmap v:ext=3D"edit" data=3D"1" />
</o:shapelayout></xml><![endif]--></head><body lang=3DEN-US link=3Dblue =
vlink=3Dpurple><div class=3DWordSection1><p class=3DMsoNormal>Hello: =
<o:p></o:p></p><p style=3D'background:white'><span =
style=3D'font-size:11.0pt;color:black'>I have been selected as the =
Routing Directorate reviewer for this draft. The Routing Directorate =
seeks to review all routing or routing-related drafts as they pass =
through IETF last call and IESG review, and sometimes on special =
request. The purpose of the review is to provide assistance to the =
Routing ADs. For more information about the Routing Directorate, please =
see<span class=3Dapple-converted-space>&nbsp;</span><a =
href=3D"http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir"><span =
class=3Dicon><span style=3D'color:#440088'>=E2=80=8B</span></span><span =
style=3D'font-size:12.0pt;color:#440088'>http://trac.tools.ietf.org/area/=
rtg/trac/wiki/RtgDir</span></a><o:p></o:p></span></p><p =
style=3D'background:white'><span =
style=3D'font-size:11.0pt;color:black'>Although these comments are =
primarily for the use of the Routing ADs, it would be helpful if you =
could consider them along with any other IETF Last Call comments that =
you receive, and strive to resolve them through discussion or by =
updating the draft.<o:p></o:p></span></p><p class=3DMsoNormal>Document: =
draft-ietf-forces-interfelfb-04.txt<o:p></o:p></p><p =
class=3DMsoNormal>Reviewer: Susan Hares<o:p></o:p></p><p =
class=3DMsoNormal>Date: 5/25/2016<o:p></o:p></p><p =
class=3DMsoNormal>IETF End Date: unknown <o:p></o:p></p><p =
class=3DMsoNormal>Intended Status: Standards track<o:p></o:p></p><p =
class=3DMsoNormal><o:p>&nbsp;</o:p></p><p class=3DMsoNormal>Summary: =
This document is basically ready for publication, but has nits that =
should be considered before publication. <o:p></o:p></p><p =
class=3DMsoNormal><o:p>&nbsp;</o:p></p><p class=3DMsoNormal>Comments: =
<o:p></o:p></p><p class=3DMsoListParagraph =
style=3D'text-indent:-.25in;mso-list:l0 level1 lfo1'><![if =
!supportLists]><span style=3D'font-family:Symbol'><span =
style=3D'mso-list:Ignore'>=C2=B7<span style=3D'font:7.0pt "Times New =
Roman"'>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; =
</span></span></span><![endif]>Document contains good technical content =
for extending the Forces work into an Inter-FE LFB. =C2=A0=C2=A0The =
document is easily read, but a few nits would improve its =
readability.<o:p></o:p></p><p class=3DMsoListParagraph =
style=3D'text-indent:-.25in;mso-list:l0 level1 lfo1'><![if =
!supportLists]><span style=3D'font-family:Symbol'><span =
style=3D'mso-list:Ignore'>=C2=B7<span style=3D'font:7.0pt "Times New =
Roman"'>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; =
</span></span></span><![endif]>My hand check of the inter-FE LFB XML =
model indicated all XML is fine.=C2=A0 If an automated check of the XML =
with the Forces LFBs, it would be useful to run this check.=C2=A0 =
<o:p></o:p></p><p class=3DMsoListParagraph =
style=3D'text-indent:-.25in;mso-list:l0 level1 lfo1'><![if =
!supportLists]><span style=3D'font-family:Symbol'><span =
style=3D'mso-list:Ignore'>=C2=B7<span style=3D'font:7.0pt "Times New =
Roman"'>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; =
</span></span></span><![endif]>The improvement in the congestion =
consideration section (5.1.3) between -03 and -04 was necessary.=C2=A0 =
<o:p></o:p></p><p class=3DMsoListParagraph><o:p>&nbsp;</o:p></p><p =
class=3DMsoNormal>Major issues: none <o:p></o:p></p><p =
class=3DMsoNormal><o:p>&nbsp;</o:p></p><p class=3DMsoNormal>Nits: =
<o:p></o:p></p><p class=3DMsoNormal>Page 9=C2=A0 figure 5. =E2=80=93 the =
between figure lines is not aligned. <o:p></o:p></p><p =
class=3DMsoNormal>This line begins with the =E2=80=9CEthernet Frame =
with:=E2=80=9D <o:p></o:p></p><p =
class=3DMsoNormal><o:p>&nbsp;</o:p></p><p class=3DMsoNormal>Page 12 =
=E2=80=93 <o:p></o:p></p><p class=3DMsoNormal>Old<o:p></o:p></p><p =
class=3DMsoNormal>/(XXX: note to editor/<o:p></o:p></p><p =
class=3DMsoNormal>New /(XXX: note to RFC editor/<o:p></o:p></p><p =
class=3DMsoNormal><o:p>&nbsp;</o:p></p><p class=3DMsoNormal>Page 15 =
<o:p></o:p></p><p class=3DMsoNormal>Old /original payload i.e. skips the =
IFE header information./<o:p></o:p></p><p class=3DMsoNormal>New =
/original payload (i.e. skips the IFE header =
information)/<o:p></o:p></p><p class=3DMsoNormal><o:p>&nbsp;</o:p></p><p =
class=3DMsoNormal>Page 21 <o:p></o:p></p><p =
class=3DMsoNormal><o:p>&nbsp;</o:p></p><p =
class=3DMsoNormal>Old<o:p></o:p></p><p class=3DMsoNormal>/This memo =
includes one IANA requests within the registry <a =
href=3D"https://www.iana.org/assignments/forces/">https://www.iana.org/as=
signments/forces/</a><o:p></o:p></p><p =
class=3DMsoNormal><o:p>&nbsp;</o:p></p><p =
class=3DMsoNormal>New<o:p></o:p></p><p class=3DMsoNormal>/This memo =
includes one IANA request within the registry =
https://www.iana.org/assignments/forces. /<o:p></o:p></p><p =
class=3DMsoNormal><o:p>&nbsp;</o:p></p><p class=3DMsoNormal>p. 22 =
<o:p></o:p></p><p class=3DMsoNormal>Old /As such, it has no impact on =
their security considerations./<o:p></o:p></p><p class=3DMsoNormal>New/ =
As such, it has no impact on these documents security =
considerations./<o:p></o:p></p><p =
class=3DMsoNormal><o:p>&nbsp;</o:p></p><p class=3DMsoNormal>Sue Hares =
<o:p></o:p></p><p class=3DMsoNormal><o:p>&nbsp;</o:p></p><p =
class=3DMsoNormal><o:p>&nbsp;</o:p></p></div></body></html>
------=_NextPart_000_01B5_01D1B672.74308980--

