Re: [sfc] AD review of draft-ietf-sfc-nsh-18

"Carlos Pignataro (cpignata)" <cpignata@cisco.com> Sat, 12 August 2017 01:33 UTC

Return-Path: <cpignata@cisco.com>
X-Original-To: sfc@ietfa.amsl.com
Delivered-To: sfc@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 5B9021321AA for <sfc@ietfa.amsl.com>; Fri, 11 Aug 2017 18:33:51 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.52
X-Spam-Level:
X-Spam-Status: No, score=-14.52 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com
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 YdIYSbczYRVQ for <sfc@ietfa.amsl.com>; Fri, 11 Aug 2017 18:33:49 -0700 (PDT)
Received: from rcdn-iport-9.cisco.com (rcdn-iport-9.cisco.com [173.37.86.80]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 69621132143 for <sfc@ietf.org>; Fri, 11 Aug 2017 18:33:49 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=6306; q=dns/txt; s=iport; t=1502501629; x=1503711229; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=53Hi+X9g2wyDGmxpCp0pvIRG5nqfcie2/bx/voXMWjs=; b=IjWfxZF2kfGzr2O8R4/vf6ZqfWgDHN8kF6kKTT3DE/hutzYM7un4/c9z jKkZ1kOtruJAAjpn/2qiFC7YU/Tu+pAADEZs+2yJ+GOI6ZldIbf70jRLx WNk4TNuEWkB0ubWHBtMBEyesRXlCPB2wTX0kwxLgUXqIXXWQW6zIrv7GP k=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0BHAQBkWo5Z/40NJK1dGQEBAQEBAQEBAQEBBwEBAQEBg1pkgRQHjgqQDYFMIog2jWGCEiENhRkCGoRcPxgBAgEBAQEBAQFrKIUYAQEBAQIBAQEbBhE6CwULAgEIDgoCAiYCAgIfBgsVEAIEDgWKFwMNCBCrKoImhzUNhCEBAQEBAQEBAQEBAQEBAQEBAQEBAQEYBYELgh2BYiCBTIFjKwuBZYEMgleFLzCCMQWYBIdmPAKHUYdzhHSCD4VdimiMMYlhAR84gQp3FUkSAYcHdgEBiC+BDwEBAQ
X-IronPort-AV: E=Sophos;i="5.41,360,1498521600"; d="scan'208";a="279524629"
Received: from alln-core-8.cisco.com ([173.36.13.141]) by rcdn-iport-9.cisco.com with ESMTP/TLS/DHE-RSA-AES256-SHA; 12 Aug 2017 01:33:48 +0000
Received: from XCH-RTP-017.cisco.com (xch-rtp-017.cisco.com [64.101.220.157]) by alln-core-8.cisco.com (8.14.5/8.14.5) with ESMTP id v7C1XmUn031121 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=FAIL); Sat, 12 Aug 2017 01:33:48 GMT
Received: from xch-rtp-020.cisco.com (64.101.220.160) by XCH-RTP-017.cisco.com (64.101.220.157) with Microsoft SMTP Server (TLS) id 15.0.1210.3; Fri, 11 Aug 2017 21:33:47 -0400
Received: from xch-rtp-020.cisco.com ([64.101.220.160]) by XCH-RTP-020.cisco.com ([64.101.220.160]) with mapi id 15.00.1210.000; Fri, 11 Aug 2017 21:33:47 -0400
From: "Carlos Pignataro (cpignata)" <cpignata@cisco.com>
To: Alia Atlas <akatlas@gmail.com>
CC: "sfc@ietf.org" <sfc@ietf.org>
Thread-Topic: [sfc] AD review of draft-ietf-sfc-nsh-18
Thread-Index: AQHTEvVe0TUBe/21qkKFipdGFMpx/aKALIoAgAAAsACAAAH8AIAAAQYAgAAC/IA=
Date: Sat, 12 Aug 2017 01:33:47 +0000
Message-ID: <B6BE0FA6-41C5-4929-99D2-0A06AB004AC2@cisco.com>
References: <CAG4d1reNXt+n0=1gP8RV3AHKRfP2v6URrTriTfxexHTcjxqjOg@mail.gmail.com> <518F75E3-A390-4539-B758-1641F2BF74B2@cisco.com> <CAG4d1rdUzq7x4QyGs+c+SwkdvVUaykwEkfeT4EcTJ6XjhU7AcA@mail.gmail.com> <C1AAEEBC-EC33-49FE-9202-69E4432A6A4B@cisco.com> <CAG4d1rfpZbvxHM1uSCD7kUOGOg-PX1M3pi1cOVpZY1KNGVTgfA@mail.gmail.com>
In-Reply-To: <CAG4d1rfpZbvxHM1uSCD7kUOGOg-PX1M3pi1cOVpZY1KNGVTgfA@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-ms-exchange-messagesentrepresentingtype: 1
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [10.118.116.133]
Content-Type: text/plain; charset="utf-8"
Content-ID: <F5C3E083CFC1A643901A1B5E79A5D86C@emea.cisco.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/sfc/kNyd9Ti6ayms_Ft9Mti6bhldX2s>
Subject: Re: [sfc] AD review of draft-ietf-sfc-nsh-18
X-BeenThere: sfc@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Network Service Chaining <sfc.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/sfc>, <mailto:sfc-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/sfc/>
List-Post: <mailto:sfc@ietf.org>
List-Help: <mailto:sfc-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/sfc>, <mailto:sfc-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 12 Aug 2017 01:33:51 -0000

