Re: [ippm] AD review of draft-ietf-ippm-stamp

"Rakesh Gandhi (rgandhi)" <rgandhi@cisco.com> Mon, 12 August 2019 21:48 UTC

Return-Path: <rgandhi@cisco.com>
X-Original-To: ippm@ietfa.amsl.com
Delivered-To: ippm@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id E3150120849; Mon, 12 Aug 2019 14:48:10 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.499
X-Spam-Level:
X-Spam-Status: No, score=-14.499 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, 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=kCdpX4Zn; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=HoQUlcDV
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 UdENDg0qLNDj; Mon, 12 Aug 2019 14:48:06 -0700 (PDT)
Received: from rcdn-iport-6.cisco.com (rcdn-iport-6.cisco.com [173.37.86.77]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 61BC4120890; Mon, 12 Aug 2019 05:36:59 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=116783; q=dns/txt; s=iport; t=1565613419; x=1566823019; h=from:to:cc:subject:date:message-id:references: in-reply-to:mime-version; bh=loZqQAVWGNLZ/O8nd1YWfX0sKsvBNvVqDiQqsXx/C1I=; b=kCdpX4Zn06t0Qxszf8ZrJOKwuc3k6E8Q7kEfQa2VTjJcXb3GtP25zXBl qZo+o0nOA5auhuwsLVkg+EOii6P2tkLSnf2T4tvQgX4BBe6eKL738so2B 2OEWL303CsSWP58BUu1XvxX18DE+itL9J+oFGI3Ann1fO+qb2l2C7oOFA E=;
IronPort-PHdr: 9a23:ffCVEhGATqHIniWtg+CMxp1GYnJ96bzpIg4Y7IYmgLtSc6Oluo7vJ1Hb+e4z1A3SRYuO7fVChqKWqK3mVWEaqbe5+HEZON0pNVcejNkO2QkpAcqLE0r+eeTlZio2HMVqX15+9Hb9Ok9QS47z
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0CPAABxXFFd/4UNJK1mGQEBAQEBAQEBAQEBAQcBAQEBAQGBZ4EWLyQsA21VIAQLKoQeg0cDixCCNiWJXI4GgUKBEANQBAkBAQEMAQEYAQoKAgEBhD8CF4JWIzgTAQQBAQQBAQQBCm2FJwyFSgEBAQECAQEBEAgBCAQZAQEpAwsBBAsCAQgRAwECASABBgMCAgIfBgsUCQgCBA4FGweDAAGBHU0DDg8BAgygIQKBOIhgc38zgnoBAQWFCw0LghQDBoE0i2QXgUA/gREnDBOCTD6CGkcBAYE6AQ4CLQkMAQkCglMygiaMKYI3MYUMI4hejW1ACQKCHY9nUoN5G4IwL4cAhBaGLYQbjQaKJotaglICBAIEBQIOAQEFgWchgVhwFTsqAYJBgUp4DBeDT4UUhT9ygSmLZAElgiwBAQ
X-IronPort-AV: E=Sophos;i="5.64,377,1559520000"; d="scan'208,217";a="613687517"
Received: from alln-core-11.cisco.com ([173.36.13.133]) by rcdn-iport-6.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 12 Aug 2019 12:36:55 +0000
Received: from XCH-ALN-019.cisco.com (xch-aln-019.cisco.com [173.36.7.29]) by alln-core-11.cisco.com (8.15.2/8.15.2) with ESMTPS id x7CCat5G021092 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Mon, 12 Aug 2019 12:36:55 GMT
Received: from xhs-rtp-003.cisco.com (64.101.210.230) by XCH-ALN-019.cisco.com (173.36.7.29) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Mon, 12 Aug 2019 07:36:54 -0500
Received: from xhs-rtp-001.cisco.com (64.101.210.228) by xhs-rtp-003.cisco.com (64.101.210.230) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Mon, 12 Aug 2019 08:36:53 -0400
Received: from NAM02-SN1-obe.outbound.protection.outlook.com (64.101.32.56) by xhs-rtp-001.cisco.com (64.101.210.228) with Microsoft SMTP Server (TLS) id 15.0.1473.3 via Frontend Transport; Mon, 12 Aug 2019 08:36:53 -0400
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=gNCraZ+rPMEe7/CEnPEkXkfQ+VwEIkTqxxt7TXMmGYdmHAqYOrFDgQguJ4sWR7TgoUD0Mmg+ZdD4H6fbYFw8ZRXMuda9f97ltmQC2MB+/pT5wYmQ770dqQu8Ttp7k+Rczj75B4zCSZT9sv7RhE0TCvXv8S2Zt3B9KPcyjdxcdIpJ4xHSH5ciWwtLZKTDy2A+4CRfZaZ7tGpZ5UVq/aI4OqGkshXuBDWAg/93P0rGUCfoNditoJssRu+GgV1lQuwwckSkRmjXG2r+W2q+OpescWhr82MZxFSpkzzew6jTOwuKPnbu28gq1G+0IF2AHttrgo4t+rZdoKhKHmz6ilwTpg==
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=loZqQAVWGNLZ/O8nd1YWfX0sKsvBNvVqDiQqsXx/C1I=; b=bKOOvNqyAHszoQas3FAtHYxNQLjUMCWb82QXnHJlTiuHKapg+5rswM1Kr78fj7K7yYBPFDVmZDZsCJzyrH113hnzt6GaOZDi4TniBrBdT1uD1gp/IFKfOFyCBv/Nd8XnxZTZoucyQOPgko159q6Q4rlseGS29d05lavVEzzPgX+xfTQkTSiI8l3MuFp9lbwxFNje/+IOKopKE4pxpHIpav9S5LlGOAmxxpvr0Y9hM3L5M+tYYTfTXYkPMR4KLL9uNaH62zsuV+DxMRQjhFUBNn4zPtuTBaloiam+RrUm60Frhm1x3MAqbJn3Vhxea2TsT3tNu92FIzqrv/H98kD/5Q==
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=loZqQAVWGNLZ/O8nd1YWfX0sKsvBNvVqDiQqsXx/C1I=; b=HoQUlcDVuSCkUjkagW2SRC/16eZTs8Vytg1dB9odWXODOv79d/GKHdP708RdcABagjXsjI8ylzB6aER6c9B66krpRHH1kXacMetJ0o1Zxj3KU+spTXnltq9jy3fjOnwnrgBN36elno+k6KmYU5u9F96qkh/FCGSz/2xw2C0GNek=
Received: from SN6PR11MB3278.namprd11.prod.outlook.com (52.135.109.11) by SN6PR11MB2669.namprd11.prod.outlook.com (52.135.91.157) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2157.18; Mon, 12 Aug 2019 12:36:52 +0000
Received: from SN6PR11MB3278.namprd11.prod.outlook.com ([fe80::d97f:e2dd:1ea6:303f]) by SN6PR11MB3278.namprd11.prod.outlook.com ([fe80::d97f:e2dd:1ea6:303f%5]) with mapi id 15.20.2157.020; Mon, 12 Aug 2019 12:36:52 +0000
From: "Rakesh Gandhi (rgandhi)" <rgandhi@cisco.com>
To: Greg Mirsky <gregimirsky@gmail.com>
CC: Rakesh Gandhi <rgandhi.ietf@gmail.com>, IPPM Chairs <ippm-chairs@ietf.org>, Mirja Kuehlewind <ietf@kuehlewind.net>, IETF IPPM WG <ippm@ietf.org>, "draft-ietf-ippm-stamp@ietf.org" <draft-ietf-ippm-stamp@ietf.org>
Thread-Topic: [ippm] AD review of draft-ietf-ippm-stamp
Thread-Index: AQHVNZ0+7z5WSZ7weEqWcphMhijoKabA4NeAgAAWDoCAAD6nAIAAJQyAgAAPUwCAACEDgIAs2sYAgAATHACAACGlgIAB7dkAgACiroCAAKl2gIAA0naAgACI2wCAAAt0AIAB63CAgAJASgA=
Date: Mon, 12 Aug 2019 12:36:51 +0000
Message-ID: <C454EF13-3ACD-4118-B2E6-8126F526D845@cisco.com>
References: <B617B303-6EBE-4E3B-AE5C-1438FF1C5D7F@kuehlewind.net> <CA+RyBmVEmKQu=LGp9eVT+x5e01LCSk_A4tQD=RE8Ett-R35BVg@mail.gmail.com> <11938018-8A65-483B-8176-A6E1C2A265A3@kuehlewind.net> <CA+RyBmX=Jx2yXrMXu4Y2VKX36iKphymb1Hkyfy0XhPGFmsUGzQ@mail.gmail.com> <B8047CA0-2F5E-48F8-9BE4-3FA41D742F12@kuehlewind.net> <CA+RyBmXPCe7TZQqPgsKsVnifZDG8O8wGafDn-nzYfGpx2OiaXQ@mail.gmail.com> <F167C330-76F4-48FC-B720-415CA190239C@broadcom.com> <CA+RyBmVtfXcwqu1RH-1JXnhpCZcbGgm30ubKGctUPnLNJCgVZQ@mail.gmail.com> <CAMZsk6e-bcFNz327p_u6KEHV2qnJUytPwPmJVgXxEWbzsQr9OA@mail.gmail.com> <CA+RyBmW01TgyXPAk3OGhdKqDTszkf0KzT+dDVTdaEhFu7GA7-Q@mail.gmail.com> <CAMZsk6eUOTxjWy=r62SNvSLzOe8KGQ8CGgbW-H2uoLgDPmPsTA@mail.gmail.com> <CA+RyBmUfB-d18A5OJ2rG9naFE+0HjXehf13Nt4D2z2do-wHBDw@mail.gmail.com> <CAMZsk6eRG0OCY_6ZRacm9+cL=YsdjUQRXXcxA8mTA=PYs5CTVw@mail.gmail.com> <CA+RyBmVEVK10=3ULnRgyOzHKb3AWaHmisKoaHqocAYXM4w_ADg@mail.gmail.com> <E549477E-0320-41AD-8741-1898F37F6AA3@cisco.com> <CA+RyBmXNWnY=GVxz2kGFT+KheQxfexTgj8_iQqA0LZzcqM_fOQ@mail.gmail.com> <13DEB6E4-DF8C-491F-94B6-1D8CD46B3618@cisco.com> <CA+RyBmUEKDrtupSnSQvMmpM6ioGBbzo-70XZdhan=si4WHzQKA@mail.gmail.com> <6A5DC26F-A582-4C02-86A4-A1F20834B27B@cisco.com> <CA+RyBmWphGJcwRkyNqs87u3yu+1Qi=0GeoT10Aqd9+Qp2wDx7A@mail.gmail.com>
In-Reply-To: <CA+RyBmWphGJcwRkyNqs87u3yu+1Qi=0GeoT10Aqd9+Qp2wDx7A@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/10.10.c.190715
authentication-results: spf=none (sender IP is ) smtp.mailfrom=rgandhi@cisco.com;
x-originating-ip: [2001:420:c0c4:1002::12]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 723a52f0-7dcd-4823-898b-08d71f21c08d
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(5600148)(711020)(4605104)(1401327)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(2017052603328)(7193020); SRVR:SN6PR11MB2669;
x-ms-traffictypediagnostic: SN6PR11MB2669:
x-ms-exchange-purlcount: 3
x-microsoft-antispam-prvs: <SN6PR11MB26692454AF76636CEEFE10A7BFD30@SN6PR11MB2669.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-forefront-prvs: 012792EC17
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(4636009)(396003)(376002)(39860400002)(136003)(346002)(366004)(189003)(199004)(54094003)(51444003)(53754006)(53946003)(6246003)(8936002)(36756003)(53936002)(6436002)(81156014)(81166006)(1411001)(606006)(86362001)(236005)(8676002)(71190400001)(30864003)(4326008)(6916009)(229853002)(91956017)(66946007)(6486002)(6506007)(64756008)(66556008)(7736002)(66446008)(66476007)(76116006)(316002)(102836004)(33656002)(53546011)(58126008)(186003)(76176011)(2616005)(11346002)(446003)(486006)(476003)(46003)(9326002)(5070765005)(14454004)(54906003)(71200400001)(5024004)(6306002)(54896002)(5660300002)(966005)(256004)(25786009)(99286004)(14444005)(478600001)(6116002)(2906002)(6512007)(569006); DIR:OUT; SFP:1101; SCL:1; SRVR:SN6PR11MB2669; H:SN6PR11MB3278.namprd11.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A: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: rsC9MsMORiP31m8/EUI6+xxIcOe/NVO0wlpasEamWmPlZpjASTRzvBSF1r/44vWyp3kUxyTVMHhYWXbriGmoJzlXG22/mWFufG1kKt+BmOHZpNF0dgw74PEGz6vPpt7Bz5ETiZXD9vUAfJVkYmHYMGzeV2b5MHFyj3d7pV+MnUnbRWcQ93fw7Ny/wHw3hleyyE6g50/WvBCkD/GSpdg7dSoEYJYujGJypd5KVzT3W8XA+mVjSFdenaKRphVIiN071Ws29RGoNJj0Iyznkn8bezSbvZ9eArPiuvDm1o59bg3p8MuRUDNB+RWP1ng0NVdtoKeN8+NdLfpmh2PmLB5yIhoAHwhVr68JZSyzJZAp8rsiS0LFUPsgWEfOoqAzMB035d3asu5/2gb6N/4fCoDmCnR6H/p6WfEyF18zt3ZX3/o=
x-ms-exchange-transport-forked: True
Content-Type: multipart/alternative; boundary="_000_C454EF133ACD4118B2E68126F526D845ciscocom_"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 723a52f0-7dcd-4823-898b-08d71f21c08d
X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Aug 2019 12:36:51.9758 (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: Qx70TvL6wZoL0Y/lwEdkDYDA2+mlEtVEKPFdJy23Xc96a6HWQAjf7xv+1tT4bJBbg9e43PCueIcGTAPhqRgr3g==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN6PR11MB2669
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.36.7.29, xch-aln-019.cisco.com
X-Outbound-Node: alln-core-11.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/ippm/2AN1Zr2u8r6KTAMGwyU2Azk4huw>
Subject: Re: [ippm] AD review of draft-ietf-ippm-stamp
X-BeenThere: ippm@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: IETF IP Performance Metrics Working Group <ippm.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ippm>, <mailto:ippm-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ippm/>
List-Post: <mailto:ippm@ietf.org>
List-Help: <mailto:ippm-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ippm>, <mailto:ippm-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 12 Aug 2019 21:48:11 -0000

Hi Greg,
The updated version addresses my comments. Many thanks for considering them and the discussions.

Thanks,
Rakesh


From: Greg Mirsky <gregimirsky@gmail.com>
Date: Saturday, August 10, 2019 at 6:21 PM
To: "=SMTP:rgandhi@cisco. com" <rgandhi@cisco.com>
Cc: Rakesh Gandhi <rgandhi.ietf@gmail.com>, IPPM Chairs <ippm-chairs@ietf.org>, Mirja Kuehlewind <ietf@kuehlewind.net>, IETF IPPM WG <ippm@ietf.org>, "draft-ietf-ippm-stamp@ietf.org" <draft-ietf-ippm-stamp@ietf.org>
Subject: Re: [ippm] AD review of draft-ietf-ippm-stamp

Hi Rakesh,
please review the updated diff and the working version of the draft. Do you think that the changes address your and Henrik's comments on the use of UDP port numbers in STAMP? You'll notice that the use of the System ports is not mentioned. I believe that this range of port numbers should not be used. What do you think?
Much appreciate your comments, suggestions.

Regards,
Greg

On Fri, Aug 9, 2019 at 1:55 PM Rakesh Gandhi (rgandhi) <rgandhi@cisco.com<mailto:rgandhi@cisco.com>> wrote:
Thanks Greg for the updates. Changes look good to me.
One outstanding issue with the Port range being discussed in another thread.
Thanks,
Rakesh


From: Greg Mirsky <gregimirsky@gmail.com<mailto:gregimirsky@gmail.com>>
Date: Friday, August 9, 2019 at 12:26 PM
To: "=SMTP:rgandhi@cisco. com" <rgandhi@cisco.com<mailto:rgandhi@cisco.com>>
Cc: Rakesh Gandhi <rgandhi.ietf@gmail.com<mailto:rgandhi.ietf@gmail.com>>, IPPM Chairs <ippm-chairs@ietf.org<mailto:ippm-chairs@ietf.org>>, Mirja Kuehlewind <ietf@kuehlewind.net<mailto:ietf@kuehlewind.net>>, IETF IPPM WG <ippm@ietf.org<mailto:ippm@ietf.org>>, "draft-ietf-ippm-stamp@ietf.org<mailto:draft-ietf-ippm-stamp@ietf.org>" <draft-ietf-ippm-stamp@ietf.org<mailto:draft-ietf-ippm-stamp@ietf.org>>
Subject: Re: [ippm] AD review of draft-ietf-ippm-stamp

Hi Rakesh, Henrik, et al.,
I've updated the working version of the draft. Attached, please find the diff and the current copy of the document. Please let me know if I've captured all the changes we've discussed.
On the question Rakesh has asked. These recommendations are part of Section 4.4 that details aspects of STAMP interoperability with TWAMP Light implementations. All the normative language used in that section is not applicable to the scenario when both systems support STAMP protocol.

Regards,
Greg

On Fri, Aug 9, 2019 at 5:05 AM Rakesh Gandhi (rgandhi) <rgandhi@cisco.com<mailto:rgandhi@cisco.com>> wrote:
Hi Greg,
Thanks for considering my comments. Please see replies inline with <RG>..

From: Greg Mirsky <gregimirsky@gmail.com<mailto:gregimirsky@gmail.com>>
Date: Thursday, August 8, 2019 at 3:40 PM
To: "=SMTP:rgandhi@cisco. com" <rgandhi@cisco.com<mailto:rgandhi@cisco.com>>
Cc: Rakesh Gandhi <rgandhi.ietf@gmail.com<mailto:rgandhi.ietf@gmail.com>>, IPPM Chairs <ippm-chairs@ietf.org<mailto:ippm-chairs@ietf.org>>, Mirja Kuehlewind <ietf@kuehlewind.net<mailto:ietf@kuehlewind.net>>, IETF IPPM WG <ippm@ietf.org<mailto:ippm@ietf.org>>, "draft-ietf-ippm-stamp@ietf.org<mailto:draft-ietf-ippm-stamp@ietf.org>" <draft-ietf-ippm-stamp@ietf.org<mailto:draft-ietf-ippm-stamp@ietf.org>>
Subject: Re: [ippm] AD review of draft-ietf-ippm-stamp

Hi Rakesh,
many thanks for your comments.

  *   I've updated MBZ to "MAY be zeroed on transmit and MUST be ignored on receipt".
<RG> Thanks.

  *   I think that the fact RFC 7750 is not mentioned in this document should be interpreted as "not supported". If you believe that something should be said explicitly, would the following be acceptable
“[RFC7750] is supported by optional extension specified in [I-D.ietf-ippm-stamp-option-tlv].”
<RG> Yes, thanks.
<RG> BTW, I see following two texts for the timestamp format in Section 4.4. Is there a reason why the Reflector only supports NTP and it is MUST whereas Sender has the flexibility with NTP and PTP with SHOULD?
"The Session-Sender SHOULD use the default format for its timestamps - NTP. And it MAY use PTPv2 timestamp format.
<snip>
“The Session-Reflector MUST be set to use the default format for its timestamps, NTP.”
Thanks,
Rakesh

Attached are, as usual, diff and the updated working version.

Much appreciate your help and commitment to making STAMP useful and practical.

Regards,
Greg

On Thu, Aug 8, 2019 at 6:24 AM Rakesh Gandhi (rgandhi) <rgandhi@cisco.com<mailto:rgandhi@cisco.com>> wrote:
Thank you Greg for the updates. They look good, I have couple of comments:

  1.  I did not see the updates for the first bullet (1) below regarding MBZ in the updated draft. Assuming it is pending.
  2.  It should still say something for the RFC 7750. Without any guidance, it can be implemented as specified in RFC 7750.

Thanks,
Rakesh


From: ippm <ippm-bounces@ietf.org<mailto:ippm-bounces@ietf.org>> on behalf of Greg Mirsky <gregimirsky@gmail.com<mailto:gregimirsky@gmail.com>>
Date: Wednesday, August 7, 2019 at 7:43 PM
To: Rakesh Gandhi <rgandhi.ietf@gmail.com<mailto:rgandhi.ietf@gmail.com>>
Cc: IPPM Chairs <ippm-chairs@ietf.org<mailto:ippm-chairs@ietf.org>>, Mirja Kuehlewind <ietf@kuehlewind.net<mailto:ietf@kuehlewind.net>>, IETF IPPM WG <ippm@ietf.org<mailto:ippm@ietf.org>>, "draft-ietf-ippm-stamp@ietf.org<mailto:draft-ietf-ippm-stamp@ietf.org>" <draft-ietf-ippm-stamp@ietf.org<mailto:draft-ietf-ippm-stamp@ietf.org>>
Subject: Re: [ippm] AD review of draft-ietf-ippm-stamp

Hi Rakesh,
thank you for your kind consideration of my responses and very pointed questions. Please find my follow-up notes in-line below under GIM>> tag.

Regards,
Greg

On Tue, Aug 6, 2019 at 11:15 AM Rakesh Gandhi <rgandhi.ietf@gmail.com<mailto:rgandhi.ietf@gmail.com>> wrote:
Hi Greg,
Thanks for your reply. Please see inline <RG>..

On Tue, Aug 6, 2019 at 12:14 PM Greg Mirsky <gregimirsky@gmail.com<mailto:gregimirsky@gmail.com>> wrote:
Hi Rakesh,
thank you for pointing to these two RFCs. Please consider my thoughts:

  *   (1) RFC 7820 is Experimental and, as I understand, the proposed solution is not seen kindly by the security experts, and for a good reason. As you've correctly pointed out, STAMP in unauthenticated mode may easily support the technique described in RFC 7820. But I'm not sure we have to do that in the base specification. What we can do is to relax language on MBZ and drop "MUST be zeroed on transmission" leaving "MUST be ignored on receipt". What do you think?
<RG> Ok with that.


  *   (2) I appreciate your interest in RFC 7750 (as one of co-authors). We've decided to support this functionality in an extension to STAMP. Class of Service TLV fully supports the functionality defined in RFC 7750 and offers the ability to instruct the Session-Reflector which DSCP value it must use for the reflected STAMP packet. Thus CoS marking consistency is verified in forward and reverse directions.
<RG> In that case, draft may say RFC7750 method is not supported by STAMP?
GIM>> Though it is not part of the base specification, the ability to test the consistency of CoS mapping on a path between STAMP Session-Sender and Session-Reflector is supported by using the Class of Service TLV. And since it has been recently adopted by IPPM WG, I don't think that such a statement will be helpful to an implementor of STAMP.
Also, I think the draft dropped supporting the server octet [RFC6038], right? If so, following text needs updating?
   o  (3) Packet Padding (reflected) is an optional variable length field..
      The length of the Packet Padding (reflected) field MUST be equal
      to the value of the Server Octets field (Figure 2).  If the value
      is non-zero, the Session-Reflector MUST copy number of octets
      equal to the value of Server Octets field starting with the Server
      Octets field.
GIM>> I'm sorry you've had an older working version of the draft. Attached is the current version and the text has been removed. Could you let me know if this change is acceptable?

Thanks,
Rakesh


Best regards,
Greg

On Tue, Aug 6, 2019 at 8:06 AM Rakesh Gandhi <rgandhi.ietf@gmail.com<mailto:rgandhi.ietf@gmail.com>> wrote:
Hi Greg,
Couple of additional comments on the draft:
There are TWAMP extensions for Checksum complement in RFC 7820 and DSCP-ECN in RFC 7750. Good to add some text for STAMP if they can be supported or not supported. I can see they can be supported as following, and should not break anything:

0                   1                   2                   3

0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

|                        Sequence Number                        |

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

|                        Transmit Timestamp                     |

|                                                               |

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

|         Error Estimate        |           MBZ                 |

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

|                      Receive Timestamp                        |

|                                                               |

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

|                      Sender Sequence Number                   |

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

|                      Sender Timestamp                         |

|                                                               |

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

|      Sender Error Estimate    |           MBZ                 |

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

|  Sender TTL   | S-DSCP-ECN    | Checksum Complement           |

+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

Thanks,
Rakesh

On Mon, Jul 8, 2019 at 10:07 PM Greg Mirsky <gregimirsky@gmail.com<mailto:gregimirsky@gmail.com>> wrote:
Hi Rakesh,
thank you for your question. In my experience, some implementations of TWAMP-Light have taken the liberty to allow using UDP port numbers outside the Dynamic/Private range. I believe that is not the right decision. In the note of IANA's Service Name and Transport Protocol Port Number Registry we read:

 Service names and port numbers are used to distinguish between different
 services that run over transport protocols such as TCP, UDP, DCCP, and
 SCTP.

 Service names are assigned on a first-come, first-served process, as
 documented in [RFC6335].

 Port numbers are assigned in various ways, based on three ranges: System
 Ports (0-1023), User Ports (1024-49151), and the Dynamic and/or Private
 Ports (49152-65535); the difference uses of these ranges is described in
 [RFC6335]. According to Section 8.1.2 of [RFC6335], System Ports are
 assigned by the "IETF Review" or "IESG Approval" procedures described in
 [RFC8126]. User Ports are assigned by IANA using the "IETF Review" process,
 the "IESG Approval" process, or the "Expert Review" process, as per
 [RFC6335]. Dynamic Ports are not assigned.

 The registration procedures for service names and port numbers are
 described in [RFC6335].

 Assigned ports both System and User ports SHOULD NOT be used without
 or prior to IANA registration.

My interpretation is that ports in System and User ranges, even if not yet assigned, must not be used without following the assignment process. Thus, regardless of whether a number had not yet been assigned to a service, it must not be used as the destination UDP port number. Also, consider operational issues if a new service is assigned a new port number from the User Ports range. One day the number was "free" and tomorrow it may be assigned. Handling such a scenario will add complexity while benefits are, in my opinion, questionable.

Regards,
Greg

On Mon, Jul 8, 2019 at 5:09 PM Rakesh Gandhi <rgandhi.ietf@gmail.com<mailto:rgandhi.ietf@gmail.com>> wrote:
Hi Greg,

Why limit the UDP port range to 49152-65535? Any free UDP port can be used, no?

Thanks,
Rakesh


On Mon, Jul 8, 2019 at 7:20 PM Greg Mirsky <gregimirsky@gmail.com<mailto:gregimirsky@gmail.com>> wrote:
Hi Shahram,
thank you for the review and questions. Please find my answers below tagged GIM>>.

Regards,
Greg

On Mon, Jul 8, 2019 at 2:02 PM Shahram Davari <shahram.davari@broadcom.com<mailto:shahram.davari@broadcom.com>> wrote:
HI Greg

I read your draft and have the following questions:

1) Does it require any UDP/TCP port number or it reuses the one from TWAMP? if it reuses from TWAMP then  how does the receiver differentiate between TWAMP and STAMP?
GIM>> STAMP uses the well-known UDP port number allocated for the OWAMP-Test/TWAMP-Test Receiver port (RFC 8545) as the default destination UDP port number.. STAMP may use destination UDP port number from the Dynamic and/or Private Ports range 49152-65535.
2) What is the benefit of STAMO compared to TWAMP?
GIM>> The work was driven by several observations, among them:

  *   challenges in achieving interoperability among implementations of TWAMP-Light;
  *   industry interest in standardizing performance monitoring in IP broadband access networks (TR-390);
  *   improve extensibility of IP performance monitoring tool to support measurements, testing of new metrics and parameters, e.g., consistency of CoS in the network.
