IETF Logo Mail Archive

[Gen-art] Last Call review of draft-ietf-sfc-architecture-08

Tom Taylor <tom.taylor.stds@gmail.com> Fri, 22 May 2015 00:53 UTC

Return-Path: <tom.taylor.stds@gmail.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id A0ECE1AC3BF for <gen-art@ietfa.amsl.com>; Thu, 21 May 2015 17:53:47 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2
X-Spam-Level:
X-Spam-Status: No, score=-2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, SPF_PASS=-0.001] autolearn=ham
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 zz4rLEe68RQ9 for <gen-art@ietfa.amsl.com>; Thu, 21 May 2015 17:53:46 -0700 (PDT)
Received: from mail-ie0-x236.google.com (mail-ie0-x236.google.com [IPv6:2607:f8b0:4001:c03::236]) (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 D4CD51AC3BE for <gen-art@ietf.org>; Thu, 21 May 2015 17:53:45 -0700 (PDT)
Received: by iesa3 with SMTP id a3so21011639ies.2 for <gen-art@ietf.org>; Thu, 21 May 2015 17:53:45 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=message-id:date:from:user-agent:mime-version:to:subject :content-type:content-transfer-encoding; bh=9BHIhV7In2+wV/rZWgvgUyROuxnQRN7OtrXLOvywlDE=; b=RhC3DQNgwHEjh/dDB42Xl9QsTxAj5u4LAPaKEgQjczcg2QGU0VT4JoGEOlAmnKx/r6 IBWtmFjDxtc/QvqNCRsIRybmGaLxVzWQ55Wu91QZtXd0omVEHsgqfhC2f6vlYeKoEn0E cVUg5FdfMiDqeSwE01bOoT7swZOkjhaxA4DX931JrBEJYk7fF1gevgMs8ljT6jaqEnVv LYl/jlDOvIYYVRSjwIXdZH5LZPUoCOpVkegsO6L+m8ldwsetfieejtnqhma45L/8wCtS kQlqeXtMQTr3/pfdedYdLJcSOgCGaATTt+Gda5bHD8EALIjomTcBIS9+RU+WcEoVJPxA yR3w==
X-Received: by 10.50.43.227 with SMTP id z3mr404074igl.12.1432256025371; Thu, 21 May 2015 17:53:45 -0700 (PDT)
Received: from [192.168.1.135] ([108.63.145.250]) by mx.google.com with ESMTPSA id qs10sm2508043igb.14.2015.05.21.17.53.44 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 21 May 2015 17:53:44 -0700 (PDT)
Message-ID: <555E7E17.9050608@gmail.com>
Date: Thu, 21 May 2015 20:53:43 -0400
From: Tom Taylor <tom.taylor.stds@gmail.com>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0
MIME-Version: 1.0
To: "Joel M. Halpern" <jmh@joelhalpern.com>, "Carlos Pignataro (cpignata)" <cpignata@cisco.com>, Jim Guichard <jguichar@cisco.com>, Alia Atlas <akatlas@gmail.com>, Gen Art <gen-art@ietf.org>
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Transfer-Encoding: 7bit
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/ATUvHw35QXqpuFHH1zwMHoZ8wTk>
Subject: [Gen-art] Last Call review of draft-ietf-sfc-architecture-08
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 22 May 2015 00:53:47 -0000

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-sfc-architecture-08
Reviewer: Tom Taylor
Review Date:        2015-05-17
IETF LC End Date:   2015-05-25
IESG Telechat date: 2015-05-28

Summary:

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.

Major issues:

Minor issues:

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"?

Nits/editorial comments:

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) ..."

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.

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.

Alternative suggestion: introduce a Section 1.3.1 at the beginning of 
the section, as follows:

"1.3.1 Key Abbreviations

    The terms listed here are defined in Section 1.3.2.

    SF	Service Function
    SFC	Service Function Chain or Service Function Chaining
    SFF	Service Function Forwarder
    SFP	Service Function Path
    RSP	Rendered Service Path"

Sec. 2.1, second para., third line from bottom: s/the the/the/

v2.23.0 | Report a Bug | By Email