Content-Type: multipart/signed;
 boundary="Apple-Mail=_931A0F47-FC8C-4801-87CA-2C06E0EEB7C7";
 protocol="application/pgp-signature"; micalg=pgp-sha512
Mime-Version: 1.0 (Mac OS X Mail 7.3 \(1878.6\))
From: Jari Arkko <jari.arkko@piuha.net>
In-Reply-To: <555E7E17.9050608@gmail.com>
Date: Thu, 28 May 2015 15:51:38 +0300
Message-Id: <B6EC16FE-1CA9-49B2-9613-CF0AAC79FF4E@piuha.net>
References: <555E7E17.9050608@gmail.com>
To: Tom Taylor <tom.taylor.stds@gmail.com>
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/KYCbpc8HsMTzmW7Z7CPufcr_GhA>
Cc: Alia Atlas <akatlas@gmail.com>,
 "Carlos Pignataro \(cpignata\)" <cpignata@cisco.com>,
 Gen Art <gen-art@ietf.org>, Jim Guichard <jguichar@cisco.com>
Subject: Re: [Gen-art] Last Call review of draft-ietf-sfc-architecture-08
Precedence: list


--Apple-Mail=_931A0F47-FC8C-4801-87CA-2C06E0EEB7C7
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain;
	charset=us-ascii

Thanks for your review, Tom

Jari

On 22 May 2015, at 03:53, Tom Taylor <tom.taylor.stds@gmail.com> wrote:

> I am the assigned Gen-ART reviewer for this draft. For background on
> Gen-ART, please see the FAQ at
>=20
> <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
>=20
> Please resolve these comments along with any other Last Call comments
> you may receive.
>=20
> Document: draft-ietf-sfc-architecture-08
> Reviewer: Tom Taylor
> Review Date:        2015-05-17
> IETF LC End Date:   2015-05-25
> IESG Telechat date: 2015-05-28
>=20
> Summary:
>=20
> There is one IPR declaration, which was repeated for two predecessor =
documents but not for the current draft. The draft is basically ready to =
go with a very minor issue and a few nits.
>=20
> Major issues:
>=20
> Minor issues:
>=20
> The Security Considerations section rightly mentions the need to avoid =
leaking SFC information. However, it does this under the heading of =
"Classification". Could I suggest that the first two sentences of the =
"Classification" bullet be separated out under the title "Boundaries"?
>=20
> Nits/editorial comments:
>=20
> Sec. 1.2, third bullet from the bottom: spell out SFF on first use, =
and give a forward reference to the next section, i.e.,
>   "...interconnect the Service Function Forwarders (SFFs, see next
>    section) ..."
>=20
> Sec. 1.2, next bullet: according to the RFC Editor Style Guide =
abbreviations list, FIB and RIB are not well-known abbreviations, hence =
need to be spelled out.
>=20
> Sec. 1.3, Service Function Forwarder, last line: spell out SFP? I know =
the definition is just a few lines down, so this is a maybe.
>=20
> Alternative suggestion: introduce a Section 1.3.1 at the beginning of =
the section, as follows:
>=20
> "1.3.1 Key Abbreviations
>=20
>   The terms listed here are defined in Section 1.3.2.
>=20
>   SF	Service Function
>   SFC	Service Function Chain or Service Function Chaining
>   SFF	Service Function Forwarder
>   SFP	Service Function Path
>   RSP	Rendered Service Path"
>=20
> Sec. 2.1, second para., third line from bottom: s/the the/the/
>=20
> _______________________________________________
> Gen-art mailing list
> Gen-art@ietf.org
> https://www.ietf.org/mailman/listinfo/gen-art


--Apple-Mail=_931A0F47-FC8C-4801-87CA-2C06E0EEB7C7
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment;
	filename=signature.asc
Content-Type: application/pgp-signature;
	name=signature.asc
Content-Description: Message signed with OpenPGP using GPGMail

-----BEGIN PGP SIGNATURE-----
Comment: GPGTools - https://gpgtools.org

iQIcBAEBCgAGBQJVZw9aAAoJEM80gCTQU46qHVIQAIwMBLvMcBz9YEddbr343xzX
6OM3+GaAWGQw5+gFIJM4uI5wE+s4PGmheWnfOHY5wM4S8kKl+nZsgkESTO4GzykK
Y7QoyyKjRJclZj7+hAyp8FOagKkRMsM1v4Vdq3bX1yYXbur2PoOrEwRcG5qws+Lq
eAPWihe7e9GUurzCSxvUmc0PJweoRTTSzL+U2uC4gDezMReclc7ZEJgo9Ank4REg
PHzTW+0MRqAwmG4z4Px5WWEC91Eb97n17iev6TmigLmcdmWwMFGo+KOIX35UJU5a
vnucMQKTVChWmes7d38FdfQWokH7wbX4U04DkqDwh+N0NeTHsTd8IFHcAvgCSXTu
kUtxzCD1hKVm8zk+2G08BnlzSOkJkYksmirlvhA4AZu8irId/ufrTUyDYpXoYcmV
ZuaOUG7WFU99+bN2j9WfMoEe4NAsSL6OnJ5DHIDHNpK+8z2uFhZrUWrew9hbsTA6
q67sqdG2rDqyc8oURTnhiy3gIt3sYKzLDM7g7Hb8DXGBtu8THuz3+X5kL/d6ZyKe
ENmkpu+FC+pBPwgJWzlO015X7Bp410gRl2qRuSyRHyau9swrpxlUTD0hT/8qdtDF
w9Qxj9Y9lEZvgqAKfoxy2nMav6ZAK1ODZ+N8X0uQ1Vs9JY5m+bZVIPFZl9VfKI7q
3Bvb2oAUtv0j5LAE0gri
=NCto
-----END PGP SIGNATURE-----

--Apple-Mail=_931A0F47-FC8C-4801-87CA-2C06E0EEB7C7--