Re: [bess] Genart last call review of draft-ietf-bess-evpn-df-election-framework-06

"Rabadan, Jorge (Nokia - US/Mountain View)" <jorge.rabadan@nokia.com> Wed, 19 December 2018 10:19 UTC

Return-Path: <jorge.rabadan@nokia.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 904641277D2; Wed, 19 Dec 2018 02:19:41 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.966
X-Spam-Level:
X-Spam-Status: No, score=-1.966 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.065, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=nokia.onmicrosoft.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 96CeOEnkgpzy; Wed, 19 Dec 2018 02:19:38 -0800 (PST)
Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-eopbgr70128.outbound.protection.outlook.com [40.107.7.128]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id C8EC1127598; Wed, 19 Dec 2018 02:19:37 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nokia.onmicrosoft.com; s=selector1-nokia-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=jpVwhTES+nKTbOK+it99R48Kp7YE3TrlmfHfvGmZtyU=; b=KWk+L8cspHOLaxas1izUSooWA7vN2lQBk9XWcxDcTlYQyD1dFlUCC/yr0CqNuS6y1jBB2alOy8aqwg1TInBarLAitAayjKoJx+lT56ag5OAebLs6+xsD6ZyESMAKQ3blWPtVujS+R8L7szNMudCw61R68bpqdPyp1v5hqwr/bI8=
Received: from VI1PR07MB3853.eurprd07.prod.outlook.com (52.134.26.15) by VI1PR07MB4687.eurprd07.prod.outlook.com (20.177.57.95) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1446.14; Wed, 19 Dec 2018 10:19:30 +0000
Received: from VI1PR07MB3853.eurprd07.prod.outlook.com ([fe80::18ee:bdbd:bb45:a0c3]) by VI1PR07MB3853.eurprd07.prod.outlook.com ([fe80::18ee:bdbd:bb45:a0c3%5]) with mapi id 15.20.1471.011; Wed, 19 Dec 2018 10:19:30 +0000
From: "Rabadan, Jorge (Nokia - US/Mountain View)" <jorge.rabadan@nokia.com>
To: Francesca Palombini <francesca.palombini@ericsson.com>, "gen-art@ietf.org" <gen-art@ietf.org>
CC: "draft-ietf-bess-evpn-df-election-framework.all@ietf.org" <draft-ietf-bess-evpn-df-election-framework.all@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>, "bess@ietf.org" <bess@ietf.org>
Thread-Topic: [bess] Genart last call review of draft-ietf-bess-evpn-df-election-framework-06
Thread-Index: AQHUk8f4OWslHDt6N0SR7WD37GLkA6WF8dqA
Date: Wed, 19 Dec 2018 10:19:30 +0000
Message-ID: <69F9BBF9-C52E-45F9-BE65-E466140D6823@nokia.com>
References: <154480398817.30540.8239762064497446504@ietfa.amsl.com>
In-Reply-To: <154480398817.30540.8239762064497446504@ietfa.amsl.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/10.14.0.181208
authentication-results: spf=none (sender IP is ) smtp.mailfrom=jorge.rabadan@nokia.com;
x-originating-ip: [88.14.53.100]
x-ms-publictraffictype: Email
x-microsoft-exchange-diagnostics: 1; VI1PR07MB4687; 6:J0yF35Yl3EHhQT0xGKrLgNm3bs5o5Q0wDcO0L/XbzHXC8Lp6ICnG7a6ZLsUkwyJLGXUjjBjJ4+KF1QH00CuIUfq/pVjz5BL5QoIHTn73HBzohceXNhyYJYqUdUNO7QQBS7xCUCNGO6YDXLyQx5sW6aksrEmEktBu6HAZvPihAzGjkGStQk0PawWi4HoMC4t1X1RY9F3SY/WnlS/RjGSSLjXlgaOu8+fGo4XXYqChhWjeJOzL/8h3VnneNATscy7Wvk8sFdVaA6vMcJAjMuUBCmmkmoLtfvfbGexq65rN4Ilcqd/jJqzu4SdX5WqG9NnZ40QdVJBEO3d6Cy11diToL3OhGPV+4rTIZoxF2LwynqiTf5QHm8GsCMowOzvKkcLf6UPxRN9Y9PG6BKqVGNrIKUIl5HPWpc3yq+qVVbCU31Rbq/yXVvmKffxvmaOhAON+RIaDz3tB+8T9lNeZsnLS4A==; 5:JGBqNGFEtngoC7YqJ8W6KR08VnQtSxxnnKKbKUotWoEJKkwJD8Sgqm5tFCkHOcEZ72aAYl25xO/nOv+1m3men4maon8OgkruUI+/IuTBMTqNkkKuneCqbuwWq9YTDq8vmbqGaJQ1i+Law5aVX5L1Jw2VmXn5yjV/utGet4747a0=; 7:Eq0IyCZld39VxpD8BKDF8ssLArlc+Gn2+NnpxJMt8RKpe89FXuBosLZoZB/SziVNdoNdp0iW1mU2zqHLS65c5AHpCHAPP+t8ix9t9J0PgV2UWSPCjVrG2ht5TbfP+rS0WpT4/irAoz0blAG9grdZmg==
x-ms-exchange-antispam-srfa-diagnostics: SOS;
x-ms-office365-filtering-correlation-id: ab9869d3-14c2-4c87-09a6-08d6659b76bd
x-ms-office365-filtering-ht: Tenant
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600074)(711020)(4618075)(2017052603328)(7193020); SRVR:VI1PR07MB4687;
x-ms-traffictypediagnostic: VI1PR07MB4687:
x-microsoft-antispam-prvs: <VI1PR07MB46876AD836B0E5B08A7FFE40F7BE0@VI1PR07MB4687.eurprd07.prod.outlook.com>
x-ms-exchange-senderadcheck: 1
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(8211001083)(3230021)(999002)(11241501185)(806100)(6040522)(2401047)(5005006)(8121501046)(3231475)(944501520)(52105112)(93006095)(93001095)(10201501046)(3002001)(6055026)(148016)(149066)(150057)(6041310)(20161123562045)(20161123564045)(20161123558120)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123560045)(201708071742011)(7699051)(76991095); SRVR:VI1PR07MB4687; BCL:0; PCL:0; RULEID:; SRVR:VI1PR07MB4687;
x-forefront-prvs: 0891BC3F3D
x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(396003)(366004)(376002)(39860400002)(136003)(346002)(189003)(199004)(13464003)(8676002)(486006)(4326008)(476003)(71200400001)(58126008)(66066001)(6246003)(81156014)(14454004)(6436002)(186003)(5660300001)(7736002)(305945005)(966005)(2501003)(71190400001)(478600001)(26005)(83716004)(97736004)(229853002)(36756003)(14444005)(256004)(4001150100001)(2906002)(86362001)(82746002)(3846002)(6116002)(6486002)(2616005)(99286004)(106356001)(25786009)(6506007)(446003)(68736007)(11346002)(102836004)(6512007)(316002)(8936002)(33656002)(110136005)(54906003)(53936002)(6306002)(76176011)(53546011)(81166006)(105586002); DIR:OUT; SFP:1102; SCL:1; SRVR:VI1PR07MB4687; H:VI1PR07MB3853.eurprd07.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1;
received-spf: None (protection.outlook.com: nokia.com does not designate permitted sender hosts)
x-microsoft-antispam-message-info: PvlI/wxXKgOkaX5xqCaNKdRfCrk3qmfMCHPOvEpyGKzHIts/vOscYtiZ8UoWNcv5ClRviwW22EdPM0o8gBiozqTda6L3T/I9qskAZlo32CnGNs53R957rSDmI3DEM/WneENkD1JYmpCsvfnIMiyo8VSguSktLEEE7uprJ4+wjvdIqzTJB39CUnt65U1JfBolZsepfYiDFN7g9FKXW3qndqJ5PGPr1u+OuJPKsCh92OFlm23yoqjaO5xu+BHp8fr7QmEOndou3b9z6vzFdT8t4RPPMhF7eDYjc632P3wMx334m3azeFiAfXxRvvLTci2t
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="utf-8"
Content-ID: <B09E4C71B6924046BCB86887EB179FDA@eurprd07.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: nokia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ab9869d3-14c2-4c87-09a6-08d6659b76bd
X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Dec 2018 10:19:30.5279 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5d471751-9675-428d-917b-70f44f9630b0
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR07MB4687
Archived-At: <https://mailarchive.ietf.org/arch/msg/bess/FJ4wdDcoNVLNGvlxpm7UQlMhr0Q>
Subject: Re: [bess] Genart last call review of draft-ietf-bess-evpn-df-election-framework-06
X-BeenThere: bess@ietf.org
X-Mailman-Version: 2.1.29
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: Wed, 19 Dec 2018 10:19:41 -0000

