Re: [bess] Last Call: <draft-ietf-bess-evpn-df-election-framework-06.txt> (Framework for EVPN Designated Forwarder Election Extensibility) to Proposed Standard

"Rabadan, Jorge (Nokia - US/Mountain View)" <jorge.rabadan@nokia.com> Wed, 19 December 2018 09:18 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 8D409130DF4 for <bess@ietfa.amsl.com>; Wed, 19 Dec 2018 01:18:40 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.967
X-Spam-Level:
X-Spam-Status: No, score=-1.967 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, RCVD_IN_MSPIKE_H2=-0.001, 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 9PkZKEHifrL6 for <bess@ietfa.amsl.com>; Wed, 19 Dec 2018 01:18:37 -0800 (PST)
Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-eopbgr70118.outbound.protection.outlook.com [40.107.7.118]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D0E77130DEF for <bess@ietf.org>; Wed, 19 Dec 2018 01:18:36 -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=RYlxG2jplq08pTT/Vpjc1KMU9oe1oKkxovkGL2v/WNc=; b=lYgdjvXvwLdQKcZC5xIgz8PtjfGJoOw6gRzbOOVBIuFAIptW3hzGzXmLwHhWCJpdZzo5i92S5g2KudJPur5bZ+yudNg71ASoz757foDO3NWiC3jJuJ8S7SihLf9R5c+6+SLWi83lvdTMPWBUBjajARgQ70GvlF3uEZlIbcXLfVw=
Received: from VI1PR07MB3853.eurprd07.prod.outlook.com (52.134.26.15) by VI1PR07MB4814.eurprd07.prod.outlook.com (20.178.8.83) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1446.9; Wed, 19 Dec 2018 09:18:28 +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 09:18:28 +0000
From: "Rabadan, Jorge (Nokia - US/Mountain View)" <jorge.rabadan@nokia.com>
To: "Satya Mohanty (satyamoh)" <satyamoh@cisco.com>, Anoop Ghanwani <anoop@alumni.duke.edu>, "bess@ietf.org" <bess@ietf.org>
Thread-Topic: [bess] Last Call: <draft-ietf-bess-evpn-df-election-framework-06.txt> (Framework for EVPN Designated Forwarder Election Extensibility) to Proposed Standard
Thread-Index: AQHUjDV6qJjRt0FJrk6kbhkpXVkNqaVy/kgAgACIdQCACz82AA==
Date: Wed, 19 Dec 2018 09:18:28 +0000
Message-ID: <56D3C9F8-136A-48A9-AFB4-A774C0FBC0C8@nokia.com>
References: <154397142840.4620.1360654856007304944.idtracker@ietfa.amsl.com> <CA+-tSzwdxwGTB99kozVhN-fb8EuxvD7d_i5t_c9b+y4Ee1dmyw@mail.gmail.com> <CAA5BD0C-1CBD-49F1-B61F-B05B430754E7@cisco.com>
In-Reply-To: <CAA5BD0C-1CBD-49F1-B61F-B05B430754E7@cisco.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
x-originating-ip: [88.14.53.100]
x-ms-publictraffictype: Email
x-microsoft-exchange-diagnostics: 1; VI1PR07MB4814; 6:CC6jHrzfvVmqDENHVdUTupJ7HTD6kwojI3oBRLR5dsC+E4Zkb/+Oq0M+youPlM5ypnCD7/1/KJVJtwNimf2GU0TfD/22fzB9vWbvXK+V/aoVMfppJNO5YAiJIpNxFMMhgsl85pBCeCxA3+YSqcCDfTWz3U3CA+kzLsaaly6WB5STKOzoiPw3FOTjkTERxfJc2u1COI1LiH8/MY9Pf5Klp9W4pXqammgykzH3KcIdMTW368IOQNn5fJp2PVND1X4GzQhgzqs4TKA1m1UjDqc6fc3Chz75Ak05De1noW116sN9zz2prwsfOFMaBca/PDmSqrRckxN5BXHG1wyfpot+Md9MKOiM9NQ2RiNiqivgp9sWPAFPA22vQB/yg+Ck9F4KjcCIsNfBMEqFZFY1EixUpEE6J0VivS4IqA2CTtXPhFu6HYtsi2v/RfSg+8JzIgERbEvypPJjK1MRzGECK9BbwQ==; 5:jCuazCjTmgRzlS89IjlCB2K7/lBAwMcan/2FHEs7NYof0IhUIQJ+dmMRXTA8i4FVUYRGC7m2SyhEhSYVuCsqr366vAyDztK2DIiFIfQoWBK+GeLL+ca3Kvzip5SocFibmk3fZkeosTypKItCAqlSDx5W2xo16uAawM39qr4yUus=; 7:do++qEHmCElRMMR07pfw6nSP+h2lvi/lg/9LN0sCrze9wtwnYgrFqvAj/uLuXIwKkhc0rkHTeyx7aWQ/VkpdEZEjZIXZw/adUVQoSejGTaasuA0oXhRvgJAhxtKPIHhMhLs98mKcc3bo+nfX9yp3dw==
x-ms-exchange-antispam-srfa-diagnostics: SOS;
x-ms-office365-filtering-correlation-id: ff779624-8578-4bd6-4348-08d66592efc6
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:VI1PR07MB4814;
x-ms-traffictypediagnostic: VI1PR07MB4814:
x-microsoft-antispam-prvs: <VI1PR07MB48142E620F233C38B28A70F6F7BE0@VI1PR07MB4814.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)(5005020)(6040522)(2401047)(8121501046)(3002001)(10201501046)(3231475)(944501520)(52105112)(93006095)(93001095)(6055026)(148016)(149066)(150057)(6041310)(20161123562045)(20161123560045)(20161123564045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123558120)(201708071742011)(7699051)(76991095); SRVR:VI1PR07MB4814; BCL:0; PCL:0; RULEID:; SRVR:VI1PR07MB4814;
x-forefront-prvs: 0891BC3F3D
x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(346002)(396003)(366004)(376002)(39860400002)(136003)(189003)(199004)(13464003)(43544003)(305945005)(14454004)(86362001)(76176011)(105586002)(66066001)(83716004)(97736004)(6116002)(478600001)(33656002)(71190400001)(6506007)(53546011)(256004)(102836004)(3846002)(6246003)(25786009)(82746002)(8676002)(6346003)(81156014)(81166006)(2171002)(53936002)(26005)(36756003)(186003)(8936002)(5660300001)(11346002)(446003)(71200400001)(316002)(6306002)(476003)(2616005)(7736002)(6512007)(2501003)(99286004)(6436002)(68736007)(966005)(106356001)(229853002)(110136005)(2906002)(58126008)(6486002)(345774005)(486006); DIR:OUT; SFP:1102; SCL:1; SRVR:VI1PR07MB4814; 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)
authentication-results: spf=none (sender IP is ) smtp.mailfrom=jorge.rabadan@nokia.com;
x-microsoft-antispam-message-info: +Lf/M/Kgw0cIktiAPOhuTAtqVvIY4ekEPgYz7KQKo5HO/xEPA4BLyDM9DOK7bizV9pV2zfidliwRiltixP61knfLTIozVBbETG2yJfBkZk5z+CjDMxpCddDFDfEQzSG/V6xVkXZjCi4wmqPOFu4eeydR2NsuqGAMYVL/NP+YWZk8VgfSWvJT+0EN1Qm9E7NLN41XCFoDx5pyozBjWWV/lUFNAXp/jf86kcwKX2cGhNsEWrOcbt09d48PPm95JUmH6be+40uoAXm0xKhJOlhouH7bkzsij/vE1fO4AnHL59zQZ/f6TFZ0+cg6wARb/uBl
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="utf-8"
Content-ID: <A797DDB820BB11498A8B5CC25390BE9C@eurprd07.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: nokia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ff779624-8578-4bd6-4348-08d66592efc6
X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Dec 2018 09:18:28.1041 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5d471751-9675-428d-917b-70f44f9630b0
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR07MB4814
Archived-At: <https://mailarchive.ietf.org/arch/msg/bess/BeAv-MUG9BIbyT6ZrNTPi71hB_w>
Subject: Re: [bess] Last Call: <draft-ietf-bess-evpn-df-election-framework-06.txt> (Framework for EVPN Designated Forwarder Election Extensibility) to Proposed Standard
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 09:18:40 -0000

