Re: [RTG-DIR] Rtgdir Last Call review of draft-ietf-lsvr-bgp-spf-13
"Acee Lindem (acee)" <acee@cisco.com> Wed, 30 June 2021 15:09 UTC
Return-Path: <acee@cisco.com>
X-Original-To: rtg-dir@ietfa.amsl.com
Delivered-To: rtg-dir@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id A12DF3A1F9F; Wed, 30 Jun 2021 08:09:35 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -9.595
X-Spam-Level:
X-Spam-Status: No, score=-9.595 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, HTML_MESSAGE=0.001, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_NONE=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 header.b=ENCV7RQh; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=ulrxxi2H
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 iXMQ5NUAWh4E; Wed, 30 Jun 2021 08:09:30 -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 3E2673A1F9E; Wed, 30 Jun 2021 08:09:30 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=75633; q=dns/txt; s=iport; t=1625065770; x=1626275370; h=from:to:cc:subject:date:message-id:references: in-reply-to:mime-version; bh=PJdN7zcrq/zEWNw6HjaQkq6rfcAN5dxaS3bHlyf8n8Q=; b=ENCV7RQh9ngAjLGRP53JdN5qg7lRzrTfzhCJDenLDi7zr/g9JC+v4aKD Y9X4bFZgFFAwtCPnY0ARfV5xC6WJ2qZ30RiJ5moD1dMiV/HOmmrz9krUZ RBDjE+oGN2dAXYCyjlYhS+aGJjgur6k+Dr5U66PpnOEQ7QiRffsakFjr9 A=;
IronPort-PHdr: A9a23:Z3XgShKqdzjgum+wcdmcuXsyDhhOgF28FgEQ45sjzblJd/fr85fjORnZ4vNgxB/MUJ7A4v1Jw+zRr+j7WGMG7JrA1RJKcJFFWxIfz8lDmQsmDZ2KDET6KLjhaClpVMhHXUVuqne8N0UdEc3iZlrU93u16zNaGhj2OQdvYOrvHYuHhMWs3Of08JrWMG11
IronPort-HdrOrdr: A9a23:PrsdZaDzHMruoDnlHegRsceALOsnbusQ8zAXPh9KKCC9I/b3qynxppsmPEfP+UkssHFJo6HmBEDyewKjyXcV2/hQAV7GZmnbUQSTXfpfBOfZsljd8mjFh5JgPMRbAutD4b/LfCJHZK/BiWHSebtNsbr3kpxAx92uskuFJjsaDZ2Imj0JcjpzZXcGPTWua6BJcKa0145inX6NaH4XZsO0Cj0uRO7YveDGk5rgfFovGwMnwBPmt0Lr1JfKVzyjmjsOWTJGxrkvtULflRbi26mlu/anjjfBym7o6YhMkteJ8KoHOCXMsLlQFtzfsHfvWG1TYczagNnzmpD21L8eqqiKn/7nBbUp15qeRBDunfKn4XiQ7N9n0Q6T9bbfuwq/nSQ8LwhKVPaoQuliA0fkAgMbzaNBOOgg5RPri7NHSRzHhyjz/N7OSlVjkVe1u2MrlaoJg2VYSpZ2Us4ckWUzxjIeLH47JlO21GnnKpgmMOjMoPJNNV+KZXHQuWdihNSqQ3QoBx+DBkwPoNac3TRalG1wixJw/r1Qol4QsJYmD5VU7eXNNapl0LlIU88NdKp4QOMMW9G+BGDBSQ/FdGiSPVPkHqcaPG+lke+53Fz03pDgRHUs9upypH3saiIuiYcCQTOnNSTV5uw+zvnkehTKYd3C8LAq26RE
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0CrCwACiNxg/4wNJK1ahAUwIy4Hd1o3MQuEPYNIA4U5iGwDilCPU4FCgREDVAsBAQENAQE1CgIEAQGEUgIXgloCJTgTAgQBAQESAQEFAQEBAgEGBHEThWgNhkUBAQEEEhEKEwEBJRIBDwIBCBEDAQIhAQkCAgIfER0IAgQBDQUigk8BgX5XAy8BDpwfAYE6AoofeoEygQGCBwEBBgQEhVQNAwiCMgmBOoJ7hAwBAYZhJxyCDYEUASccgmE+giA3CwKBRjoGgnE2gi6CMF8MBgE9GwsBAxQvDyMNIQE0GzoOARQLBgUeCgSSB4JHRYgdjSeQcjpbCoMgiiCOHYVgBSaDYIFFiW2WfZVoghmKDoMskAgmhGQCBAIEBQIOAQEGgkUqJIFZcBVlAYI+CUcXAg6SD4UUhUkBcwI2AgYBCQEBAwl8iD4tgQcBgRABAQ
X-IronPort-AV: E=Sophos;i="5.83,312,1616457600"; d="scan'208,217";a="814171979"
Received: from alln-core-7.cisco.com ([173.36.13.140]) by rcdn-iport-9.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 30 Jun 2021 15:09:28 +0000
Received: from mail.cisco.com (xbe-aln-001.cisco.com [173.36.7.16]) by alln-core-7.cisco.com (8.15.2/8.15.2) with ESMTPS id 15UF9Sf0003558 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Wed, 30 Jun 2021 15:09:28 GMT
Received: from xfe-aln-001.cisco.com (173.37.135.121) by xbe-aln-001.cisco.com (173.36.7.16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.792.15; Wed, 30 Jun 2021 10:09:28 -0500
Received: from xfe-rcd-002.cisco.com (173.37.227.250) by xfe-aln-001.cisco.com (173.37.135.121) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.792.15; Wed, 30 Jun 2021 10:09:27 -0500
Received: from NAM04-MW2-obe.outbound.protection.outlook.com (72.163.14.9) by xfe-rcd-002.cisco.com (173.37.227.250) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.792.15 via Frontend Transport; Wed, 30 Jun 2021 10:09:27 -0500
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ke/vnc+7oAXMM2cj7iHHCD3CI+IDv7NKEKSoOiDCkcRYgt2sMw/Tgw40L/VpuwVXve5V++C7WQeqN7UjI7UPmOVnTAWS7fwhD3uIeamc10rgWAQG+W7mVTukTymr+wkvSU3qLjVHPeKHt7AklNahfBLdABIarBZ7m5MVHjGgWDMNaLjd6GsLY2GgZow2+48Pe8Be/fknOjKnrnzo/GVVw/dpvLdWYP7akSPOxazrJPytiGjiZtj//q1dfOQmHV7diX8D6IYjMUWnocCZkonW4HL5pufp1ZDITDN4Cz4O20rMhL/zWkSNk90a5deAxetBrfPKLchn0iCiQhos8bY0zA==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=PJdN7zcrq/zEWNw6HjaQkq6rfcAN5dxaS3bHlyf8n8Q=; b=dzxaBMLSd1uZ1Oxrf9mmnDB5VXyoVobJvBcnRiJQWOtiUYSM3fDr66LSBxCD2FIJyj8CkGrtXK55eyEhlfkfjFKq5tyFGnOPGN+mryBBhnaS/D7/GCOGKMxYRRxdVR5xVQQB1M8YCZRTqX3/nW+JJj/gKNN+GYvDFseK4AgxDJ8LzC4iOPb0A0HLosTlKSiBRQPbrFnQfHqNjhas3Kp6hEk9Orb2TjsLGten+Bag3H6x/IeNQ1DEeX1Z/6v8GNu7JU2p5NTjYLFNaUSyRqmtZUEgRbvhWkFKI90WyyBBYguuXbNiRYDisOv0ec++aKxysLtltEFLkKAQipDB+THWQQ==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=cisco.com; dmarc=pass action=none header.from=cisco.com; dkim=pass header.d=cisco.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cisco.onmicrosoft.com; s=selector2-cisco-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=PJdN7zcrq/zEWNw6HjaQkq6rfcAN5dxaS3bHlyf8n8Q=; b=ulrxxi2HLYgTmBbeTSpy8ZNOAkKKY5SMhZKH6ofwc+4yC71NQumX/iXUZpriSfAhv8yLIDpEXl4F9SrFgbpUyMA8gAiElgmUTH8eyIzmxkd4Ta2zpSfYRku4XWi7UQNlOogAuTW+OLxJZC4BPM2eEN4psmd4AKtZ6EA7Jz4AB9o=
Received: from BL0PR11MB2884.namprd11.prod.outlook.com (2603:10b6:208:72::25) by MN2PR11MB4318.namprd11.prod.outlook.com (2603:10b6:208:17a::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4264.19; Wed, 30 Jun 2021 15:09:20 +0000
Received: from BL0PR11MB2884.namprd11.prod.outlook.com ([fe80::592e:30fd:2fa1:e919]) by BL0PR11MB2884.namprd11.prod.outlook.com ([fe80::592e:30fd:2fa1:e919%5]) with mapi id 15.20.4264.026; Wed, 30 Jun 2021 15:09:20 +0000
From: "Acee Lindem (acee)" <acee@cisco.com>
To: Yingzhen Qu <yingzhen.ietf@gmail.com>, "rtg-dir@ietf.org" <rtg-dir@ietf.org>, "draft-ietf-lsvr-bgp-spf@ietf.org" <draft-ietf-lsvr-bgp-spf@ietf.org>, "lsvr@ietf.org" <lsvr@ietf.org>
CC: "guntervandeveldecc@icloud.com" <guntervandeveldecc@icloud.com>, Luc André Burdet <laburdet.ietf@gmail.com>, "Yemin (Amy)" <amy.yemin@huawei.com>
Thread-Topic: Rtgdir Last Call review of draft-ietf-lsvr-bgp-spf-13
Thread-Index: AQHXZ59ANzP3r4lPy0uer8J93loFPasscX4A
Date: Wed, 30 Jun 2021 15:09:20 +0000
Message-ID: <F6C9EA73-1BCF-4C3C-AC9E-9261D180C1C7@cisco.com>
References: <305FC57A-8B6B-4718-B61E-A5E7D30431D1@gmail.com>
In-Reply-To: <305FC57A-8B6B-4718-B61E-A5E7D30431D1@gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/16.50.21061301
authentication-results: gmail.com; dkim=none (message not signed) header.d=none;gmail.com; dmarc=none action=none header.from=cisco.com;
x-originating-ip: [136.56.133.70]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 6cd52763-9d6c-4ddb-54a0-08d93bd90992
x-ms-traffictypediagnostic: MN2PR11MB4318:
x-microsoft-antispam-prvs: <MN2PR11MB4318756FB3CCA34BD645B5F3C2019@MN2PR11MB4318.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:9508;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: OXqgQdFvNEmosiucQkcTIWKYDMt9hCrSb4R5yaxjmLaZax80/l3K0t92S7b9Qq1Vccj4XooeOYziXzSJWfs5JrMPXIHGM5IDLguhLOHtix65iWetvBmCo0NqLymuCMBLeNOAVEP3i1Nu7TOPcuT2eE71s4yfEPwqIazuhix8niIC+ZstUGnzEgzC4d0CYUlfOiLC4BuMEPgYsU1ec4pqgieZBdld2mSN0s36Tlp7CH53wuCsP816Wb5PcYzk9DxoPJRfB2hHHgioXSSHZPkrP+i1+ykpdCYtgogKFJH98IR7HtE+0REiGjNNERlexRy5QValLUS41drcdBse4NdaCheiHqh52On4K3YbioRiBHnX91PP2+DDP4FLOb2m8qQWvKutqSxc8glLi0oygItaEwbg4Fy0VhUP+FeEN0fslPI8YVQEJ1nBuiRTRLBVy5WbNMVjcoByhA3+VbznTg8JgYqUqykuyIjRwZVB2kjK1hmWpEmmEIsr/YhExgVvX8jom3C6UU9hZFQDIybaM4Vt7KyjkKAzIYdfn0UvMqMZHE+QFs3gOv3Yzgy3ciRpozVYYyE5JW2RkduG0mk2yvTbnAqCHzhKfEFvnjiJxqwlH+Smb1hq8v9ZGP493cWkuNgKO7uFJfGDdqiBcy3dKRBFvxLqT8njFgIHKnjhMhjfmIMVzpzukQUuzdeLeanDbKNB8pCbC9JJCl0BR6JB9LAwECwX0/pfSjBz5yn51PZw00HbRHjPn3i6qX3OFdqJLemT/xPjGvUBKV3+zzFRWMsFiTnohHeD0BdrZf7p56uf51WNi5MI/aIYuY+xfbP/9zeQrzX60x8xjx+T6v4T+hxywQ==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:BL0PR11MB2884.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(366004)(346002)(376002)(136003)(39860400002)(396003)(2906002)(83380400001)(8936002)(66476007)(66556008)(91956017)(76116006)(166002)(8676002)(66946007)(4326008)(64756008)(66446008)(2616005)(5660300002)(38100700002)(86362001)(45080400002)(6512007)(966005)(186003)(71200400001)(122000001)(478600001)(26005)(36756003)(316002)(9326002)(6486002)(54906003)(110136005)(6506007)(33656002)(53546011)(45980500001)(579004); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: UoEjxof4Jp5geQBEG5REFNPSUyRxulPWTzIltVeBzGAnErHqasuYsWOHN0qI44Z8mQFrGywET9FsbvX8jcPOU1LtqflSt993W2vz6q1tES3ZycUd+8KjAOGGHKJo+NTLqH7/eukQU7XFu5mzo8dPYwH4nhk3+jxBxigaoCZ1ASbCLgaIGrCJHQikOc3kfHHbVPOes8gm0g29EV6Oi+xEFkHy9wf1liNnb5Ul2XEQeu+iv+T/bTKxWTLL9jD4OXSqKS1eyC293j01z05h23v5jTltHUBRyKN/lc6yTv6MqNY5beWKJFAabMGBey+6MkE9GO6sVgWjnHOZq/ecTEZZ2Uw52XFAtvSlndZYsSK+5Dc0S5SG/OHUGKN6eVKzjgWoVZqSRcglsbAXR/Bwtlg7GAEsIy7zut8OMEzlXUWiZU0zJiFwwP+AFZ/g8d5AqhqnJIPifEi/vWuJjldTb0XmNnKMngR9kYLtvELRX6EWvg9ZVc4eHs/N0JrkEF/zcnDObdwjqTmcoVwQgH2mpdZkXd2xPCA2UeuSiJfUY1FXPHD58D8NU0i58uL77rEU7kJ945mwVbcyM+lBkeGV1WN6egksfNjM6+mmuvnxuzGU0DDhb04Qlyy7o5vTGTUXPIws996g2lrz8k3jUgES5VGR2kZnTaBi+WMgfOCmGSVl9W1+ozm1Xfa6FoG7iyYbhd3TOw57uP2A/o9GSYSdwTZkLlh2c4d/wDA+qy8XJbJ6EyTHo3kSGDPTHy+lcCouYoBshqImu3esw7d39GuG2HFykzeShUPYrhObROBON8CsJxwegCOr2jYBD0OkGXbZohrCfEwP/7vZjBG49QvshAcxsvG1DVMDOi0XHmzMBCp6+fDMxfhbHdpWiiYjW4BxH2cOztEt29xOdlfC+UWWdu4z+Ot+a7f3a8X6d/B4PNzASIoFNbLess7QWLqtP8UDJjLN6nqLRVYDMb2Mia4X8WwfBm6CMprfMGn5iG2l/5OdhXaSZm1MvmLhqr/5AC251kosXBF5cijDjqTHDcGBb3kTh6KdpqcLDdcqd0NVJbscCLl/Zwfz6xzVWqtLKmOMuak/s8jgbGOdZdZ5FdI0A75LKidAHqODG4JgcQHo0DoDZAdwlOrEnZUIB4py65u9rzmCvnbXVKfPJmOgJVMcSemGJDqUkZKEaLHiua6pDSLXs9gcfDBFMogxsFvpcj0OiykbrP8YQ5SYd9fxvi4olRAGiA8mZrZH9aC8rNWkShQD1JySW6kIdzxxRhlkOCa3IaKZx8/YRDZPPvfxmjWgHFso88zHQO31BDgz09RrOAEZjT1AfNZGfqqR/JsvULxNhEJi/7SeAi+Yq5C5chxZJchobw==
x-ms-exchange-transport-forked: True
Content-Type: multipart/alternative; boundary="_000_F6C9EA731BCF4C3CAC9E9261D180C1C7ciscocom_"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BL0PR11MB2884.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6cd52763-9d6c-4ddb-54a0-08d93bd90992
X-MS-Exchange-CrossTenant-originalarrivaltime: 30 Jun 2021 15:09:20.2563 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5ae1af62-9505-4097-a69a-c1553ef7840e
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: r8qwEWdj3jaVLfZsXG+xim3vmOELsq2TUq9ecrL4hHqkOniXnoE736gHJ6KH67NS
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4318
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.36.7.16, xbe-aln-001.cisco.com
X-Outbound-Node: alln-core-7.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtg-dir/vRwTCBaiBNhULpF1Daw_O4caMPY>
Subject: Re: [RTG-DIR] Rtgdir Last Call review of draft-ietf-lsvr-bgp-spf-13
X-BeenThere: rtg-dir@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Routing Area Directorate <rtg-dir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtg-dir/>
List-Post: <mailto:rtg-dir@ietf.org>
List-Help: <mailto:rtg-dir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 30 Jun 2021 15:09:36 -0000
Hi Yingzhen, Thanks for the review. We’ve incorporated most of your comments into the -14 version. From: Yingzhen Qu <yingzhen.ietf@gmail.com> Date: Tuesday, June 22, 2021 at 3:46 PM To: Routing Directorate <rtg-dir@ietf.org>, "draft-ietf-lsvr-bgp-spf@ietf.org" <draft-ietf-lsvr-bgp-spf@ietf.org>, "lsvr@ietf.org" <lsvr@ietf.org> Cc: "guntervandeveldecc@icloud.com" <guntervandeveldecc@icloud.com>, Luc André Burdet <laburdet.ietf@gmail.com>, "Yemin (Amy)" <amy.yemin@huawei.com> Subject: Rtgdir Last Call review of draft-ietf-lsvr-bgp-spf-13 Resent-From: <alias-bounces@ietf.org> Resent-To: Acee Lindem <acee@cisco.com>, Shawn Zandi <szandi@linkedin.com>, Wim Henderickx <wim.henderickx@nokia.com>, Keyur Patel <keyur@arrcus.com> Resent-Date: Tuesday, June 22, 2021 at 3:45 PM I have been selected as the Routing Directorate reviewer for this draft. The Routing Directorate seeks to review all routing or routing-related drafts as they pass through IETF last call and IESG review, and sometimes on special request. The purpose of the review is to provide assistance to the Routing ADs. For more information about the Routing Directorate, please see http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir Although these comments are primarily for the use of the Routing ADs, it would be helpful if you could consider them along with any other IETF Last Call comments that you receive, and strive to resolve them through discussion or by updating the draft. Document: draft-ietf-lsvr-bgp-spf Reviewer: Yingzhen Qu Review Date: Jun 15th, 2021 Intended Status: Standards Track Summary: This document has some issues that should be at least considered prior to publication. Comments: Major issues: · Considering the SPF Status attribute TLV has the same type for Node, Link and Prefix NLRI, it is implied that a BGP update message can only contain a single kind of NLRI (for example, Node or Link). I’d suggest make the draft explicitly state it. I’ve added a paragraph on this in 5.1.1. If we were to make this a restriction, we’d need to discuss in WG. · About SPF Status TLV, use the Node NLRI as an example (section 5.2.1.2), it says in the draft: “If the SPF Status TLV is not included with the Node NLRI, the node is considered to be up and is available for transit traffic.”, then later, “If a BGP speaker received the Node NLRI but the SPF Status TLV is not received, then any previously received information is considered as implicitly withdrawn and the update is propagated to other BGP speakers. “. These two statements seem to conflict. I’ve fixed this for all the status TLVs. Comments inline: [Line numbers from idnits] 78 5.2.1. Node NLRI Usage . . . . . . . . . . . . . . . . . . . 11 79 5.2.1.1. Node NLRI Attribute SPF Capability TLV . . . . . 11 80 5.2.1.2. BGP-LS-SPF Node NLRI Attribute SPF Status TLV . . 12 [Minor]: section 5.2.1.1 should be changed to "BGP-LS-SPF Node NLRI Attribute SPF Capability TLV" to be consistent with other titles. Fixed. 226 Another potential advantage of BGP SPF is that both IPv6 and IPv4 can 227 both be supported using the BGP-LS-SPF SAFI with the same BGP-LS-SPF 228 NLRIs. [Minor]: potential advantage? I'd suggest remove "potential". [nits]: both IPv6 and IPv4 can both be supported Fixed both. 315 The BGP SPF extensions reuse the Node, Link, and Prefix NLRI defined 316 in [RFC7752]. The usage of the BGP-LS NLRI, metric attributes, and 317 attribute extensions is described in Section 5.2.1. [minor]: metric attributes? Might be better just remove it. s/Section 5.2.1/Section 5.2 Changed to just “attributes” and fixed reference. 357 hop sessions) and the direct connection discovery and liveliness 358 detection for the interconnecting links are independent of the BGP 359 protocol. the scope of this document. For example, liveliness [nits]: unfinished sentence: the scope of this document. Removed. 470 5.2. Extensions to BGP-LS 472 [RFC7752] describes a mechanism by which link-state and TE 473 information can be collected from IGPs and shared with external 474 components using the BGP protocol. It describes both the definition 475 of the BGP-LS-SPF NLRI that advertise links, nodes, and prefixes 476 comprising IGP link-state information and the definition of a BGP 477 path attribute (BGP-LS attribute) that carries link, node, and prefix [Major]: line 475: should be "BGP-LS NLRI" Good catch – fixed. 556 If the SPF Status TLV is received and the corresponding Node NLRI has 557 not been received, then the SPF Status TLV is ignored and not used in 558 SPF computation but is still announced to other BGP speakers. An 559 implementation MAY log an error for further analysis. If a BGP 560 speaker received the Node NLRI but the SPF Status TLV is not 561 received, then any previously received information is considered as 562 implicitly withdrawn and the update is propagated to other BGP 563 speakers. A BGP speaker receiving a BGP Update containing a SPF 564 Status TLV in the BGP-LS attribute [RFC7752] with a value that is 565 outside the range of defined values SHOULD be processed and announced 566 to other BGP speakers. However, a BGP speaker MUST not use the 567 Status TLV in its SPF computation. An implementation MAY log this 568 condition for further analysis. [nits]: s/MUST not/MUST NOT [minor]: s/BGP speaker/BGP SPF speaker There are multi places in the draft using "BGP speaker" instead of "BGP SPF speaker", please go over and fix when applicable. For example: 1408 excessive SPF calculations. When a BGP speaker detects that its section 7.1 Used “BGP SPF Speaker” consistently and fixed several “MUST not” instances. 864 When a BGP SPF speaker completely loses its sequence number state, 865 i.e., due to a cold start, or in the unlikely possibility that that [nits]: extra "that". Removed. 943 o Local Route Information Base (LOC-RIB) - This routing table 944 contains reachability information (i.e., next hops) for all 945 prefixes (both IPv4 and IPv6) as well as BGP-LS-SPF node 946 reachability. Implementations may choose to implement this with 947 separate RIBs for each address family and/or Prefix versus Node 948 reachability. It is synonymous with the Loc-RIB specified in 949 [RFC4271]. [nits]: s/Loc-RIB/LOC-RIB In this case, RFC 4271 uses “Loc-RIB”. 1017 * If the Current-Prefix's corresponding prefix is in the LOC-RIB 1018 and the cost is less than the current route's metric, the [major]: I think this meant to be "more than the current route's metric" Clarified this so that there is no ambiguity as to which metric is referenced. 1134 operate today (i.e., "Ships-in-the-Night" mode). There is no 1135 implicit route redistribution between the BGP address families. [major]: what about redistribution from other protocols, say OSPF? This has typically been out of scope in routing protocol documents – although I just saw a new draft dealing with BGP redistribution. In any case, I think this topic would warrant further discussion in a different draft. 1167 prior to withdrawal. If the link becomes available in that period, 1168 the originator of the BGP-LS-SPF LINK NLRI SHOULD advertise a more 1169 recent version of the BGP-LS-SPF Link NLRI without the SPF Status TLV 1170 in the BGP-LS Link Attributes. [major]: "without SPF Status TLV", this is related with the second “Major issues” above. Since all the SPF Status conditions indicate unreachability or a degraded state, this is correct. 1263 A BGP-LS-SPF Speaker MUST perform the following syntactic validation 1264 of the BGP-LS-SPF NLRI to determine if it is malformed. [minor]: there are a few places in the draft using "BGP-LS-SPF Speaker|speaker", may change to "BGP SPF speaker"? Yup – fixed all these. 1425 Within a BGP SPF Routing Domain, the IGP metrics for all advertised 1426 links SHOULD be configured or defaulted consistently. [major]: in section 5.2.2, it says "One possible default for metric would be to give each interface a cost of 1 making it effectively a hop count.". Why not define a default value so all implementations will be consistent? We discussed and don’t want to add a default as different deployments could have totally different requirements. Also, other IGPs don’t have default metrics (at least, the link-state ones). Thanks, Acee
- [RTG-DIR] Rtgdir Last Call review of draft-ietf-l… Yingzhen Qu
- Re: [RTG-DIR] Rtgdir Last Call review of draft-ie… Acee Lindem (acee)
- Re: [RTG-DIR] Rtgdir Last Call review of draft-ie… Yingzhen Qu
- Re: [RTG-DIR] Rtgdir Last Call review of draft-ie… Acee Lindem (acee)
- Re: [RTG-DIR] Rtgdir Last Call review of draft-ie… Yingzhen Qu
- Re: [RTG-DIR] Rtgdir Last Call review of draft-ie… Acee Lindem (acee)