> On Aug 11, 2017, at 9:23 PM, Alia Atlas <akatlas@gmail.com> wrote:
> 
> 
> 
> On Fri, Aug 11, 2017 at 9:19 PM, Carlos Pignataro (cpignata) <cpignata@cisco.com> wrote:
> 
>> On Aug 11, 2017, at 9:12 PM, Alia Atlas <akatlas@gmail.com> wrote:
>> 
>> 
>> 
>> On Fri, Aug 11, 2017 at 9:09 PM, Carlos Pignataro (cpignata) <cpignata@cisco.com> wrote:
>> Thanks, Alia. Please see inline.
>> 
>>> On Aug 11, 2017, at 6:58 PM, Alia Atlas <akatlas@gmail.com> wrote:
>>> 
>>> As is customary, I have done my AD review of draft-ietf-sfc-nsh-18.  First, I would like to thank the editors, Carlos, Uri, and Paul, as well as the many contributors for their work on this document.  It has improved substantially.
>>> 
>>> I do have a few minor comments from my review.  They are below.  I will proceed to ask for an IETF Last Call now and schedule this draft for Aug 31; for that telechat date, I will require a very active shepherd as well as editors.
>>> 
>>> Please do SUBMIT the latest editor's version of this draft ASAP.  Version numbers are cheap and it is much better for reviewers to see the latest version. This must be the process going forward to avoid the same points being brought up multiple times.
>> 
>> Sounds good.
>> 
>>> 
>>> 1) As Acee already noted, receivers of unassigned flags MUST ignore the value.  There are a couple places to fix this.
>>> 
>> 
>> Agreed. Done.
>> 
>>> 2) Given the consensus in NVO3 and that VXLAN-GPE will be progressed as an informational document described an alternative that was not selected for standardization, I would prefer a different example in Sec 6.1 Table 1.
>> 
>> This is just an example, non normative, non-binging, and list VXLAN-GPE along with GRE and Ethernet. What would you prefer the document uses?
>> 
>> Geneve?  MPLS? UDP?
>> 
>> Something Standards Track please.
>> 
> 
> > I think that would give the wrong impression that SFC can run only atop Standards Track IETF technology. 
> 
> > The document currently lists: GRE, Standards Track. UDP in Table 3, Standards Track. VXLAN-GPE, Informational. And Ethernet. Sounds like a diverse mix.
> 
> VXLAN-GPE will be Informational - in the sense of an alternative considered and not taken.  

Does NSH say that it can only run on Standards-Track transports? I do not believe this spec places any restrictions on the “pedigree” of the tunnel.

> 
> There is no reason to fail to respect the consensus in NVO3.  

I fail to see how this is disrespecting consensus in the NVO3 WG.

The Transports, as far as NSH are concerned, are Informative references, non-normative examples (which include IETF standards track technology, non-IETF technology, etc).

There is actual purpose in mentioning technologies that are also not standards track: to show that NSH can actually be deployed on them, independently of the NSH spec.

> If you want to toss in VXLAN, that'd be fine.   
> 

I have not desire to "toss in" tunneling technologies without understand why. 

I am happy to add others as you or the WG see fit.

Thanks,

Carlos.

>  
> > If you insist, I could add more rows to the table to also be more inclusive in addition to diverse? I can also add more IPv6 NHs if so?
> 
> > Thanks!
> 
> > Carlos.
> 
>> Thanks,
>> Alia
>>  
>>> 3) In Sec 7.1, there is a sentence "In some cases they may terminate, and be able to inspect encrypted traffic."  Unless there is a strong technical need to point this out, I would pick a different example.  There is a great deal of current controversy and discussion happening in TLS currently - and this is likely to trigger that discussion unnecessarily.
>> 
>> Ack. Gone.
>> 
>>> 
>>> 4) I am happy to see that IEEE (https://regauth.standards.ieee.org/standards-ra-web/pub/view.html#registries) is now showing the EtherType 0x894F as referring to this draft.  Thanks!
>> 
>> Anytime :-)
>> 
>> Thanks!
>> 
>> Carlos.
>> 
>>> 
>>> Regards,
>>> Alia
>>> 
>>> _______________________________________________
>>> sfc mailing list
>>> sfc@ietf.org
>>> https://www.ietf.org/mailman/listinfo/sfc
>> 
>> —
>> Carlos Pignataro, carlos@cisco.com
>> 
>> “Sometimes I use big words that I do not fully understand, to make myself sound more photosynthesis."
> 
> —
> Carlos Pignataro, carlos@cisco.com
> 
> “Sometimes I use big words that I do not fully understand, to make myself sound more photosynthesis."

—
Carlos Pignataro, carlos@cisco.com

“Sometimes I use big words that I do not fully understand, to make myself sound more photosynthesis."