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

Francesca Palombini <francesca.palombini@ericsson.com> Thu, 03 January 2019 15:01 UTC

Return-Path: <francesca.palombini@ericsson.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id B6401130E04 for <gen-art@ietfa.amsl.com>; Thu, 3 Jan 2019 07:01:36 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.365
X-Spam-Level:
X-Spam-Status: No, score=-4.365 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.065, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=unavailable autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=ericsson.com header.b=JrOpH78g; dkim=pass (1024-bit key) header.d=ericsson.com header.b=ge0euI/d
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 MqABLw9vni94 for <gen-art@ietfa.amsl.com>; Thu, 3 Jan 2019 07:01:30 -0800 (PST)
Received: from sessmg23.ericsson.net (sessmg23.ericsson.net [193.180.251.45]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 50213130DD9 for <gen-art@ietf.org>; Thu, 3 Jan 2019 07:01:27 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; d=ericsson.com; s=mailgw201801; c=relaxed/relaxed; q=dns/txt; i=@ericsson.com; t=1546527684; x=1549119684; h=From:Sender:Reply-To:Subject:Date:Message-ID:To:CC:MIME-Version:Content-Type: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=vqkFxLj4jsdts6j4FssLZDZ8BnNzh+c53n/CuDSw2P0=; b=JrOpH78g+pUpid6dakOXnrO5D9DCkKqi1CBHegDrQu3kJ6C0J4j0UgjhYBVSLw+T OrL7IO2Y1PQqXSx+bLkniQIewfagNGO36I1o3r3Lqg5yL6CRuOnOcVZHza52V4sK s3ReTD7xQDMg3X1fPmuEu1wkoaHm9F56bHwusxtemng=;
X-AuditID: c1b4fb2d-2198b9e00000062f-a3-5c2e23c4ba7b
Received: from ESESBMB503.ericsson.se (Unknown_Domain [153.88.183.116]) by sessmg23.ericsson.net (Symantec Mail Security) with SMTP id 1F.CE.01583.4C32E2C5; Thu, 3 Jan 2019 16:01:24 +0100 (CET)
Received: from ESESSMB503.ericsson.se (153.88.183.164) by ESESBMB503.ericsson.se (153.88.183.170) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1466.3; Thu, 3 Jan 2019 16:01:21 +0100
Received: from EUR02-AM5-obe.outbound.protection.outlook.com (153.88.183.157) by ESESSMB503.ericsson.se (153.88.183.164) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1466.3 via Frontend Transport; Thu, 3 Jan 2019 16:01:20 +0100
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ericsson.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vqkFxLj4jsdts6j4FssLZDZ8BnNzh+c53n/CuDSw2P0=; b=ge0euI/dZ3cbQFF/ebHfmHDkzzRadchHs4EPRp02gq3kre3iTR2t+VlCQQZ3Lr1AV8lPDao/DYW3WxIggy8N5H3aev/JDwtDJTTueQY9dXMjCoOzzgmLk4seiFqDcYSkoBW/ZQ2glCyGNEBgcYnEsXHU3yIWMH5d+WjVOXFmheY=
Received: from HE1PR0701MB2746.eurprd07.prod.outlook.com (10.168.188.140) by HE1PR0701MB2809.eurprd07.prod.outlook.com (10.168.91.135) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1495.6; Thu, 3 Jan 2019 15:01:18 +0000
Received: from HE1PR0701MB2746.eurprd07.prod.outlook.com ([fe80::1f9:b4c9:ef3d:32c9]) by HE1PR0701MB2746.eurprd07.prod.outlook.com ([fe80::1f9:b4c9:ef3d:32c9%4]) with mapi id 15.20.1516.000; Thu, 3 Jan 2019 15:01:18 +0000
From: Francesca Palombini <francesca.palombini@ericsson.com>
To: "Rabadan, Jorge (Nokia - US/Mountain View)" <jorge.rabadan@nokia.com>, "gen-art@ietf.org" <gen-art@ietf.org>, "martin.vigoureux@nokia.com" <martin.vigoureux@nokia.com>
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: AQHUl4RZLEwucbegYkGgkR+NMODVWaWdzBWA
Date: Thu, 03 Jan 2019 15:01:18 +0000
Message-ID: <21361E84-BDBF-4035-9B02-427EAD998D4E@ericsson.com>
References: <154480398817.30540.8239762064497446504@ietfa.amsl.com> <69F9BBF9-C52E-45F9-BE65-E466140D6823@nokia.com>
In-Reply-To: <69F9BBF9-C52E-45F9-BE65-E466140D6823@nokia.com>
Accept-Language: en-GB, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [158.174.219.143]
x-ms-publictraffictype: Email
x-microsoft-exchange-diagnostics: 1; HE1PR0701MB2809; 6:JhVI4Y82xFvGuhoFTmC/5GpMZiEe/eOgUJOvvByB38UqmXvyPP2JlxuLJfayrcIY2g2rD/lC1cQuEGA9ex45Du1vgnTLzwuLF19nQxOEaLdHNBVXMkDPgG5YwbuckBVS6yMAtUfaRvisf8xu2McRS+tgnkpmLdmIiHzD/7VGSeEArMdbogjRbb5cK54rOHZ+UJ4LK96VlIXhYpet3wEYzlslBTDdIPpurD2FCr04mOCjUo+oXPjmN5+0SzdKLE2bfstghrnrdQlENk/yFFTY9Ec+6v7OevYjUsc4vKUmcEhBMjfNoUxQStftlQGvsIonBM157cRf09ADDEZyAYXjQj8UbEPBTDNDLHFe6IaNXdWmknTlO6csz8mCmQlGtC79jeR7ZThL4w02RgVZdH3xv0VV2bvJl75RpJmASEthSQAtk8eBkIHY5V89WrrnseDY+h5gvTXVB8Hz5PXiAJkiGg==; 5:PGQp9b+5b8BCrzjMq9t+mkkSA8aMMsZZcKxFw23dVqn3LXfztUPC2gs20DyMSIKu3yzaoxhEz7pTbypo1kJkOLc38WcsOGayXzv/thPOwSB1I9AtWFNMEh5c19T4bH0+wY+rETNcbNGAIkpgHNy/md6E0STLB6AB15MiVb8YGk4mpxlR/ygMHyUikcoWRNEr4CDxEWqAL+E0VEqn80KsNA==; 7:VkhcrJtFAYnvsIny+ZkstpC4ZN6EYtwDbpmaqEMNRXBGxpIAD6iPfrHa5OYxZ5VTr+tHV1XeiUb+/iv1TKKfUPBMYtAghae/0Tm+gYfJy6yZ+vA/rddKv/MAURmNlrax5wUIjVqTHhlRmEulZ0z8mQ==
x-ms-exchange-antispam-srfa-diagnostics: SOS;
x-ms-office365-filtering-correlation-id: c0a57063-ca48-4ffa-b851-08d6718c50ff
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600109)(711020)(2017052603328)(7153060)(7193020); SRVR:HE1PR0701MB2809;
x-ms-traffictypediagnostic: HE1PR0701MB2809:
x-microsoft-antispam-prvs: <HE1PR0701MB28097C13EBD096804890193E988D0@HE1PR0701MB2809.eurprd07.prod.outlook.com>
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(8211001083)(3230021)(908002)(999002)(5005026)(6040522)(8220060)(2401047)(8121501046)(10201501046)(3002001)(93006095)(93001095)(3231475)(944501520)(52105112)(6041310)(20161123558120)(20161123562045)(20161123564045)(201703131423095)(201703031522075)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123560045)(201708071742011)(7699051)(76991095); SRVR:HE1PR0701MB2809; BCL:0; PCL:0; RULEID:; SRVR:HE1PR0701MB2809;
x-forefront-prvs: 0906E83A25
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(346002)(39860400002)(136003)(366004)(376002)(396003)(199004)(189003)(13464003)(51444003)(478600001)(6246003)(86362001)(256004)(14444005)(11346002)(6346003)(25786009)(33656002)(8936002)(36756003)(5660300001)(446003)(99286004)(71190400001)(83716004)(71200400001)(6486002)(2501003)(97736004)(186003)(6512007)(4001150100001)(229853002)(6306002)(305945005)(966005)(102836004)(2201001)(2906002)(14454004)(81156014)(81166006)(7736002)(8676002)(6436002)(476003)(6506007)(76176011)(486006)(82746002)(6116002)(3846002)(2616005)(66066001)(4326008)(296002)(54906003)(106356001)(110136005)(105586002)(316002)(53936002)(44832011)(53546011)(68736007)(26005); DIR:OUT; SFP:1101; SCL:1; SRVR:HE1PR0701MB2809; H:HE1PR0701MB2746.eurprd07.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1;
received-spf: None (protection.outlook.com: ericsson.com does not designate permitted sender hosts)
authentication-results: spf=none (sender IP is ) smtp.mailfrom=francesca.palombini@ericsson.com;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam-message-info: LF9VkUD3F+QJHkVGojX1Kbpi41XbCwKnqlAQSlWjdWkSCmSqJ3+W2dVcvazSx7CmfPEuihsM19tfqJHdJDBOi8XAhMIB9bfzKynWYz6g3Z+qOlWnfhnEqrwNqi65p4bn7vvB6IREdXos8mPrjOs9CZpPkZMzmLPQCd7/8yXglF6MvBXqaB5E//oeBzFin46fdJMA08xlKTY+0HWnem+mIb4RWQEYQHl9XwlAA+HyFM4xh7i1H5xeKfiCcPojxzZtgnzZtMwb4eWN5FaCOEGd2rEfwSlTBX7kP45Vu3/a4tr3KgoJykDKd1ptCwXW87L1
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="utf-8"
Content-ID: <F94BA04188BB1C4F844AFC5EC4D5856B@eurprd07.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: c0a57063-ca48-4ffa-b851-08d6718c50ff
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Jan 2019 15:01:18.6393 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 92e84ceb-fbfd-47ab-be52-080c6b87953f
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0701MB2809
X-OriginatorOrg: ericsson.com
X-Brightmail-Tracker: H4sIAAAAAAAAA02Sa0hTYRjHec85246j0eua+WD6wRlBpdNp2PJaSHiJIhGki2QrDypeJjsq KX0w0rSti4gmW+Qlh1oGXgq00sI1vJUo00JMY85LaomWGkZmbTsGffv9n///eXjeh5cmxY94 bnRqZjajzlSmS/lCSnemPdvH5CVL8Otrkygae3WkwjzfQSjeL65SirnWakoxufGbr6j6XEwd 5UcZDD+JqMlxM3GaOCcMSWLSU3MZtW/YRWGKTqMls+bjrgw3/uEVoNFYDXKiAR+Cd7NWQoOE tBibEJjWblGcWEfQUlPH40QdAYPWEdIuKFxKgvZmHeKcewQM3N3cHjCNoL6ml2+fzMchMDy1 7OiX4GYE+rZVRwuJuxBUNDwl7Kld+AJYyi08O0twIqwt/KA49gdzZb/AzhTeC5OLtaSdRTgc rFuNjl4xzoX1oVFH3gmHgrakwMEIe8DatSZHnsSuMD5TTXBvxWDoHCI5doGF6S0ex16wPPwW cewB5mqtY1HAhQLo1PRsGz6wUlGx3XwSpprGBVzIjGCmpWTb8IaVxQoBxyoY0w8SXGiOhPaF LtthaJtwh8lX8Vy9mw8fHhhRKZLr/9tWb4uReD80v/DlylFQWdjL49gTyrVTAr3jGM7Qr5uh ahDvMXJhGZbNSPYPkDHq1Mssq8qUZTLZbcj2gbqf/fLpQE1fjhkRppF0h6jJXZYg5ilz2bwM IwKalEpEu1nvBLEoSZmXz6hVieqcdIY1oj00JXUVbYqdE8Q4WZnNpDFMFqP+5xK0k1sBOmXx fBJp+vgmqLosOqg1OC0nWssMHKZV9d+TIpbO+pX39/nJc4hvzuyIplXKxKTtM/WUHWwuqm2Q RASMPQ+9qsq/n18aaIicKAqT6IrSJ+LlO1+fMAUvXT9/6bhryWDVkYefTON3kudvW2INy+yN mGJ3T2uU5WV4R9LG5tfA2TgpxaYo5QdINav8C5cISb48AwAA
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/-KC1oKs25v1wGsTFe_MpKVkH7Qw>
Subject: Re: [Gen-art] [bess] Genart last call review of draft-ietf-bess-evpn-df-election-framework-06
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
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: <https://mailarchive.ietf.org/arch/browse/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: Thu, 03 Jan 2019 15:01:48 -0000

