Re: [bess] Adam Roach's No Objection on draft-ietf-bess-evpn-etree-13: (with COMMENT)

"Ali Sajassi (sajassi)" <sajassi@cisco.com> Fri, 27 October 2017 23:06 UTC

Return-Path: <sajassi@cisco.com>
X-Original-To: bess@ietfa.amsl.com
Delivered-To: bess@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id E4C691397F3; Fri, 27 Oct 2017 16:06:50 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.521
X-Spam-Level:
X-Spam-Status: No, score=-14.521 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, 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 mXXuEhT3aqip; Fri, 27 Oct 2017 16:06:49 -0700 (PDT)
Received: from alln-iport-8.cisco.com (alln-iport-8.cisco.com [173.37.142.95]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id EB844139561; Fri, 27 Oct 2017 16:06:48 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=4031; q=dns/txt; s=iport; t=1509145609; x=1510355209; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=qY72+EhwBc/RbEdCwzPKTprzXVijkeROfLG5bpOW8NQ=; b=V7oWp2peZrC14CuB8U2Nc/NAfvilvdIQgS/ebyYvEiqT1W1LFKIvenle FfNpFmFC+r+5cF26dtsb8P55nw7GdU7LyOCpVEm0umQOGd4j02IHPlGf6 AbrjqCLv9Z/94q4gfnpYRqZ1FOuQDLp7KnHZfCLFcXoX/QszC7cVhld3n A=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0CaAACFu/NZ/51dJa1cGQEBAQEBAQEBA?= =?us-ascii?q?QEBBwEBAQEBg19kbicHjhKPE5g8EIIBCiOFGAKESz8YAQIBAQEBAQEBayiFHgZ?= =?us-ascii?q?tDBACAQhGMiUCBAENBYojEKsfiwYBAQEBAQEBAQEBAQEBAQEBAQEBAQEYBYMug?= =?us-ascii?q?geDOAGDKoRSARIBUYVEBYoliAKPXAKHZI0WghWGAIsYiiyCM4kEAhEZAYE4AR8?= =?us-ascii?q?4gQNMGXoVgy2EX3cBiUmBJIERAQEB?=
X-IronPort-AV: E=Sophos;i="5.44,306,1505779200"; d="scan'208";a="22721548"
Received: from rcdn-core-6.cisco.com ([173.37.93.157]) by alln-iport-8.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 27 Oct 2017 23:06:48 +0000
Received: from XCH-RTP-003.cisco.com (xch-rtp-003.cisco.com [64.101.220.143]) by rcdn-core-6.cisco.com (8.14.5/8.14.5) with ESMTP id v9RN6lFG015510 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=FAIL); Fri, 27 Oct 2017 23:06:48 GMT
Received: from xch-rtp-005.cisco.com (64.101.220.145) by XCH-RTP-003.cisco.com (64.101.220.143) with Microsoft SMTP Server (TLS) id 15.0.1320.4; Fri, 27 Oct 2017 19:06:47 -0400
Received: from xch-rtp-005.cisco.com ([64.101.220.145]) by XCH-RTP-005.cisco.com ([64.101.220.145]) with mapi id 15.00.1320.000; Fri, 27 Oct 2017 19:06:46 -0400
From: "Ali Sajassi (sajassi)" <sajassi@cisco.com>
To: Adam Roach <adam@nostrum.com>, The IESG <iesg@ietf.org>
CC: "draft-ietf-bess-evpn-etree@ietf.org" <draft-ietf-bess-evpn-etree@ietf.org>, Thomas Morin <thomas.morin@orange.com>, "bess-chairs@ietf.org" <bess-chairs@ietf.org>, "bess@ietf.org" <bess@ietf.org>, Alvaro Retana <aretana.ietf@gmail.com>
Thread-Topic: Adam Roach's No Objection on draft-ietf-bess-evpn-etree-13: (with COMMENT)
Thread-Index: AQHTLPd73X7Zo0dFnkWriRw87SeUB6L4ZEOA
Date: Fri, 27 Oct 2017 23:06:46 +0000
Message-ID: <D618C748.2270B8%sajassi@cisco.com>
References: <150535195008.12516.4893099114969931163.idtracker@ietfa.amsl.com>
In-Reply-To: <150535195008.12516.4893099114969931163.idtracker@ietfa.amsl.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/14.7.7.170905
x-ms-exchange-messagesentrepresentingtype: 1
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [10.19.76.53]
Content-Type: text/plain; charset="iso-8859-1"
Content-ID: <0D9E5E3B75DD7846AAFB193478E5F45C@emea.cisco.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/bess/_Qm_D157QylyfACwHUZx3to6vlg>
Subject: Re: [bess] Adam Roach's No Objection on draft-ietf-bess-evpn-etree-13: (with COMMENT)
X-BeenThere: bess@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: BGP-Enabled ServiceS working group discussion list <bess.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/bess>, <mailto:bess-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/bess/>
List-Post: <mailto:bess@ietf.org>
List-Help: <mailto:bess-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/bess>, <mailto:bess-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 27 Oct 2017 23:06:51 -0000