Hi Anoop,

Thank you for your review. We took all your comments.
Please see in-line how we are resolving them in rev 07.

Thanks.
Jorge

-----Original Message-----
From: BESS <bess-bounces@ietf.org> on behalf of "Satya Mohanty (satyamoh)" <satyamoh@cisco.com>
Date: Friday, December 7, 2018 at 6:09 PM
To: Anoop Ghanwani <anoop@alumni.duke.edu>, "bess@ietf.org" <bess@ietf.org>
Subject: Re: [bess] Last Call: <draft-ietf-bess-evpn-df-election-framework-06.txt> (Framework for EVPN Designated Forwarder Election Extensibility) to Proposed Standard

    Hi Anoop,
    
    Thank you very much for your editorial comments and review.
    We will take care of it.
    
    Best,
    --Satya
    
    
    On 12/7/18, 1:01 AM, "BESS on behalf of Anoop Ghanwani" <bess-bounces@ietf.org on behalf of anoop@alumni.duke.edu> wrote:
    
        I have reviewed the doc and I have mostly editorial comments.
        
        Thanks,
        Anoop
        
        ==
        
        Throughout
        
        VLAN Bundle, VLAN bundle, VLAN-Bundle, VLAN-bundle -- make consistent
        VLAN Aware Bundle, VLAN-aware bundle, VLAN-Aware Bundle -- make consistent
        bridge table, Bridge Table -- make consistent (also add definition to
        terminology section)
        DF election, DF Election -- make consistent
        Default DF Election, default DF Election -- make consistent
        non-DF -> NDF
