Re: [ippm] draft-ippm-ioam-data-09 WGLC comments

"Frank Brockners (fbrockne)" <fbrockne@cisco.com> Thu, 28 May 2020 13:06 UTC

Return-Path: <fbrockne@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 C46BD3A0EA1; Thu, 28 May 2020 06:06:12 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -9.597
X-Spam-Level:
X-Spam-Status: No, score=-9.597 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_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=LlM1u/aA; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=nTAVmlya
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 CSk1N9DVQyhr; Thu, 28 May 2020 06:06:10 -0700 (PDT)
Received: from rcdn-iport-3.cisco.com (rcdn-iport-3.cisco.com [173.37.86.74]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 8AE483A0E9E; Thu, 28 May 2020 06:06:10 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=22036; q=dns/txt; s=iport; t=1590671170; x=1591880770; h=from:to:subject:date:message-id:references:in-reply-to: mime-version; bh=P42GX9C5nZVOPJMzKzg2CI4KuW8ZfgxNYgiw71ER7w8=; b=LlM1u/aA9wToR4pi7kSlKZFHXfnqP1NvNFqrN74FMQjDu+/3D99qHB72 tOAqFozeTb4OQEZuQc7pQrrbobdGQ+svA5Ujtn7pp5SXlcci3rQ1brbGE UXp82c1hIqvzsyMjyvH81CkavvipUyz7I3eW00qV+o+kNKYj5VjqK0QmR o=;
IronPort-PHdr: 9a23:aJPuSRZkKwMxcnITsZprUWD/LSx94ef9IxIV55w7irlHbqWk+dH4MVfC4el21QaVD4re4vNAzeHRtvOoVW8B5MOHt3YPONxJWgQegMob1wonHIaeCEL9IfKrCk5yHMlLWFJ/uX3uN09TFZXxYlTTpju56jtBUhn6PBB+c+LyHIOahs+r1ue0rpvUZQgAhDe0bb5oahusqgCEvcgNiowkIaE0mRY=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0AaCADitc9e/4cNJK1mglOBIS9SB29YLywKhBuDRgONP4l9iWSEZ4EugSQDVQsBAQEMAQEjCgIEAQGBUIJ0AheCBAIkNgcOAgMBAQsBAQUBAQECAQYEbYVXDIVyAQEBAQMSEQoTAQEpDw8CAQYCEQQBASsCAgIfER0IAgQBEggTB4MFgX5NAy4BDpM8kGcCgTmIYXaBMoMBAQEFhS4NC4IOAwaBOIJkiWAagUE/gRFDgk0+gh5JBIFlK4JnM4ItjluDEIYmiwiPJjNMCoJUlAWEfIJkiQQFkiAdkDqBXYpgkS0CBAIEBQIOAQEFgVoLJ4FWcBU7gmlQGA2UMoUUhUJ0AgE0AgYBBwEBAwl8i08BgQ8BAQ
X-IronPort-AV: E=Sophos;i="5.73,444,1583193600"; d="scan'208,217";a="758443991"
Received: from alln-core-2.cisco.com ([173.36.13.135]) by rcdn-iport-3.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 28 May 2020 13:06:08 +0000
Received: from XCH-ALN-003.cisco.com (xch-aln-003.cisco.com [173.36.7.13]) by alln-core-2.cisco.com (8.15.2/8.15.2) with ESMTPS id 04SD67Zi015051 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Thu, 28 May 2020 13:06:07 GMT
Received: from xhs-rtp-003.cisco.com (64.101.210.230) by XCH-ALN-003.cisco.com (173.36.7.13) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Thu, 28 May 2020 08:06:07 -0500
Received: from xhs-rcd-002.cisco.com (173.37.227.247) by xhs-rtp-003.cisco.com (64.101.210.230) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Thu, 28 May 2020 09:06:06 -0400
Received: from NAM04-SN1-obe.outbound.protection.outlook.com (72.163.14.9) by xhs-rcd-002.cisco.com (173.37.227.247) with Microsoft SMTP Server (TLS) id 15.0.1497.2 via Frontend Transport; Thu, 28 May 2020 08:06:06 -0500
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=akDvxCimST+YqlXV1KCA9aQAOI8qAtO5PBsz823JRLS7MH/hG1GSmEe47jnOLZ5w6ubrdH3BNAjHvkvgGh+d6EQOTNmGqL7a3cRyG1Z+VIfMR/GXQwxg76e4/0sMP4kRPwNZefdltsBY8KnbFDQiPM1Lb20KW/mt8V6N0GKsjQsjZ1N7I2ea2knjdrchOKvnB4SLhd8epmLDksoWzXVwU0MknthlFwiAzBsRl+60n9JuI+5PlLAWWJD2XDkZLzIwz8ih7gayY009YSozgGjoudD7cY7Bx67KI3N8xU1y73XOVbDfUdLNJU2/c+jfmWEjXUXk0lFG3paZjKiovenLYQ==
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=P42GX9C5nZVOPJMzKzg2CI4KuW8ZfgxNYgiw71ER7w8=; b=Xbat40YyhifPu9wYdbS6I2oMOJzQwBBfRBSDZ2xFQEL31AyN8/aN9x7M+rMolIjvFGmQ4ewHewtB8yMpFfI7WkN+/+q0J8rg5u1JTrDHyAaBXDtpGAqaeQxRhCJYqvsHhr+DpIDMfq34Pawgztm349PPrX0zx5I34ZpkNjHW8aEgMPMHU2E9/+B5qGYF5jVPOZdScXjUANM6co/8DuOOmMUvfxRlxhpPgfRtdmA4pFs/PPmmzuNLQYv7JZwCQgjf3DfBvPudTCzOgWKQWrvpDkXXJdOuwBifCnBXUtjpv4JlZJPWzvf/5aP7KIFk2kpM1B/i0V7zGq5dGvb+tt2Epw==
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=P42GX9C5nZVOPJMzKzg2CI4KuW8ZfgxNYgiw71ER7w8=; b=nTAVmlyaXMe/XN3/xIwT+SCQTqofZlAZXrvOCfcnMmo5j3SfruLYn49fz63ZJHwHGZ4D+M6L/aXw3SeQa7YR4wMH+zkXkTgwn93Ll5E50dVYdS7iyLOjlty5jzZbMOlQd6k47IUNq+ywzyd/mjEVveeDm/Ty5/qOQZZ1Bhk/nPk=
Received: from BYAPR11MB2584.namprd11.prod.outlook.com (2603:10b6:a02:c8::31) by BYAPR11MB2855.namprd11.prod.outlook.com (2603:10b6:a02:ca::26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3045.19; Thu, 28 May 2020 13:06:06 +0000
Received: from BYAPR11MB2584.namprd11.prod.outlook.com ([fe80::d8d7:dbc7:25a8:a4bd]) by BYAPR11MB2584.namprd11.prod.outlook.com ([fe80::d8d7:dbc7:25a8:a4bd%3]) with mapi id 15.20.3045.018; Thu, 28 May 2020 13:06:06 +0000
From: "Frank Brockners (fbrockne)" <fbrockne@cisco.com>
To: Martin Duke <martin.h.duke@gmail.com>, "draft-ietf-ippm-ioam-data.all@ietf.org" <draft-ietf-ippm-ioam-data.all@ietf.org>, "ippm@ietf.org" <ippm@ietf.org>
Thread-Topic: draft-ippm-ioam-data-09 WGLC comments
Thread-Index: AQHWM7TMn06Qdq+Zlk6VHV9SjuSJYKi9Fu8g
Date: Thu, 28 May 2020 13:06:05 +0000
Message-ID: <BYAPR11MB2584CD62C9004D8E4BC5D299DA8E0@BYAPR11MB2584.namprd11.prod.outlook.com>
References: <CAM4esxT2DdaeQO5DVQJftdcjAG3ZX6S3PYJXu1+5Bt3v=Ctsng@mail.gmail.com>
In-Reply-To: <CAM4esxT2DdaeQO5DVQJftdcjAG3ZX6S3PYJXu1+5Bt3v=Ctsng@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
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: [173.38.220.33]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 6c2e281c-3431-43db-b5ae-08d80307e1c1
x-ms-traffictypediagnostic: BYAPR11MB2855:
x-microsoft-antispam-prvs: <BYAPR11MB28554B35DF66B1C3E160E6CCDA8E0@BYAPR11MB2855.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:9508;
x-forefront-prvs: 0417A3FFD2
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: w/QqTsXD2RvjxQEeM8APAz5KxJ2yd/3Ed+CpPMIolhwzSYZJ/lBbclvrab5rMp0HaL++vxvrst7xV4RyjFQBX+Qzak2IjOtS5ZbP/1unbgXIcSqW7mt1K4cULBzfkXomk4NvDis77656guWVosE1IwRxBFcd+f7tdxQeyIAdJHISTYI5sJr4SageF8B6lqV4lznl6HH6pEE6bYF8PELb5yTjfYgVScvjDeuxMPUL8ZA7DHwlzCsCshml3xypnf3RKfXnuSYmTbJSjrHOkv3HmpmPVHBDQulIGek6pSDdFovjVWfd/pboeTuPBk4akFhZ/ByIN35WRJmNmDjZKTSeRd4pZWH5nMLZOsXrk2IzyJBRw+uvDSI4O0FyqKKzOkoVrgDbugI0W33ZlrRrw49zng==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:BYAPR11MB2584.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:(4636009)(39860400002)(136003)(366004)(396003)(376002)(346002)(83380400001)(5660300002)(8676002)(9326002)(71200400001)(9686003)(55016002)(166002)(66946007)(86362001)(33656002)(66446008)(76116006)(66556008)(64756008)(66476007)(2906002)(8936002)(52536014)(478600001)(6506007)(7696005)(26005)(53546011)(110136005)(316002)(186003); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: ztNcdywuf6K3rsQ09/WYSQ8O1wWk2Ecbh7Mdrj7PeL7FlMgsH1eOUJYo4qWPc8O0pAQ4C6F+cpD/U7oHF6bOINZ30WBdxQ84md2d7zL8HzNhnNOLb7hEfiIjFTrE/qwmJ0XL3GfGG2zmdhCq8CNbfFiP6zfDMKsDWB59fQKzG90AKhAIF/JAL26NusIg3mIQ/O5O6PR2DcnRFHu8lrgE43SXsQZPjz6F82mM96oZjs8BShc/PAzGQ5oF8GA70w10cRE6nC9/+ATmp6hq8u4k392pmuBQpyBnEi7BzWR/gy0H7tmq2ZsXGdfnXoQpBfh5DRIEtHx7cO86dUj/gG4ajFWt8B7zupdwxWd+j4gX/s1CNTXFesxMFEQL2PjDpuValcxU4JmxCq5oL3kp6rWnSGocFkzfRf5MVHoHZtQlmQe515u0p1XuK5Wo73xb/f5QrJJeV8SQsCDUfHjVAy7uge3XzggbOdk5OJ3uygzCUaVNGWl89sQvARJ4TalsA8dW
x-ms-exchange-transport-forked: True
Content-Type: multipart/alternative; boundary="_000_BYAPR11MB2584CD62C9004D8E4BC5D299DA8E0BYAPR11MB2584namp_"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 6c2e281c-3431-43db-b5ae-08d80307e1c1
X-MS-Exchange-CrossTenant-originalarrivaltime: 28 May 2020 13:06:05.8479 (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: bQyo5D/VyrAda3x+YbCzMnd4G/a1rpQAzyiTyzJFbQkeg9h2kidGPWeDWnnYg8ik2AgZRZZNuNMknd4FXvHhJA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR11MB2855
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.36.7.13, xch-aln-003.cisco.com
X-Outbound-Node: alln-core-2.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/ippm/LvUX8i41dsQ4TPscJxOWUCQUqSA>
Subject: Re: [ippm] draft-ippm-ioam-data-09 WGLC comments
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: Thu, 28 May 2020 13:06:13 -0000

Hi Martin,

Thanks a lot for the detailed review. Please see inline (“…FB”).

From: Martin Duke <martin.h.duke@gmail.com>
Sent: Mittwoch, 27. Mai 2020 01:24
To: draft-ietf-ippm-ioam-data.all@ietf.org; ippm@ietf.org
Subject: draft-ippm-ioam-data-09 WGLC comments

This document is almost ready. I have a few review comments and a trail of nits.

Comments:
1. It's RFC Ed policy to have no more than 5 authors, and you have 12. Please follow these guidelines<https://www.rfc-editor.org/policy.html#policy.authlist> to trim the list down.
…FB: The topic was brought up and discussed several times, though there is no easy solution, given that IOAM had indeed many cooks and at least I am not able to find appropriate reasons to prune the list of authors to 5. I’m hoping for some guidance from the WG chairs on how to handle the issue.

2. Throughout the document, there are many lower-case 'must', 'should', or 'may.' These should be reviewed, and most of them changed to upper-case. Non-upper-case should generally use synonyms like 'might' or 'could'.
…FB: We’ll review all the lower-case must/should/may statements once more and see whether there is a need to change to uppercase. My current understanding (please correct me if I’m wrong) was that per RFC 8174, the use of lower case must/may/should would be ok – we can of course see whether we can improve the wording to make sure things are not misread.

3. Section 4.2 says that transit nodes "will update at most one of these Option Types". I think you mean "at least one of each of these option types", at least if I'm reading 4.4 correctly.
…FB: It is indeed “will update at most one of these Option Types” - because for tracing it is either prealloc-update OR incremental-update OR no-operation, i.e. a node will either do nothing or update one of the Option-Types, but never both at the same time.

4. Section 4.4.1 has many normative requirements for the options. What should transit nodes do if these are not followed (e.g. remainingLen is a garbage number)? Perhaps there should be an "error flag" in addition to the Overflow flag?
…FB: IMHO we can follow typical design principles here, i.e. be loose/tolerant on what you accept, strict on what you send. We can add a sentence that if a node receives a packet with IOAM data fields that it cannot decode/understand, it would simply ignore the IOAM data fields. Having a node figure out whether the IOAM data is really corrupted or not might be hard to achieve (e.g. it could be some future version that the node just does not understand).

5. does the pre-allocated option have any sort of initial value the encapsulation node should set all the trace option data to? Would this allow the transit nodes to detect if there was a mistake in RemainingLen?
…FB: If we go with the above assumption, that nodes that do not understand the IOAM data fields would simply ignore them and leave the packet untouched, we would not benefit from any pre-set value on allocation. Depending on the implementation, pre-defined values may impose an additional burden on the encap node.

Nits:
Sec 3.
s/using for example packet/using, for example, packet
s/but do not have to share/but do not have to, share
s/data-plane/data planes

Sec 4.3 s/at average/on average

Sec 4.4
s/hardware and software implementations IOAM/hardware and software implementations, IOAM
s/data.The/data. The
s/devices which either/devices with either

Sec 4.4.1
suggest the sentence
'This bit is set by the network element if there are not enough octets left to
         record node data, no field is added and the overflow "O-bit"
         must be set to "1" in the IOAM-Trace-Option header.'
be rewritten as
'If there are not enought octets left to record node data, the network element MUST NOT add a field and MUST set the overflow "O-bit" to "0" in the IOAM-Trace-Option header.'

s/When set indicates/When set, indicates [multiple times in this section]

Sec 4.4.2
This section could use some subheaders for each node data field

s/hence recommend/and hence RECOMMEND
s/standard unit/standard units

Sec 4.5
Please add a reference for SSSS

Sec 5.3
s/run Linux/run the Linux

Sec 7.8
RFC 8126 says you are supposed to provide guidelines for the expert reviewer.

…FB: Thanks for catching all those nits! Greatly appreciated.

Cheers, Frank

-- Martin