Hi Adam,

Please refer to my replies inline.

On 9/13/17, 6:19 PM, "Adam Roach" <adam@nostrum.com> wrote:

>Adam Roach has entered the following ballot position for
>draft-ietf-bess-evpn-etree-13: No Objection
>
>When responding, please keep the subject line intact and reply to all
>email addresses included in the To and CC lines. (Feel free to cut this
>introductory paragraph, however.)
>
>
>Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
>for more information about IESG DISCUSS and COMMENT positions.
>
>
>The document, along with other ballot positions, can be found here:
>https://datatracker.ietf.org/doc/draft-ietf-bess-evpn-etree/
>
>
>
>----------------------------------------------------------------------
>COMMENT:
>----------------------------------------------------------------------
>
>Section 3.3.2 says:
>
>   The PEs implementing an E-Tree service need not perform MAC learning
>   when the traffic flows between Root and Leaf sites are mainly
>   multicast or broadcast.
>
>Does this mean to say "mainly"? I would have expected "only", as in
>section
>4.3.  In particular, if "mainly" is correct, I'm unsure how unicast
>traffic is
>supposed to be handled. Is it simply flooded out (modulo filters) in the
>same
>way as broadcast traffic? If that's the intention, I think some
>additional text
>here saying as much would be useful.

Ali> Added the following sentence:
Ali> "In such scenarios, the small amount of unicast traffic (if any) is
sent as part of BUM traffic."
>
>----
>
>Section 5.1:
>
>   The reserved bits SHOULD be set to zero by the transmitter and SHOULD
>   be ignored by the receiver.
>
>The "SHOULD" here seems that it might make assigning meaning to these
>bits in
>the future problematic. If implementations decide to either assign local
>meaning to these bits, or decide that they don't need to be initialized,
>then
>future IETF specs that try to use them might be in for some pretty nasty
>deployment surprises. If these need to be "SHOULD" instead of "MUST,"
>please
>add some motivating text to the document for the sake of people who might
>want
>to extend the protocol in the future.

Ali> changed the second ³SHOULD² to ³MUST²:
Ali > "The reserved bits SHOULD be set to zero by the transmitter and MUST
be ignored by the receiver."
>
>----
>
>The IANA handling of "Composite Tunnel" seems problematic: although
>several
>values in this "Reserved for Composite Tunnel" range have well-defined
>values
>(e.g., 0x81 means "RSVP-TE P2MP LSP with composite tunnel"), they look
>unallocated/reserved in the resulting table. I think what you really want
>to do
>here is update the introductory text for the table to make it clear that
>values
>now take the range 0x00 - 0x7F and modify 0x7B through 0x7F as you've
>proposed
>doing.

Ali> updated the section to make it more clear - please refer to rev14 of
this draft. The only changes for this document is for the range of
0x7B-0xFA which was previously unassigned. The decomposition of this range
is explained in the IANA section.
>
>On top of this, I have the same concerns as Warren does regarding the
>impact of
>this change on in-the-field use of experimental tunnel types. I think the
>only
>reasonable way to retrofit this mechanism onto the existing system would
>be to
>to say that the "Composite Tunnel" bit MUST be ignored for tunnel types
>0x7B-0x7E, and possibly allocate some additional experimental codepoints
>(e.g.,
>0x77-0x7A) so that people can run experiments with tunnel types that also
>include composite tunnel behavior.

Ali> There shouldn¹t be any impact. The current tunnel types are in the
range of 0x00-0x07 [RFC7385]. The max range for the future will be in the
range of 0x00-0x7A. The mirror image of this range with the composite
tunnel type would be in the range of 0x80-FA. There is complete backward
compatibility with existing experimental values.
>
>