[RTG-DIR] Review of draft-ietf-idr-bgpls-segment-routing-epe

Ravi Singh <ravis@juniper.net> Fri, 14 April 2017 06:02 UTC

Return-Path: <ravis@juniper.net>
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 26F15129407; Thu, 13 Apr 2017 23:02:50 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.921
X-Spam-Level:
X-Spam-Status: No, score=-1.921 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H4=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_HELO_PASS=-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=junipernetworks.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 C0dOF6-C6fvH; Thu, 13 Apr 2017 23:02:48 -0700 (PDT)
Received: from NAM01-BN3-obe.outbound.protection.outlook.com (mail-bn3nam01on0105.outbound.protection.outlook.com [104.47.33.105]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 2A7A1128B90; Thu, 13 Apr 2017 23:02:47 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=junipernetworks.onmicrosoft.com; s=selector1-juniper-net; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=Jf7cg3xCs8WMOSKb6vWyZTacrKWEwy7rWsOCNAoO400=; b=Jppgy70AlAskIFh9kE+NW1Lv3yWj9E42tf5MUxPi1mu2x1UtHtKLlccnIG1oDnJMP90EUGaKsDR9yO5XlnPN7jZ7YFDqXTO5gnWuCceDO70AdsKoXQtlGUr12QDa2gN3edbV/DvNhC8kYB3kzwPz20L4vs/SMf56CxX88yiN/b8=
Received: from CY1PR05MB2521.namprd05.prod.outlook.com (10.167.10.136) by CY1PR05MB2524.namprd05.prod.outlook.com (10.167.10.139) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1034.5; Fri, 14 Apr 2017 06:02:45 +0000
Received: from CY1PR05MB2521.namprd05.prod.outlook.com ([10.167.10.136]) by CY1PR05MB2521.namprd05.prod.outlook.com ([10.167.10.136]) with mapi id 15.01.1047.006; Fri, 14 Apr 2017 06:02:45 +0000
From: Ravi Singh <ravis@juniper.net>
To: "rtg-dir@ietf.org" <rtg-dir@ietf.org>
CC: "draft-ietf-idr-bgpls-segment-routing-epe@ietf.org" <draft-ietf-idr-bgpls-segment-routing-epe@ietf.org>, "idr@ietf.org" <idr@ietf.org>
Thread-Topic: Review of draft-ietf-idr-bgpls-segment-routing-epe
Thread-Index: AdK044TTNOR7tL3ASl6OLN2+c3aCsQ==
Date: Fri, 14 Apr 2017 06:02:45 +0000
Message-ID: <CY1PR05MB25215A10F4E4372407A31F13AB050@CY1PR05MB2521.namprd05.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: ietf.org; dkim=none (message not signed) header.d=none;ietf.org; dmarc=none action=none header.from=juniper.net;
x-originating-ip: [66.129.239.10]
x-microsoft-exchange-diagnostics: 1; CY1PR05MB2524; 7:nU/a7Nc5BpIiZUbJ+4/LJTNTfbZhYLbmmSUSztHxv7Gt8d9A0h3SxV78SbRtoI7Xus9GcGKM+Ov5Gns/9F4riFLgeqK/IR+z+uimXBBJtK4B+REe65eoXkYtxBZmwynIPROk4y4IlXdhJOWILaZe0wjfKcMQLfzjaz/fSjDqzmkPpsXj0S2OComqNJqg/tNu+8kh0QeU71H3Vga4SndjrGR2tIBIhbul+of6LV5sYVNK6oGY87Yy9cK3HnJj7esXgxM9tqPE82xxqgIx/uL0PVQ8y+dnOUshDB+Ay+otqUvx7BSSPr01izinYdWKcA9+shVFRI5nJIbRot+lJnGeTQ==
x-ms-office365-filtering-correlation-id: fe060ba1-19bc-41bd-b95e-08d482fbdf1e
x-ms-office365-filtering-ht: Tenant
x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017030254075)(48565401081)(201703131423075)(201703031133081)(201702281549075); SRVR:CY1PR05MB2524;
x-microsoft-antispam-prvs: <CY1PR05MB252421EA683B7F5956A0A196AB050@CY1PR05MB2524.namprd05.prod.outlook.com>
x-exchange-antispam-report-test: UriScan:(192374486261705)(21748063052155);
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(6040450)(601004)(2401047)(8121501046)(5005006)(93006095)(93001095)(10201501046)(3002001)(6055026)(6041248)(201703131423075)(201702281528075)(201703061421075)(20161123564025)(20161123560025)(20161123555025)(20161123562025)(6072148); SRVR:CY1PR05MB2524; BCL:0; PCL:0; RULEID:; SRVR:CY1PR05MB2524;
x-forefront-prvs: 02778BF158
x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(6009001)(39840400002)(39450400003)(39860400002)(39410400002)(39400400002)(39850400002)(790700001)(3846002)(102836003)(86362001)(6116002)(5660300001)(189998001)(25786009)(450100002)(68736007)(54356999)(50986999)(7736002)(110136004)(38730400002)(74316002)(4326008)(5630700001)(33656002)(230783001)(2501003)(2900100001)(77096006)(6916009)(9326002)(2351001)(9686003)(55016002)(54906002)(81166006)(8936002)(54896002)(8676002)(6306002)(3280700002)(2906002)(99286003)(6506006)(53936002)(3660700001)(122556002)(66066001)(5640700003)(7696004)(6436002); DIR:OUT; SFP:1102; SCL:1; SRVR:CY1PR05MB2524; H:CY1PR05MB2521.namprd05.prod.outlook.com; FPR:; SPF:None; MLV:ovrnspm; PTR:InfoNoRecords; LANG:en;
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: multipart/alternative; boundary="_000_CY1PR05MB25215A10F4E4372407A31F13AB050CY1PR05MB2521namp_"
MIME-Version: 1.0
X-OriginatorOrg: juniper.net
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Apr 2017 06:02:45.7266 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: bea78b3c-4cdb-4130-854a-1d193232e5f4
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY1PR05MB2524
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtg-dir/zpgbX55jcGakhSTu47rBAQI0JTk>
Subject: [RTG-DIR] Review of draft-ietf-idr-bgpls-segment-routing-epe
X-BeenThere: rtg-dir@ietf.org
X-Mailman-Version: 2.1.22
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: Fri, 14 Apr 2017 06:02:50 -0000