[JORGE] done, thx
        
        Section 1
        
        double Q-in-Q tags -> Q-in-Q tags
        
        double is redundant
[JORGE] done, thx
        
        Section 2.1
        
        Fig 1 is a bit confusing.  If the idea of the rectangle is to show a
        core, then why have connections between PE1 and PE2, PE3, but not
        between PE1 and PE4?
[JORGE] good point, fixed it, thx
        
        Change
        >>>
        Layer-2 devices are particularly susceptible to forwarding loops
        because of the broadcast nature of the Ethernet traffic.
        >>>
        to
        The effect of forwarding loops in a Layer-2 network is particularly
        severe because of the broadcast nature of Ethernet traffic and the
        lack of a TTL.
[JORGE] done, thanks.
        
        Section 2.2.1
        
        a v4 or v6 peering -> an IPv4 or IPv6 peering
[JORGE] done, thanks.
        
        >>>
        >From a forwarding perspective, this is
        a churn, as it results in re-programming the PE ports as either
        blocking or non-blocking at potentially all PEs when the DF changes.
        >>>
        
        Why would the reprogramming change at all PEs?  It should change for
        at most 2 PEs for each (ES,EVI) being reprogrammed.  Maybe authors
        were trying to convey something else?
[JORGE] changed to: 
***From a forwarding perspective, this is
   a churn, as it results in re-programming the PE ports as either
   blocking or non-blocking at the PEs where the DF state changes.***
        
        
        Section 2.3
        
        >>>
        DF Election procedure Generally
        >>>
        Missing a period.
[JORGE] done, thanks.
        
        
        Section 3
        
        specification in EVPN -> EVPN specification
[JORGE] done, thanks.
        
        
        Section 3.1
        
        DF WAIT, DF_WAIT -- make consistent
[JORGE] done, thanks.

        DF Wait timer -- where is this defined?
[JORGE] This is defined in [RFC7432]. I added a reference.

        Ethernet Segment Route -> Ethernet Segment route
        stop DF timer ->  stop DF wait timer (?)
        start DF timer -> start DF wait timer (?)
[JORGE] done, thanks.
        
        Section 4
        
        rather than the state of the server states -> rather than the state of
        the server (?)
[JORGE] done, thanks.
        
        Section 4.2
        
        Si is the IP address of server i -> Si is the IP address of PE i
[JORGE] done, thanks.

        operator chooses so -> operator so chooses
[JORGE] done, thanks.

        Note 0 <= i,j <= Number of PEs -- should this be "< Number of PEs"?
        Weight(V, Es, Sk) -> Weight(v, Es, Sk)
        Pseudo-random -> pseudo-random
        efficient deterministic -> efficient and deterministic
        V4 -> IPv4
        V6 -> IPv6
[JORGE] all of them changed. Thanks.
        
        _______________________________________________
        BESS mailing list
        BESS@ietf.org
        https://www.ietf.org/mailman/listinfo/bess
        
    
    _______________________________________________
    BESS mailing list
    BESS@ietf.org
    https://www.ietf.org/mailman/listinfo/bess