3) Why is there so much MBZ byte?
GIM>> It was agreed to make the symmetrical size of STAMP test packets the default. RFC 6038 defined it for TWAMP and TR-390 requires it to be supported by TWAMP-Light implementations.

Thx
Shahram

On Jul 8, 2019, at 10:17 AM, Greg Mirsky <gregimirsky@gmail.com<mailto:gregimirsky@gmail.com>> wrote:

Hi Mirja,
thank you for the suggested text. The new paragraph now reads as:
      Load of STAMP test packets offered to a network MUST be carefully
      estimated, and the possible impact on the existing services MUST
      be thoroughly analyzed before launching the test session.
      [RFC8085] section 3.1.5 provides guidance on handling network load
      for UDP-based protocol.  While the characteristic of test traffic
      depends on the test objective, it is highly recommended to stay in
      the limits as provided in [RFC8085].

If it is acceptable, I'd like to upload the updated version of draft-ieff-ippm-stamp before the cut-off deadline.

Regards,
Greg

On Mon, Jul 8, 2019 at 8:58 AM Mirja Kuehlewind <ietf@kuehlewind.net<mailto:ietf@kuehlewind.net>> wrote:
Hi Greg,

See below.

> On 8. Jul 2019, at 16:54, Greg Mirsky <gregimirsky@gmail.com<mailto:gregimirsky@gmail.com>> wrote:
>
> Hi Mirja,
> thank you for the reference to RFC 8085. I agree that the document is very much relevant and a reference to RFC 8085 in STAMP is useful. While reading Section 3.1.3 I came to think that the discussion and guidance in other sections of RFC 8085, particularly, Section 3.1.5 Implications of RTT and Loss Measurements on Congestion Control. Would adding the reference to that section in the new text proposed for the Security Considerations section work? I'll put RFC 8085 as Informational reference as it is BCP.
> NEW TEXT:
>       Load of STAMP test packets offered to a network MUST be carefully
>       estimated, and the possible impact on the existing services MUST
>       be thoroughly analyzed using [RFC8085] and its Section 3.1.5 in
>       particular before launching the test session....