Hi Francesca,

Thank you very much for your review.
Please see in-line how we are resolving your comments in the next revision (07, to be published asap).

Thanks.
Jorge

-----Original Message-----
From: BESS <bess-bounces@ietf.org> on behalf of Francesca Palombini <francesca.palombini@ericsson.com>
Date: Friday, December 14, 2018 at 5:13 PM
To: "gen-art@ietf.org" <gen-art@ietf.org>
Cc: "draft-ietf-bess-evpn-df-election-framework.all@ietf.org" <draft-ietf-bess-evpn-df-election-framework.all@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>, "bess@ietf.org" <bess@ietf.org>
Subject: [bess] Genart last call review of draft-ietf-bess-evpn-df-election-framework-06

    Reviewer: Francesca Palombini
    Review result: Ready with Nits
    
    I am the assigned Gen-ART reviewer for this draft. The General Area
    Review Team (Gen-ART) reviews all IETF documents being processed
    by the IESG for the IETF Chair.  Please treat these comments just
    like any other last call comments.
    
    For more information, please see the FAQ at
    
    <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
    
    Document: draft-ietf-bess-evpn-df-election-framework-06
    Reviewer: Francesca Palombini
    Review Date: 2018-12-14
    IETF LC End Date: 2018-12-18
    IESG Telechat date: Not scheduled for a telechat
    
    Summary: This draft is basically ready for publication, but has nits that
    should be fixed before publication.
    
    Major issues: N/A
    
    Minor issues:
    
    I agree with the reviewers comments saying that this document should update
    RFC7432 and RFC8124. In particular, quoting RFC2232
    (https://tools.ietf.org/html/rfc2223#section-12):
    
       [...] A document that
       merely updates an earlier document cannot stand on its own; it is
       something that must be added to or inserted into the previously
       existing document, and has limited usefulness independently.  The
       terms Supercedes and Replaces are no longer used.
    
       Updates
    
          To be used as a reference from a new item that cannot be used
          alone (i.e., one that supplements a previous document), to refer
          to the previous document.  The newer publication is a part that
          will supplement or be added on to the existing document; e.g., an
          addendum, or separate, extra information that is to be added to
          the original document.
    
    (Yes, RFC2232 is obsolete, but I could not find the same text in the more
    recent RFC7322)

[JORGE] I think this document "can stand on its own" and it is "useful independently" of RFC7432, although the latter document is a normative reference of course. Please see the resolution to Adrian's comment: https://www.ietf.org/mail-archive/web/bess/current/msg03760.html 
Martin, please let us know if you are not okay with our resolution.

    
    Nits/editorial comments:
    
      "but they do not require
       any changes to the EVPN Route exchange and have minimal changes to
       their content per se."
    
    * what does their refer to?
[JORGE] changed to the following for clarity:
"These mechanisms do involve changes to the Default DF Election algorithm, but they do not require any changes to the EVPN Route exchange and have minimal changes in the EVPN routes."
    
    * Section 2.2.2: expand MAC-VRF on first usage for readability (or add a
    reference to its definition)
[JORGE] added to the terminology section.
    
    * Figure 3: add a definition for ANY STATE (the figure is clear, but for
    consistency I would add that in the text as well)
[JORGE] Added:
"5.  ANY_STATE: Refers to any of the above states."
    
    * Figure 3: add "or" between VLAN_CHANGE, RCVD_ES, LOST_ES (again, not
    necessary, suggested for readability of the figure)
[JORGE] done, thx
    
    * Section 3.1: the term "re-entering" needs clarifying: I would consider a loop
    as re-entering the state, but from bullet 8. it seems like you don't.
[JORGE] good point. Changed 8 to:
"8.  DF_CALC on VLAN_CHANGE, RCVD_ES or LOST_ES: do *****as in transition 7.******"
    
    * suggestion for figure 4 (otherwise it looks like there are 2 fields Bitmap of
    1B each):
    
          0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
         +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
         | Type=0x06     | Sub-Type(0x06)| RSV |  DF Alg |    Bitmap     ~
         +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
         ~               |            Reserved                           |
         +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
[JORGE] done, thanks.

    
    * Section 3.2: why was Bit 0 left unassigned in Bitmap?
[JORGE] there are implementations of https://tools.ietf.org/html/draft-ietf-bess-evpn-pref-df-02 using that bit.
    
    * IANA considerations: I think you want to specify that the policy for Alg 31
    is Experimental use (right now the text describing the policy only says "RFC
    required", with no distinction for different values).
[JORGE] ok, done.
    
    
    _______________________________________________
    BESS mailing list
    BESS@ietf.org
    https://www.ietf.org/mailman/listinfo/bess