Hi Jorge,

Thanks for your update. I am ok with the resolution of the nits / editorial, thank you.

After looking more into the updating issue, I think that the main question here is: do you consider that this will be widely deployed? (As in, you would expect future deployments to widely implement these mechanisms?) 

In answer to Adrian you wrote:
"The intend is definitively not to update RFC7432 but to specified new procedures, that was the agreement so far. In other words, this work does not mandate an upgrade of all the systems supporting RFC7432. The RFC7432 are still fine. Maybe we need to rephrase that sentence? "

I just want to clarify that saying that your doc updates RFC7432 doesn't mean that it becomes mandatory for all the systems.
I understand that you do not want to upgrade all the systems that already support RFC7432. But from my understanding an update to a document can also be an extension or optional mechanism (which is what you have here), which is considered to be an important addition to the existing RFC. We have an example in the CoRE wg with RFC7252 (Constrained Application Protocol) which is updated by RFC7959 (Block-Wise Transfers in the Constrained Application Protocol), which is an optional features, but which specifies:

   A CoAP implementation that does not support these options generally
   is limited in the size of the representations that can be exchanged,
   so there is an expectation that the Block options will be widely used
   in CoAP implementations.  Therefore, this specification updates
   RFC 7252.

This is the only "open point" I have left, and is more of a question than a strong concern. I cc Martin for more input, but if we don't get any, this will be my Gen-ART review of v-07. Then it's up to the IESG to say what's the right way.

Thanks for all your work!
Francesca

On 19/12/2018, 11:19, "Rabadan, Jorge (Nokia - US/Mountain View)" <jorge.rabadan@nokia.com> wrote:

    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