Not sure if “using” is the right word but otherwise fine for me. Or you could have a separate sentence like:

“RFC8085 section 3.1.5 provides guidance on handling network load for UDP-based protocol. While the characteristic of test traffic depends on the test objective, it is highly recommended to say in the limits as provided in RFC8085.”

Or something similar…

BCP is the same maturity level as PS. So it wouldn’t be a downref. However, I think having this as informational ref is fine.

Mirja



>
> Regards,
> Greg
>
> On Mon, Jul 8, 2019 at 2:37 AM Mirja Kuehlewind <ietf@kuehlewind.net<mailto:ietf@kuehlewind.net>> wrote:
> Hi Greg,
>
> Thanks a lot for you reply. Changes are good. I wonder if it would be useful to provide a reference to RFC8085 because it has a lot of information about congestion control of UDP based traffic? It recommends to send not more than 1 packet per 3 seconds (if RTT is unknown). I guess it doesn’t make sense to require this for testing traffic, however, it could maybe still be a good recommendation? What do you think?
>
> Also I’ve just resend my review to the IPPM list, as I unfortunately cc’ed only the IPPM chairs instead of the whole list. Can you resend you proposed changes to the list, so other people are aware of these changes. Sorry for the unconvience.
>
> Mirja
>
>
> > On 6. Jul 2019, at 17:46, Greg Mirsky <gregimirsky@gmail.com<mailto:gregimirsky@gmail.com>> wrote:
> >
> > Hi Mirja,
> > thank you for your thorough review, very pointed and helpful comments. Please find my responses in-lined and tagged GIM>>. Attached the diff.
> >
> > Regards,
> > Greg
> >
> > On Thu, Jul 4, 2019 at 9:10 AM Mirja Kuehlewind <ietf@kuehlewind.net<mailto:ietf@kuehlewind.net>> wrote:
> > Hi authors, hi all,
> >
> > Thanks for this well-written document and very good shepherd write-up! I would like discuss one point before I start IETF last call.
> >
> > I believe this document should say something about network load and congestion (control). OWAMP and TWAMP discuss quite a bit sender scheduling, however, as this is a simplified version, so I think it could at least be good to put a waring in this document that packet sending should be somehow rate limited. I know it might be hard to provide more concrete guidance but at least having some discussion or warning in this document could be good.
> > GIM>>  Thank you for your suggestion. Security Considerations section points to the fact that STAMP does not include control and management components:
> >    Because of the control
> >    and management of a STAMP test being outside the scope of this
> >    specification only the more general requirement is set:
> > adding the new text here:
> >       Load of STAMP test packets offered to a network MUST be carefully
> >       estimated, and the possible impact on the existing services MUST
> >       be thoroughly analyzed before launching the test session.
> >
> >
> > Another comment: You only say at the very end that a certain UDP port is used, which implies that STAMP runs over UDP. However, I think you should mention at the very beginning that this is a UDP-based protocol. Just to make things crystal clear.
> > GIM>> Adding the reference to "UDP transport" into the first sentence of Theory of  Operations section:
> >    STAMP Session-Sender transmits test packets over UDP transport toward STAMP Session-Reflector.
> >
> > Mirja
> >
> > P.S.:
> > Nit: s/This document defines active performance measurement test protocol/ This document defines an active performance measurement test protocol/
> > -> “an” missing
> > GIM>> Thank you. Done.
> > <Diff_ draft-ietf-ippm-stamp-06.txt - draft-ietf-ippm-stamp-07....txt.html>
>
_______________________________________________
ippm mailing list
ippm@ietf.org<mailto:ippm@ietf.org>
https://www.ietf.org/mailman/listinfo/ippm

_______________________________________________
ippm mailing list
ippm@ietf.org<mailto:ippm@ietf.org>
https://www.ietf.org/mailman/listinfo/ippm