Re: [alto] Chair review of unified-props (Part 1 of 2)

"Randriamasy, Sabine (Nokia - FR/Paris-Saclay)" <sabine.randriamasy@nokia-bell-labs.com> Tue, 26 January 2021 16:47 UTC

Return-Path: <sabine.randriamasy@nokia-bell-labs.com>
X-Original-To: alto@ietfa.amsl.com
Delivered-To: alto@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 104A23A0A63; Tue, 26 Jan 2021 08:47:19 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.247
X-Spam-Level:
X-Spam-Status: No, score=-0.247 tagged_above=-999 required=5 tests=[DKIMWL_WL_HIGH=-0.25, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=0.001, NORMAL_HTTP_TO_IP=0.001, NUMERIC_HTTP_ADDR=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=nokia.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 Dp4HrnXPcPUm; Tue, 26 Jan 2021 08:47:16 -0800 (PST)
Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-he1eur04on0707.outbound.protection.outlook.com [IPv6:2a01:111:f400:fe0d::707]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 8632F3A0A5E; Tue, 26 Jan 2021 08:47:16 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=hKGJB5YDj1aG2M5j24xImRVGJE4HQR5tUcBxrfa0XfjH5gzaE/DSRF0xlcGAgwEKjtEoJ6uE4n0hiNJ0QHX4kvjozn7zkjBuP7s2n1AJRHgQseI/LRoPjQydbsL6UNxJA9dG84GdmpncQSr7eJDNCwxPfYrrHidNRirsaN7s3RoPaJGd9UN2XkfTUwHrgmT80vJT6odDpCvHDX31TPDpi0MNXmGXRXOSY/M6NkFKdTYrWnybISc5TRxUbEh0/tgBUubhn9rldL6DGcUe7j+Rc0EL1xDyBdSGzjyF8BLpRv0mQWFpCZdDi1ZWBujB0HQ6TENvzOkMBgU8A8TYndA33Q==
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=wZE68TCwgxSoYwCvZdh3FkXrmaLHdKK2oNFX0k1CiEU=; b=TMyAommETQ/4Sd6APjmyjDu7dQoimNH08mpKTvC29iglT0GgNIGntKbbH2A9iaSpNEgVDqShzTcv/c9x0LCPnxheYMDD/ejb0dSVnMP3ssrLd3rAQNLeqz5XLp3pBeh221IKld/a9RdnKDUGRVUQ/jOQauG19hNGaC0gz0snhK1To/hcnTuIB/KC2619bZp4nJkKTrrgWcaSiHyzlghnvPD0eFE3KsHIr7BIrktzBb08nWgK80s7Us0bpdv7XCYD5duGsZENNSd07n+pkY+S3mNQZey1riqiw6+XSuo81DYQYetWLUHpVCo28eLRf/O0zHptBA+EwvZXop2pu3v3ig==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nokia-bell-labs.com; dmarc=pass action=none header.from=nokia-bell-labs.com; dkim=pass header.d=nokia-bell-labs.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nokia.onmicrosoft.com; s=selector1-nokia-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=wZE68TCwgxSoYwCvZdh3FkXrmaLHdKK2oNFX0k1CiEU=; b=X+df4Op0nJy5s1T7sKTr5R61H5M8wrxVITJQEJJA0iCEA9DIgXxHU6/iwh8obj8NcOJg+wPJAqo3/5+d20OAVFE61udwAYzooL56TZiGNE0b0S1rvIO/yeW+gTWxjEUrzL/aOY8/DSlQsQA/oU01oXyVn+qW9B2t1RPp8gq5b6c=
Received: from (2603:10a6:102:7d::13) by PR3PR07MB6826.eurprd07.prod.outlook.com (2603:10a6:102:7f::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3805.14; Tue, 26 Jan 2021 16:46:59 +0000
Received: from PR3PR07MB7018.eurprd07.prod.outlook.com ([fe80::9077:6938:8b93:1883]) by PR3PR07MB7018.eurprd07.prod.outlook.com ([fe80::9077:6938:8b93:1883%6]) with mapi id 15.20.3805.015; Tue, 26 Jan 2021 16:46:59 +0000
From: "Randriamasy, Sabine (Nokia - FR/Paris-Saclay)" <sabine.randriamasy@nokia-bell-labs.com>
To: Vijay Gurbani <vijay.gurbani@gmail.com>, "draft-ietf-alto-unified-props-new@ietf.org" <draft-ietf-alto-unified-props-new@ietf.org>
CC: IETF ALTO <alto@ietf.org>
Thread-Topic: Chair review of unified-props (Part 1 of 2)
Thread-Index: AQHW8/1O5IzFkDlpQ0egnX2yBrXt6Ko6G+ng
Date: Tue, 26 Jan 2021 16:46:59 +0000
Message-ID: <PR3PR07MB70180D38FF5896732002B26295BC0@PR3PR07MB7018.eurprd07.prod.outlook.com>
References: <CAMMTW_Jzw3xhaQKkACT4YU_GHS5AQOhLA=6DYLNOEScDZvvyog@mail.gmail.com>
In-Reply-To: <CAMMTW_Jzw3xhaQKkACT4YU_GHS5AQOhLA=6DYLNOEScDZvvyog@mail.gmail.com>
Accept-Language: fr-FR, 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=nokia-bell-labs.com;
x-originating-ip: [131.228.32.169]
x-ms-publictraffictype: Email
x-ms-office365-filtering-ht: Tenant
x-ms-office365-filtering-correlation-id: dd858447-0811-4f15-623a-08d8c219ffda
x-ms-traffictypediagnostic: PR3PR07MB6826:
x-microsoft-antispam-prvs: <PR3PR07MB68265FC473508B394481FE2C95BC0@PR3PR07MB6826.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:7219;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: uSut6Rr2NvSPJFTjaztIFl+qQJ2bqddq42ZXMpg08vqK7QS+rFS4cJ8+fvpbZmb85oAozr8lU0jvbdD9/bDZ+xtx3hm9Ebt6RkcFuWR2mbJqNNitnkv8x7d4ViQmY0moIbgGfjquV6o+3Cp0UmFfTDCUPLrxIgYm7RnQ8xXm2C+CL0Jct8cCG+G3wJyayHakVIYrqsINmoZq7e1mp7CY3FqWjTK1Peedm6/BfttBLuTy047bPtsgf2OFsa2JS14yppiTGFHHXwqXFEhpn0mpmiuAev8x8lYtF4tp3CdSgPnmMrsuzPGo9OrRCs6hlbB3WRBr+v35epVE7eWo8hzuji7bIutSLRdaPV2BcUuK87IufzuDiDuYjbLGR/xPR2MwA/95gVzaE0WIsup/oSduYRqCAjcOyBK4jMl8M+JAWiZVgxhQNDHZFIWs3abWWCdSQ5jhY8oQTD5WLZNEaD+fyqDBYJzo0kawKMy7ZyRBcRssuOTe+H49cgw6XOgSju0m5360KUemWQsTE+ALBR0p/w==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PR3PR07MB7018.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(6019001)(4636009)(136003)(346002)(396003)(366004)(39860400002)(376002)(269900001)(5660300002)(166002)(52536014)(186003)(7696005)(2906002)(66574015)(55016002)(33656002)(83380400001)(6506007)(478600001)(8676002)(53546011)(76116006)(8936002)(26005)(66476007)(110136005)(316002)(66556008)(66446008)(9686003)(86362001)(64756008)(66946007)(71200400001)(4326008)(16193025007); DIR:OUT; SFP:1102;
x-ms-exchange-antispam-messagedata: =?utf-8?B?SW5Zdk5uOE14NHpNME1PRWI0U1ZRNC8wTjJ0cGFzUEtFWURtMDR0Vk12RkFj?= =?utf-8?B?RjFSQmNXU2VYRkpjWjRHNTVRSVUrRGdCT2Z3RzZOM1V4VEZhT3FXbkJDMSs5?= =?utf-8?B?Ym1OdExvL3l4UEloMWx4WDVsTlNIRU13dUZFSzNtMHhXWldLdkQ1eS95ZElM?= =?utf-8?B?bjBNeDB2Mys0Q2swZVdzaUhsNG1lL0FVSnRZWGVKZ0Y1Q3RGTUZ3SFRVYUky?= =?utf-8?B?MUROK3kvdG15LzdyTldpZEE3eTdCVERlTGxVNGVBZkVEL2xwd0dwY25KNkMz?= =?utf-8?B?ZVoxQklRMlpyUEVkcGVSelhrNGhmblR5RVlpYVpERkdNM0xVcm51MnlNSzd5?= =?utf-8?B?SDJxMFF1NGlJWklTS3ZZOEkzT3Y1bGRjRDB1MEY1WWl4aFk1QU9uRUxUVGxK?= =?utf-8?B?ZmxWd0FDOE0wNTNXYzlTN0pTTkRjRnpoRlplTnFFV2lncE9yK2p3VDFoWC9O?= =?utf-8?B?VE9vLzUwV09SNm5oS3ZsSVhVcFFzOHdlM2dUUWRsVFVnc0JnOXI0OTFuemFu?= =?utf-8?B?MWJIbklzM1E4RnVnN3ZNZXU3YWZCR0RXMGt3YkRPeVpiYmtORFJwUFZnb3kv?= =?utf-8?B?eTZKTmdLdVl6SDdoQjA3YjlDUWZlVG9QNlhkNldvdE4zNE5EVUF4QjZYcDJ1?= =?utf-8?B?by9BZGM2ZlFCelltYlZaNzdOLzRuNGw5OTBFdzlNdmdMbXNreklndVpwbjk0?= =?utf-8?B?bG5lZk5nTnNzd3dYRGwySDdjemxqUXV1RmMzQ29lUlBRY3k3V0VsTTkvSGJN?= =?utf-8?B?bEhudzNDYVZyc1VJMEtZZHFyVkFnRlZPeEphbVU1eGs1VENpMi9IbEpVREpu?= =?utf-8?B?SkgwZWM4elJhb3lHYzM1a2p1VWZQRnJUTG83VlVFTjNqQi9OWERyRDErOXBw?= =?utf-8?B?SnFuelUraTRRQld5bkNDK2pBZ2tITWgvRXRIS2FMNWpGeWEyOFFVbFJxWlZC?= =?utf-8?B?R3BJdTkycU00bWZmckl0Ti8wM3ZTTHAyMWZwNFVRdXh6MUNIZTNxc1EvR1dU?= =?utf-8?B?NGpUN0Z1Q0RNZStPZXJ1OHp1ODNabGxuaXQvWWtGaHUwQ0FuY25sekhiV1V5?= =?utf-8?B?UXFtS2twNDZldnQvNjVzck55WWc5enpjTnFBTlZOa2FDYUNEbHA0RWZ0SDZa?= =?utf-8?B?bU5hU2ZIQWRyQy9DMWdHbUhuYkRsaUVsSE00VnQxRjJzd2dXdnB3S3E5RWRB?= =?utf-8?B?ZUl6WHJIMFJjemgzZEl6Q0w1ajAvZmoxVm1uL0wrc0Q5MVRiVVAyR25SOGdW?= =?utf-8?B?YU5pQXFhZk55RlBTcCttdC9CM1JDdzd3RXRzbUpwSUxqeWNvV2tUV0RYUmVu?= =?utf-8?B?UmMwZmZJOVhvWnMzVHVtdmF6K1hPUytPcnhYb28xdmErRVdIMm9VWW5nYW1j?= =?utf-8?B?Z2x4bVk5c2JGUEFROUs5UDNCVXZjZDZEUWNlVHZydU84SFVmRWRUYlhsenJE?= =?utf-8?Q?zUTMGEIm?=
x-ms-exchange-transport-forked: True
Content-Type: multipart/alternative; boundary="_000_PR3PR07MB70180D38FF5896732002B26295BC0PR3PR07MB7018eurp_"
MIME-Version: 1.0
X-OriginatorOrg: nokia-bell-labs.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PR3PR07MB7018.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: dd858447-0811-4f15-623a-08d8c219ffda
X-MS-Exchange-CrossTenant-originalarrivaltime: 26 Jan 2021 16:46:59.4412 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5d471751-9675-428d-917b-70f44f9630b0
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: IYlx97T0uflvfrbULKlxlZgCcmFf3feDtEW+pKQmYJ1DhJdImuGZVRdbyeps+oBTHAvPb38x9DG5bO2G9+X3LbAcLxZ+frlwC3wvJHywmrs/WZt7WuWJU5BUJHRlcrEs
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PR3PR07MB6826
Archived-At: <https://mailarchive.ietf.org/arch/msg/alto/rsPjUgm2N3b3AcXa00_qgaiK6Y8>
Subject: Re: [alto] Chair review of unified-props (Part 1 of 2)
X-BeenThere: alto@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "Application-Layer Traffic Optimization \(alto\) WG mailing list" <alto.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/alto>, <mailto:alto-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/alto/>
List-Post: <mailto:alto@ietf.org>
List-Help: <mailto:alto-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/alto>, <mailto:alto-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 26 Jan 2021 16:47:19 -0000

Hi Vijay,

Great thanks for your thorough review and guidance.
We will attend to your comments and get back with our proposed updates.
Best regards,
Sabine

From: Vijay Gurbani <vijay.gurbani@gmail.com>
Sent: Tuesday, January 26, 2021 5:06 PM
To: draft-ietf-alto-unified-props-new@ietf.org
Cc: IETF ALTO <alto@ietf.org>
Subject: Chair review of unified-props (Part 1 of 2)

All: My apologies for the late start on the chair reviews of the documents I am shepherding.  However, I have started the review.

Below is the first (of two parts) review of unified-props.  This review includes all sections from Abstract to Section 4.7.1 (inclusive).

Please let me know if you have any questions.  I will post the second part tomorrow.

Chair review from Abstract - Section 4.7.1 (inclusive).

Major:

- Abstract: "...by generalizing the concept of "endpoint properties" to generic
type of entities..." ==> Note that the antecedent ("endpoint properties") and
the consequent ("type of entities") do not match.  Perhaps better to say: "...by
generalizing the concept of "endpoint properties" as applied to endpoints
defined by IP addresses to endpoints a wider set of objects..."

More concretely:

OLD:
This document extends the base Application-Layer Traffic Optimization
(ALTO) Protocol by generalizing the concept of "endpoint properties"
to generic types of entities, and by presenting those properties as
maps, similar to the network and cost maps in the base ALTO protocol.

NEW:
This document extends the base Application-Layer Traffic Optimization
(ALTO) Protocol by generalizing the concept of "endpoint properties"
as applied to endpoints as defined by IP addresses to endpoints defined
by a wider set of objects.  Further, these properties are presented
as maps, similar to the network and cost maps in the base ALTO protocol.

- S3.2.1: "An entity domain type is expected to be registered at the IANA" ==>
you mean "MUST be registered with IANA"?  Or "SHOULD be registered with IANA"?
Best to use normative language here, unless you have a specific reason not
to.

- S3.2.2: What does this mean: "As a consequence, entities in such
domains may be defined in a resource handling this domain type but
not in other resources handling this same domain type."?  I am unable to
parse this, I think you are saying that of all the resources handling a
particular domain type, the entity must be defined in only one of them.  If
so, perhaps best to reword; something like:
   OLD:
   As a consequence, entities in such domains may be defined in a
   resource handling this domain type but not in other resources handling
   this same domain type.
   NEW:
   As a consequence, of all the resources defining a particular domain
   type, the entity must be defined in only one resource.

- S4.2.2, first paragraph: Is this example valid?  From my experience, ASN's
contain IPv4 addresses defined by a CIDR block.  I think it is highly unlikely
that a service provider will define a CIDR block (192.0.1.0/24<http://192.0.1.0/24>) and have that
block span ASNs, but perhaps I am mistaken.  Perhaps someone from network
operations may want to look at this example and bless it, or if you are sure
that networks are architected in such a manner, then we can let it stay.

- S4.6.2: "When an ANE has a persistent identifier, say, "entity-4", the
latter", here what do you mean by "latter"?  In this sentence, I do not see two
things that can be characterized as "former" and "latter"...?

- S4.7.1: This subsection appears as an afterthought; it is not "defining
resource media-types" as much as simply "listing resource media-types".  It does
not appear to be comprehensive since it only includes two examples, which leads
me to think that perhaps these examples are best put in parts of the text that
refer to these property types.  Furthermore, the media type for property "pid"
is already defined in RFC7285, so if you want to give an example here, simply
refer to RFC7285 for it.  And, the media type "alto-cdnifci+json" is defined in
draft-ietf-alto-cdni-request-routing-alto, not in this section.  My advice would
be to remove this subsection, I don't think it is comprehensive and just adds to
the confusion.

Minor:
- S3.1: in the bullet items, please be consistent when using "IPv4 or IPv6"
versus "ipv4 or ipv6".  Both forms are used, best to choose one and be
consistent in the section.  (I recognize that lower case "ipv4" and "ipv6" are
used elsewhere in the document to define entity domains, that is okay; just be
consistent in S3.1.)

- S3.1, last bullet item: What os a "routable network node"?  Is a network node
that performs the routing function (a router)?  or is it a network node that is
the recipient of a packet routed to it (an endpoint)?

- S4.4.3: s/sets in the upper level./subsets./
Rationale: "Upper level" of a set consists of elements that are subsets. Since
you are using set theory here, perhaps best to use nomenclature in that
domain.  (You may edit it as "subsets (upper levels)." if that helps.

- S4.4.3: Similarly, "lower level" is "superset".

Nits:
- S1: s/which is also not globally/that may not be globally/

- S3.2.2: s/A PID is defined relatively to a network map./A PID is defined
relative to a network map./

- S3.4: s/thus conveying values of all property values/thus conveying all
  property values/

- S4.3: s/So that if/Thus, if/

- S4.5: s/ALTO Server may just not provide a/ALTO Server may choose not to
provide a/

- S4.6: s/define them, that is, the set of addresses included in this
PID./define the set of addresses included in this PID./