Hi
I had been designated as the RTG-DIR reviewer for this draft.

Overall the draft is clear.
However, it could be use some editorial revision for improved readability.

Specific comments listed below:

1.       Document title could be less heavy on consecutive adjectives: some suggestions:
a.       BGP-LS extensions for Segment Routing BGP Egress Peer Engineering
b.      BGP-LS extensions for "BGP-EPE using SR"
c.       Segment Routing BGP Egress Peer Engineering extensions for BGP-LS
...
2.       Section 1:
"   This document defines new types of segments: a Peer Node segment
   describing the BGP session between two nodes; a Peer Adjacency
   Segment describing the link (one or more) that is used by the BGP
   session; the Peer Set Segment describing an arbitrary set of sessions
   or links between the local BGP node and its peers."
Above has an unintended meaning. This should instead have stated that this doc defines BGP-LS extensions to communicate the above listed SIDs that are defined in "draft-ietf-spring-segment-routing"

3.       Section 3:
"
   This document defines the BGP-EPE Peering Segments:

   o  Peer Node Segment (Peer-Node-SID)

   o  Peer Adjacency Segment (Peer-Adj-SID)

  o  Peer Set Segment (Peer-Set-SID)"

Same issue as listed above for section 1 above.
Section 5 has the same issue.

4.       Section 4.2: I'd suggest that this be broken into 2 separate sub-sections: each listing the mandatory/optional sub-TLVs for the local and remote node descriptors respectively. That will make for improved readability.

5.       Section 4.2: please clarify in text as to Why no "BGP-LS ID" in link NLRI in the remote node descriptor.

6.       Section 4.3:
a.       values of the flags are specified only for the Per-Adj-SID. Explicit text should be listed for per-node-SID and per-set-SID.
b.      "
   The Peer-Node-SID MUST be present when BGP-LS is used for the use
   case described in [I-D.ietf-spring-segment-routing-central-epe] and
   MAY be omitted for other use cases."
Is there really a need to state what other use-cases might do, considering that this doc is specific to BGP-LS for SR-EPE? Perhaps can be reworded. Similarly for "Peer-Adj-SID and Peer-Set-SID SubTLVs MAY" in next paragraph.

7.       There is some repetition of info between sections 4 & 5. eg. What local and remote node descriptors contain. Please reword these sections to avoid the repetition while still presenting the info.

8.       Sections 5.1 & 5.2: have lots of repetitive text. Tabular presentation of the text in these sections would allow describing per-node-sid and per-adj-sid without the repetition.

9.       Section 9: typo in language in first sentence.

10.   Section 10: please explicitly call out any additional (beyond rfc7752) security considerations or include text stating that none exist.

11.   Typos
a.        Section 3:
                                       i.            "an BGP-EPE" -> "a BGP-EPE"
b.      Section 4:
                                       i.            "Link-type" NLRI -> "link-state" NLRI?

Regards
Ravi