Re: comments on draft-ali-6man-spring-srv6-oam-03

"Zafar Ali (zali)" <zali@cisco.com> Sun, 03 November 2019 12:52 UTC

Return-Path: <zali@cisco.com>
X-Original-To: ipv6@ietfa.amsl.com
Delivered-To: ipv6@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 45A181200EC; Sun, 3 Nov 2019 04:52:45 -0800 (PST)
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, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-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=mpNPBkLy; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=li5ubYaW
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 j4a4_IB5_zp5; Sun, 3 Nov 2019 04:52:41 -0800 (PST)
Received: from alln-iport-7.cisco.com (alln-iport-7.cisco.com [173.37.142.94]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 926F3120089; Sun, 3 Nov 2019 04:52:39 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=23273; q=dns/txt; s=iport; t=1572785561; x=1573995161; h=from:to:cc:subject:date:message-id:references: in-reply-to:mime-version; bh=Al9AQvjVhagUI+rCFKseIu637WD7Pfr+x+QdXwewcdI=; b=mpNPBkLyWdykxuK+Njm7+23hm+5czdWqbWE9gXustDF7CNlB4T+ClQPX /IGkHioOC7IJBrM9tzABTIGsxUuW2KGf4HbYtY30g8Df5l2NDSrOaLmh9 7lqI1gYBq+vRfaRvr8j88Qae+yZI26jeU8Pij25Yct7bHcnVVpNLxlB8y c=;
IronPort-PHdr: =?us-ascii?q?9a23=3A8O6T/B2foKukFdWtsmDT+zVfbzU7u7jyIg8e44?= =?us-ascii?q?YmjLQLaKm44pD+JxGCt+51ggrPWoPWo7JfhuzavrqoeFRI4I3J8TgZdYBUER?= =?us-ascii?q?oMiMEYhQslVdyMDUzTJ//xZCt8F8NHBxdo?=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0CwAACkzL5d/5RdJa1lGgEBAQEBAQE?= =?us-ascii?q?BAQMBAQEBEQEBAQICAQEBAYF9gRwvUAVsWCAECyqEKYNGA4p0gl5/gWaGcYl?= =?us-ascii?q?GhGGBQoEQA1QJAQEBDAEBLQIBAYRAAheDZCQ4EwIDCwEBBAEBAQIBBQRthTc?= =?us-ascii?q?MhVEBAQEBAxIRChMBATcBDwIBCBEDAQIrAgICMB0IAgQBDQUigwABgXlNAy4?= =?us-ascii?q?BAqVyAoE4iGB1gTKCfgEBBYUXGIIXCYE2jBMYgUA/gRImH4FOSTU+ghuBcli?= =?us-ascii?q?CcDKCLIx5gwWFO4k5jkNBCoIkkSeBVII7G4I8h1qPT40OgTSKP48YAgQCBAU?= =?us-ascii?q?CDgEBBYFpIoFYcBVlAYJBUBEUgwYMF4NQilN0AYEnjUUBAQ?=
X-IronPort-AV: E=Sophos;i="5.68,263,1569283200"; d="scan'208,217";a="356159907"
Received: from rcdn-core-12.cisco.com ([173.37.93.148]) by alln-iport-7.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 03 Nov 2019 12:52:38 +0000
Received: from XCH-ALN-010.cisco.com (xch-aln-010.cisco.com [173.36.7.20]) by rcdn-core-12.cisco.com (8.15.2/8.15.2) with ESMTPS id xA3CqcdZ021936 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Sun, 3 Nov 2019 12:52:38 GMT
Received: from xhs-rtp-003.cisco.com (64.101.210.230) by XCH-ALN-010.cisco.com (173.36.7.20) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Sun, 3 Nov 2019 06:52:37 -0600
Received: from xhs-aln-001.cisco.com (173.37.135.118) by xhs-rtp-003.cisco.com (64.101.210.230) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Sun, 3 Nov 2019 07:52:36 -0500
Received: from NAM04-CO1-obe.outbound.protection.outlook.com (173.37.151.57) by xhs-aln-001.cisco.com (173.37.135.118) with Microsoft SMTP Server (TLS) id 15.0.1473.3 via Frontend Transport; Sun, 3 Nov 2019 06:52:36 -0600
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=kd5rnxrEgrABAA8Uifj5V9SYoQM9IFsXJdrEx9ijX4Va8xo9BH4y3DC1uNGZ26nb8xRQ+akWYiO7AZnXNsb/c162s6pqZZw6PsSJbxkII5O/rVrTU81kEs6VITB7Ed/OQABMCwUHlprbBnAQF/1jPxLJE0NL6+CqUJS5S3P9Tlh4fqD6Vp19F8+k62chtSXYc2PBqSHJf1SkCWuy1l185TIod0bSKlzVesZh3oO5NtO4zb4J3BfHqZ5DkURhPC+kCXvzwj/ay3MAlp7gUyPL9+coEPtN4pyaHPp7oE/bOrjlhPbTmk5+30m4so4Mful0Y3cBY81rMTfIQhVsrfL9bw==
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=Al9AQvjVhagUI+rCFKseIu637WD7Pfr+x+QdXwewcdI=; b=XGTCZkzvsz0O0DPvbM4XaHu1a+AAIMmEEHRWZcDNDOBrpj5Q30xBd79jAqsnJPeIMF5symjqWK2zQkp0rQMB99V0V720PkKDTgbGmDVeqmPxN3o2XjmLT0GRX0FjAts8J+0iGrNtlx2JngkN2StOTd6xTsPxoyFG8R9YQAfHYgED+sDfdPX+FPnzOddSiCGb4qkMdDYNZUmH/qo9UkFlx4vVurELDQuR6ywI3G4uAtjD1d4mxf+PK1cQ+nQu/sMxuHMFCSGkwYhLtlvX0E0ZfwnWskCczJcw3dsZFWRAK6HWacyph1EWtZaiVaDtEgbGglEmszVds2XY7g01vChiVg==
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=Al9AQvjVhagUI+rCFKseIu637WD7Pfr+x+QdXwewcdI=; b=li5ubYaW8Q+2oSA2sm88a8fcYpooKWPHVg5T9nHx0vzuyvuYyUtgGPJd+eIWnNQ1D6HkcX/vaM9ZQ2xJgDrQ9vieGRxpT+/EoPZe6TeaRqeAiCryXI0s+5YpgXdcmP040UCc8hEuU6SRQ69gi9xBnjU9KhtmpKn1BNmoBy0hnpc=
Received: from DM6PR11MB3129.namprd11.prod.outlook.com (20.177.217.75) by DM6PR11MB2650.namprd11.prod.outlook.com (20.176.98.18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2408.24; Sun, 3 Nov 2019 12:52:29 +0000
Received: from DM6PR11MB3129.namprd11.prod.outlook.com ([fe80::f1b6:5b6f:8771:6a40]) by DM6PR11MB3129.namprd11.prod.outlook.com ([fe80::f1b6:5b6f:8771:6a40%7]) with mapi id 15.20.2408.024; Sun, 3 Nov 2019 12:52:29 +0000
From: "Zafar Ali (zali)" <zali@cisco.com>
To: li zhenqiang <li_zhenqiang@hotmail.com>, draft-ali-6man-spring-srv6-oam <draft-ali-6man-spring-srv6-oam@ietf.org>, spring <spring@ietf.org>
CC: "Zafar Ali (zali)" <zali@cisco.com>, "ipv6@ietf.org" <ipv6@ietf.org>, "6man@ietf.org" <6man@ietf.org>
Subject: Re: comments on draft-ali-6man-spring-srv6-oam-03
Thread-Topic: comments on draft-ali-6man-spring-srv6-oam-03
Thread-Index: AQHVWZ0TLsET2a8RX06Wt0hHfChkDqd5hAwA
Date: Sun, 3 Nov 2019 12:52:29 +0000
Message-ID: <545E41FF-475C-4A51-8C99-66677CD5679F@cisco.com>
References: <SG2PR06MB3159DF7B3D79BA2AA1876C74FCA40@SG2PR06MB3159.apcprd06.prod.outlook.com>
In-Reply-To: <SG2PR06MB3159DF7B3D79BA2AA1876C74FCA40@SG2PR06MB3159.apcprd06.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/10.1e.0.191013
authentication-results: spf=none (sender IP is ) smtp.mailfrom=zali@cisco.com;
x-originating-ip: [2001:420:c0cc:1004::3e]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: c8e13e64-a301-42e6-25fe-08d7605caf87
x-ms-traffictypediagnostic: DM6PR11MB2650:
x-ms-exchange-transport-forked: True
x-microsoft-antispam-prvs: <DM6PR11MB265038A99C0DB21B73227BFDDE7C0@DM6PR11MB2650.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:9508;
x-forefront-prvs: 0210479ED8
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(4636009)(376002)(366004)(396003)(39860400002)(346002)(136003)(199004)(189003)(5660300002)(81156014)(86362001)(99286004)(45080400002)(2906002)(76176011)(478600001)(66446008)(66946007)(46003)(6506007)(53546011)(91956017)(4326008)(71200400001)(71190400001)(11346002)(446003)(2616005)(486006)(14444005)(36756003)(256004)(14454004)(102836004)(186003)(25786009)(8676002)(6512007)(54896002)(6306002)(81166006)(229853002)(76116006)(54906003)(66556008)(66476007)(8936002)(476003)(64756008)(6246003)(6436002)(6116002)(790700001)(58126008)(316002)(7736002)(110136005)(9326002)(33656002)(6486002); DIR:OUT; SFP:1101; SCL:1; SRVR:DM6PR11MB2650; H:DM6PR11MB3129.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: BCL:0;
x-microsoft-antispam-message-info: sP/8NJL2V7JjxVINgld0IHsSIlzSNmtYuWvQ/aCaLQZ6kn/io5T29Cbh7t5h7tF1mAKC3r/5jeZdCb1cOY+ofBQTsGxMKJ7TmJX4MwwID+cuDWzHEf6Td5lh2bND3Vk102Ng/DusAJ0oLCdtX1wL0ZJdw9Te7N00SHiSFTzrCd+yxdfj6yaqaZNP+WhJ8q4RQ9MFQEsEbe8yZurT35IORUtNIykr3l1YXHbTd67hsge1c5ZFm5gtE8+JOD2EgGSV5BYTNZRP2MidJ6qLaeICVyOkT0GAkG8869LRKOXlsJTXFM26vI0ecvbom7zCsv0/M+LaQaxbnrJe/SdCTJSq1cD1cHOQal8ZO9+juoOk4DNTdT0695g8mOu2Bl1IlTwMStoUnQNuwp+siGVnd2OSkNtRspJW1cfI9R00Z0xl9bLfoUvPYIU9U0AtHFeRaraF
Content-Type: multipart/alternative; boundary="_000_545E41FF475C4A518C9966677CD5679Fciscocom_"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: c8e13e64-a301-42e6-25fe-08d7605caf87
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Nov 2019 12:52:29.3182 (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: +deBcHdYBApvQsl5aRInx1ue9twgZ8C7wLQFhza/V3NI8I0vw5oSCiQ0iBgkQBm8
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB2650
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.36.7.20, xch-aln-010.cisco.com
X-Outbound-Node: rcdn-core-12.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/ipv6/WCoftkPVCl3vcRKP3Ah4UQWMeXQ>
X-BeenThere: ipv6@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "IPv6 Maintenance Working Group \(6man\)" <ipv6.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ipv6>, <mailto:ipv6-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ipv6/>
List-Post: <mailto:ipv6@ietf.org>
List-Help: <mailto:ipv6-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ipv6>, <mailto:ipv6-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 03 Nov 2019 12:52:45 -0000

Hi Zhenqiang,

Many thanks for your detailed review.
Adding 6man WG.

Please see [ZA] in-line.

Thanks

Regards … Zafar

From: li zhenqiang <li_zhenqiang@hotmail.com>;
Date: Friday, August 23, 2019 at 6:25 AM
To: draft-ali-6man-spring-srv6-oam <draft-ali-6man-spring-srv6-oam@ietf.org>;, spring <spring@ietf.org>;
Subject: comments on draft-ali-6man-spring-srv6-oam-03
Resent-From: <alias-bounces@ietf.org>;
Resent-To: <zali@cisco.com>;, <cfilsfil@cisco.com>;, <satoru.matsushima@g.softbank.co.jp>;, <daniel.voyer@bell.ca>;, <mach.chen@huawei.com>;
Resent-Date: Friday, August 23, 2019 at 6:25 AM

Dear authors and all,

Zhenqiang Li from China Mobile. Please consider the following comments.

1. Section 4.1.2.1, bullet 4, When node N4 receives the packet.
I think it is better to change "The packet gets punted to the ICMPv6 process for processing" to "The packet gets time-stamped and punted to the ICMPv6 process for processing".

[ZA] Agreed. I am making the change in draft-ietf-6man-spring-srv6-oam-01.

2. Section 4.1.2.2, the last bullet.
The sentence "The ICMPv6 process at node N4 checks if its local SID (B:2:C31) is locally programmed or not and responds to the ICMPv6 Echo Request" should be "The ICMPv6 process at node N4 checks if its local SID (B:4:C52) is locally programmed or not and responds to the ICMPv6 Echo Request".

[ZA] Thank for picking the typo. I am fixing it in the latest version.

Since Flags.O=1 is used in this case, N4 should execute the local SID B:4:C52 after the execution of the pseudo-code specified in section 3.1.1. What is the proper behavior of N4 here?  Generate an ICMP parameter problem message to the Source with Error code "SR Upper-layer Header Error" according to section 4.3.1.2 of draft-ietf-6man-segment-routing-header-22?

[ZA] The O-bit is processed before the local SID. Also, you were reading an older version of the draft. In draft-ietf-6man-spring-srv6-oam-00 (Aug. 9), there is a “Ref2” added in section 3.4. The ICMP parameter problem "SR Upper-layer Header Error" is not generated.

To align with the bullets listed in section 4.1.2.1, we'd better add one new bullet here with the text "If the target SID is not locally programmed, N4 responses with the ICMPv6 message (Type:"SRv6 OAM (TBA)", Code: "SID not locally implemented (TBA)"); otherwise a success is returned".

[ZA] Addressing in the new revision.

3. Section 4.2.2.1, the last bullet.
I think it is better to change "The packet gets punted to the traceroute process for processing" to "The packet gets time-stamped and punted to the traceroute process for processing"

[ZA] Addressing in the new revision.

To align with the bullets listed in section 4.1.2.1, we'd better add one new bullet here with the text "If the target SID B:4:C52 is locally programmed, node N4 responses with the ICMPv6 message (Type: Destination unreachable, Code: Port Unreachable). If the target SID B:4:C52 is not a local SID, node N4 silently drops the traceroute probe".

[ZA] Addressing in the new revision.

4. Section 4.2.2.2, the last bullet.
The sentence "The traceroute process at node N4 checks if its local SID (B:2:C31) is locally programmed" should be "The traceroute process at node N4 checks if its local SID (B:4:C52) is locally programmed".

[ZA] Addressing in the new revision.

Since Flags.O=1 is used in this case, N4 should execute the local SID B:4:C52 after the execution of the pseudo-code specified in section 3.1.1. What is the proper behavior of N4 here?  Generate an ICMP parameter problem message to the Source with Error code "SR Upper-layer Header Error" according to section 4.3.1.2 of draft-ietf-6man-segment-routing-header-22?

[ZA] Same as similar comment above for the ICMP section.

To align with the bullets listed in section 4.1.2.1, we'd better split this bullet into two.

[ZA] Addressing in the new revision.

5. Figure 5
The SL for the response from 2001:DB8:1:2:21:: should be 1. The SL for the response from 2001:DB8:3:4::41:: should be 0.

[ZA] Good catch! Addressing in the new revision.

6. Section 6.2
Don't we need to define PSP and USP for END.OP and END.OTP?

[ZA] No. This is because END.OP and END.OTP SIDs are always followed by a target SID.

My pleasure to work together to move this draft forward.

[ZA] Many thanks and if you are traveling to Singapore, look forward to meeting you face-to-face.


Best Regards,
Zhenqiang Li
________________________________
li_zhenqiang@hotmail.com