Re: [Idr] Roman Danyliw's No Objection on draft-ietf-idr-bgp-ls-segment-routing-ext-15: (with COMMENT)

"Ketan Talaulikar (ketant)" <ketant@cisco.com> Thu, 27 June 2019 17:37 UTC

Return-Path: <ketant@cisco.com>
X-Original-To: idr@ietfa.amsl.com
Delivered-To: idr@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 63D74120295; Thu, 27 Jun 2019 10:37:57 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.5
X-Spam-Level:
X-Spam-Status: No, score=-14.5 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-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=bnjlbxH8; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=ysfgYxoH
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 Wfrtu5oS_Li5; Thu, 27 Jun 2019 10:37:55 -0700 (PDT)
Received: from rcdn-iport-7.cisco.com (rcdn-iport-7.cisco.com [173.37.86.78]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 43B9712027B; Thu, 27 Jun 2019 10:37:55 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=5038; q=dns/txt; s=iport; t=1561657075; x=1562866675; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=NIpBPHCixjNzVajqUVtpJPQxSirKM1RbOyIGCsReECM=; b=bnjlbxH8Xd8Z0xbzyk04bG/86Yw+kzmjq26qkwQXoD/PUbzSMIiCcMku mZGgZxrUf92onwYgndVQeMDovbTWwf9QPrDSPSya05vAUc7QFOZ6m4VBT 5uFXtGYo/hg9GTjzMfl2rrbRwSKvbE3XiTBj+B6l6mq3wcAdW4TXOPwIN o=;
IronPort-PHdr: =?us-ascii?q?9a23=3A69alDhUhD3FgtsgcTh36/L7rGmfV8LGuZFwc94?= =?us-ascii?q?YnhrRSc6+q45XlOgnF6O5wiEPSANSJ8OpK3uzRta2oGXcN55qMqjgjSNRNTF?= =?us-ascii?q?dE7KdehAk8GIiAAEz/IuTtank5EdhLUkNN9HCgOk8TE8H7NBXf?=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0AIAAD0/RRd/5xdJa1lGgEBAQEBAgE?= =?us-ascii?q?BAQEHAgEBAQGBUwUBAQEBCwGBQ1ADalUgBAsohBmDRwOEUooIglt+iE2NdIE?= =?us-ascii?q?uFIEQA1QJAQEBDAEBJQgCAQGEQAIXgmkjNAkOAQMBAQQBAQIBBW2KNwyFSgE?= =?us-ascii?q?BAQQSEREMAQE3AQsEAgEIDgMEAQEDAiYCAgIfERUICAIEAQ0FCBqDAYFqAx0?= =?us-ascii?q?BDpsrAoE4iGBxgTKCeQEBBYE2Ag5BgwENC4IRAwaBDCgBi14XgUA/gRABRoI?= =?us-ascii?q?XNT6CGkcBAQECAYEqARIBBxokgmQygiaMD4IgL5phBiY/CQKCF4ZShGeESwS?= =?us-ascii?q?ECIIrhxeEDIoQjSmBMIYIgXCNdgIEAgQFAg4BAQWBUDhnWBEIcBWDJ4JBDBc?= =?us-ascii?q?UgzqFFIU/cgGBKIs1gkMBAQ?=
X-IronPort-AV: E=Sophos;i="5.63,424,1557187200"; d="scan'208";a="578844994"
Received: from rcdn-core-5.cisco.com ([173.37.93.156]) by rcdn-iport-7.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 27 Jun 2019 17:37:54 +0000
Received: from XCH-ALN-006.cisco.com (xch-aln-006.cisco.com [173.36.7.16]) by rcdn-core-5.cisco.com (8.15.2/8.15.2) with ESMTPS id x5RHbss0007262 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Thu, 27 Jun 2019 17:37:54 GMT
Received: from xhs-rcd-003.cisco.com (173.37.227.248) by XCH-ALN-006.cisco.com (173.36.7.16) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Thu, 27 Jun 2019 12:37:53 -0500
Received: from xhs-rcd-003.cisco.com (173.37.227.248) by xhs-rcd-003.cisco.com (173.37.227.248) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Thu, 27 Jun 2019 12:37:53 -0500
Received: from NAM05-BY2-obe.outbound.protection.outlook.com (72.163.14.9) by xhs-rcd-003.cisco.com (173.37.227.248) with Microsoft SMTP Server (TLS) id 15.0.1473.3 via Frontend Transport; Thu, 27 Jun 2019 12:37:53 -0500
ARC-Seal: i=1; a=rsa-sha256; s=testarcselector01; d=microsoft.com; cv=none; b=q3GkecSQ+TalJ9AtGvxk4pnu2rTgvWIRbSF5Ke9ZnYGrHkfrEmiFKIeJ1EP0rmVpUUx8lAfN+Sg+1hPaQ64X4Qj0HSn0FXIXEve+ZaWCdmKGSCqp0H+mNBMyfPImKyWfu30pzCcEzg2cPe85+vTfe3KXwUj+XKkKzas4Ucq/qMM=
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=testarcselector01; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=NIpBPHCixjNzVajqUVtpJPQxSirKM1RbOyIGCsReECM=; b=am8nEW19Inxvgp0w7+Glezu2UTUNBDmyuCEJDDHcGQomqB+kqovZg9fWmwNh4oaWM9IX+2WwvyG95S19RfdpRT7leA25J/NTKwn7KLMtbLri3ljPiefS5xjIjqIra4wCLSYWfdvlo9JOAvLUwqjcdzw85c/YDvNucuA6wboLI+M=
ARC-Authentication-Results: i=1; test.office365.com 1;spf=none;dmarc=none;dkim=none;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=NIpBPHCixjNzVajqUVtpJPQxSirKM1RbOyIGCsReECM=; b=ysfgYxoHTNoCzTE5pDbJ3a3jthHo8nlJbfPP0gJ5JYWf1TIA+90lG06vNru0P9piZmTexNwcBuMMe3VjYZpPKuDmruJTvD4noBvMkdxmZypaw6QjpoGFRvafaXm1e3zgUuyZ5UFzq4l/poSUXwkggFVP9jGJEOLB14wJxbyYLrs=
Received: from DM5PR11MB2027.namprd11.prod.outlook.com (10.168.103.22) by DM5PR11MB1913.namprd11.prod.outlook.com (10.175.87.148) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2008.16; Thu, 27 Jun 2019 17:37:51 +0000
Received: from DM5PR11MB2027.namprd11.prod.outlook.com ([fe80::a17c:926e:d76e:b6e5]) by DM5PR11MB2027.namprd11.prod.outlook.com ([fe80::a17c:926e:d76e:b6e5%7]) with mapi id 15.20.2008.018; Thu, 27 Jun 2019 17:37:51 +0000
From: "Ketan Talaulikar (ketant)" <ketant@cisco.com>
To: Roman Danyliw <rdd@cert.org>, The IESG <iesg@ietf.org>
CC: "draft-ietf-idr-bgp-ls-segment-routing-ext@ietf.org" <draft-ietf-idr-bgp-ls-segment-routing-ext@ietf.org>, Susan Hares <shares@ndzh.com>, "aretana.ietf@gmail.com" <aretana.ietf@gmail.com>, "idr-chairs@ietf.org" <idr-chairs@ietf.org>, "idr@ietf.org" <idr@ietf.org>
Thread-Topic: Roman Danyliw's No Objection on draft-ietf-idr-bgp-ls-segment-routing-ext-15: (with COMMENT)
Thread-Index: AQHVIL6KSN36ZpQs8kOy1khTyp/E96avg09Q
Date: Thu, 27 Jun 2019 17:37:51 +0000
Message-ID: <DM5PR11MB2027D4DAAC0834EB9390EB64C1FD0@DM5PR11MB2027.namprd11.prod.outlook.com>
References: <156030307145.5955.16632008914934707005.idtracker@ietfa.amsl.com>
In-Reply-To: <156030307145.5955.16632008914934707005.idtracker@ietfa.amsl.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=ketant@cisco.com;
x-originating-ip: [2001:420:c0e0:1007::310]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 7fb69754-e756-409f-adf6-08d6fb262e0d
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600148)(711020)(4605104)(1401327)(2017052603328)(7193020); SRVR:DM5PR11MB1913;
x-ms-traffictypediagnostic: DM5PR11MB1913:
x-ms-exchange-purlcount: 3
x-microsoft-antispam-prvs: <DM5PR11MB19138DB7700619FBFD98E7CAC1FD0@DM5PR11MB1913.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:4502;
x-forefront-prvs: 008184426E
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(39860400002)(346002)(396003)(136003)(376002)(366004)(199004)(189003)(13464003)(68736007)(74316002)(6246003)(71200400001)(54906003)(55016002)(8936002)(53936002)(9686003)(71190400001)(316002)(99286004)(6436002)(66476007)(4326008)(73956011)(64756008)(66446008)(66946007)(76116006)(6306002)(5660300002)(110136005)(81166006)(305945005)(81156014)(8676002)(66556008)(2906002)(6116002)(476003)(966005)(33656002)(76176011)(186003)(86362001)(229853002)(14444005)(256004)(46003)(486006)(478600001)(52536014)(7736002)(11346002)(25786009)(14454004)(6506007)(53546011)(7696005)(102836004)(446003); DIR:OUT; SFP:1101; SCL:1; SRVR:DM5PR11MB1913; H:DM5PR11MB2027.namprd11.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1;
received-spf: None (protection.outlook.com: cisco.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam-message-info: M+c2r8nxdQQi7Z1xsUnJgC7xc73ncFOK3tv3u4vfyDF/mXFIxghnHPft1arf+YA2SxzrGj0UmdfIPdgkeLN7URizmCpCFykMRB3mExL1uPKFkCvs4kQFtvJu3g6z6FU5OtfTcceK2SZ3kr4JI6gABgG6SnVNGUHbTEPzHJFnnGAOt6QOKaAFRCuBbTsACLJHTACYH//0P/UJQmMNhJ7c/UZEJLDTYyzfxHDnWKlq4gbGxVve9pfAzfpnzeYwj7+E7cc6eyn0ExPQw/lmFz4e0Lw0NbJnaHvof08FwXCac7GnZh9kgChQ9sYnGm06PmjRWfjFr3kENCObUtsAB3NxHnZKFRQgMIjaLhe4OWqoDRY/ybtiQwAfOhLOczMqIyGFu/iz8iaYpMxvK2MxoQ1s2ODvL2YK+zpxFADUyPNtuO0=
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 7fb69754-e756-409f-adf6-08d6fb262e0d
X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Jun 2019 17:37:51.8200 (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: ketant@cisco.com
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR11MB1913
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.36.7.16, xch-aln-006.cisco.com
X-Outbound-Node: rcdn-core-5.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/idr/emHvs6LI8urwdLj93K_DvmdJyKQ>
Subject: Re: [Idr] Roman Danyliw's No Objection on draft-ietf-idr-bgp-ls-segment-routing-ext-15: (with COMMENT)
X-BeenThere: idr@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Inter-Domain Routing <idr.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/idr>, <mailto:idr-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/idr/>
List-Post: <mailto:idr@ietf.org>
List-Help: <mailto:idr-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/idr>, <mailto:idr-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 27 Jun 2019 17:38:01 -0000

Hi Roman,

Thanks for your review. I've updated the draft for all your comments and please check inline below for response.

The version 16 of the draft has been posted to address your and other comments from the IESG review:
https://tools.ietf.org/html/draft-ietf-idr-bgp-ls-segment-routing-ext-16 

Thanks,
Ketan

-----Original Message-----
From: Roman Danyliw via Datatracker <noreply@ietf.org> 
Sent: 12 June 2019 07:01
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-idr-bgp-ls-segment-routing-ext@ietf.org; Susan Hares <shares@ndzh.com>; aretana.ietf@gmail.com; idr-chairs@ietf.org; shares@ndzh.com; idr@ietf.org
Subject: Roman Danyliw's No Objection on draft-ietf-idr-bgp-ls-segment-routing-ext-15: (with COMMENT)

Roman Danyliw has entered the following ballot position for
draft-ietf-idr-bgp-ls-segment-routing-ext-15: No Objection

When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-idr-bgp-ls-segment-routing-ext/



----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

(1) Section 2.0.  Does the sentence, “This document adds additional BGP-LS Attribute TLVs in order to encode SR information” imply that this document should explicitly update RFC7752?

[KT] As clarified by Alvaro, this document only add new TLVs to BGP-LS for SR functionality. As such, it does not update RFC7752.

(2) Add more detailed citations
-- Section 2.x.  Section 2.1.x.  When citing a reference to explain a given TLV/sub-TLV, consider adding the relevant section number to improve readability.  For example:

OLD: Section 2.1.1, IS-IS, as defined by the SID/Label sub-TLV in [I-D.ietf-isis-segment-routing-extensions].

NEW: Section 2.1.1, IS-IS, as defined by the SID/Label sub-TLV in Section 3.2 of [I-D.ietf-isis-segment-routing-extensions].

-- Section 2.1.2  Per the octet of flags, please specify the specific section number of [I-D.ietf-isis-segment-routing-extensions].

-- Section 2.2.*.  Consider adding the appropriate section number in the references when described the flags field.

[KT] The section references were previously there in the tables in sections 2.4 and 2.5 but were later removed as part of the AD review. Now adding back the same sections inline.

(3) Section 2.1.4.  Per “SID/Label sub-TLV (as defined in Section 2.1.1) which encodes the first label in the range”, why is this just the first label?  Isn’t there the potential of a series of labels each in a distinct SID/Label sub-TLV?

[KT] The Range TLV can carry multiple sub-ranges and it should be read as "first label in the sub-range". I've clarified this text in the update.

(4) Section 2.2.2.  Is there a reference to explain the semantics of the weight field?

[KT] It is described in section 3.4 of RFC8402 and I've added the same reference.

(5) Section 2.3.1.  Is there a reference to explain the semantics of the algorithm field?

[KT] It is described in section 3.1.1 of RFC8402 and I've added the same reference.

(6) Editorial Nits:
-- Section 2.1.1.  Missing word:
s/The TLV and has the following format:/ /The TLV and sub-TLV has the following format:/

[KT] Fixed

-- Section 2.1.5.  Typo.  s/a unsigned/an unsigned/

[KT] Fixed.