Re: [alto] Benjamin Kaduk's Discuss on draft-ietf-alto-unified-props-new-21: (with DISCUSS and COMMENT) - DISCUSS + COMMENTS on section
"Randriamasy, Sabine (Nokia - FR/Paris-Saclay)" <sabine.randriamasy@nokia-bell-labs.com> Tue, 01 February 2022 18:58 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 7B8553A1208; Tue, 1 Feb 2022 10:58:23 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.477
X-Spam-Level:
X-Spam-Status: No, score=-2.477 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.576, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_BLOCKED=0.001, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-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 fDaHxACKQ_mH; Tue, 1 Feb 2022 10:58:18 -0800 (PST)
Received: from EUR03-AM5-obe.outbound.protection.outlook.com (mail-eopbgr30117.outbound.protection.outlook.com [40.107.3.117]) (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 1072B3A11DC; Tue, 1 Feb 2022 10:58:17 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=JvDxfaslLxdCFg/Cr7/S8QuAseZmqa6wZm9tJM96Kcm9LUKGf4kvnZ0OP+04gUB+MyGNyeP+FPMKfGIJLPZ/NdBahENPV6ic32nHTd2jX+yx2zgHv8aDSefJhWFbomOvEC4uiGqmY81giL4N/x/9XY+r7Jl1FwCGnAmGS5PFXokD0/6ILhkQ9KHorxnRUdhTbYvTcjxQVYrjOwFgE34MTvHQI9nE+5ojA/KhZZu/7n4lG7LxRqtQQx9VLkDX9Z/lAbCHVhYIzO/nOVxpj3y4GaYuljBTe3SWRhA4Kt2hKuxTZZ70rqNSwJ6Dr5hYrPpg4St6ojh+O92l+5RgH2b7zg==
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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=sMnuXr4Y/PZvoKrR0oh4W1KM9n+t2F+Eg97u95fG3b8=; b=TdYvEsrV9VhAag2bLoXFvlbITM76QDjGtvXBD8/DLf4VvD+pVOhFk7iAIwi61YyrO135VfVW4YQajXHWfZQboPZT50rjmy6GrcVHdF1chytFgO3bAi4+NIXFvdPaQ100N0ESgfMbdFeKClZuDOqpr234FO7S+HM02NFhcDPWm5fQl3O3v4Sg8cOo+HDud2TA9iCassrMWhIrzRQbiAiECukhdohJJsHzOqmP07oZapcDX/DX/U8HTofE58wxVlfKpb5zpfEA42LtFyz/XiaYlQxmn/nlEzB6WhRBATZZ+isw9z2uJDhFbu94UsAHyox8hEjY/Qa1Uzw5ZVQ8mqJQ7w==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; 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=sMnuXr4Y/PZvoKrR0oh4W1KM9n+t2F+Eg97u95fG3b8=; b=oTTGYglY0OYap+W82BIXFw1fhNCoFn+Lf4Epm2SQNnRzjc//SwbQGC46Nr0eIPZEAB8EsBjB+TAoSJOp+U30gNUvAsNjeYD/8xOAKlgSlfDrWFco39oFd3XulsfTDfvNeqZOjRzBWyHG0YKsNhtHHJ8kHUYXynTJbsVrFhblC+8=
Received: from PAXPR07MB7806.eurprd07.prod.outlook.com (2603:10a6:102:13a::19) by AM5PR0701MB2450.eurprd07.prod.outlook.com (2603:10a6:203:10::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4951.12; Tue, 1 Feb 2022 18:58:05 +0000
Received: from PAXPR07MB7806.eurprd07.prod.outlook.com ([fe80::f445:333b:9039:5d0]) by PAXPR07MB7806.eurprd07.prod.outlook.com ([fe80::f445:333b:9039:5d0%4]) with mapi id 15.20.4951.010; Tue, 1 Feb 2022 18:58:05 +0000
From: "Randriamasy, Sabine (Nokia - FR/Paris-Saclay)" <sabine.randriamasy@nokia-bell-labs.com>
To: Benjamin Kaduk <kaduk@mit.edu>
CC: The IESG <iesg@ietf.org>, "draft-ietf-alto-unified-props-new@ietf.org" <draft-ietf-alto-unified-props-new@ietf.org>, "alto-chairs@ietf.org" <alto-chairs@ietf.org>, "alto@ietf.org" <alto@ietf.org>, Vijay Gurbani <vijay.gurbani@gmail.com>
Thread-Topic: Benjamin Kaduk's Discuss on draft-ietf-alto-unified-props-new-21: (with DISCUSS and COMMENT) - DISCUSS + COMMENTS on section
Thread-Index: Adf2cFh/j6dIehEySwiDNiTSpNKmTgbf0pUgAIPHuAAA54sAgA==
Date: Tue, 01 Feb 2022 18:58:04 +0000
Message-ID: <PAXPR07MB7806B8930A086557679DB23095269@PAXPR07MB7806.eurprd07.prod.outlook.com>
References: <PR3PR07MB701862A2070F8761B23CD2AA957C9@PR3PR07MB7018.eurprd07.prod.outlook.com> <PAXPR07MB7806AF0BE3FF61955EA558C0955F9@PAXPR07MB7806.eurprd07.prod.outlook.com> <20220128042310.GW11486@mit.edu>
In-Reply-To: <20220128042310.GW11486@mit.edu>
Accept-Language: fr-FR, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nokia-bell-labs.com;
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: a4c2fcd0-7cd9-4b59-b159-08d9e5b4c767
x-ms-traffictypediagnostic: AM5PR0701MB2450:EE_
x-microsoft-antispam-prvs: <AM5PR0701MB24503D93EF2C5A1A0EAE926295269@AM5PR0701MB2450.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-ms-exchange-senderadcheck: 1
x-ms-exchange-antispam-relay: 0
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: FwiwGgSZAVYgcyWsaEQRwuV8UhldtjQ9mJRwbv00MY0+kkbihmiuBKhfKY6IkIMb96jQ2EIUPOY1e/rjTSUAyW6Ug1i3aRpvgvVxe0muBQRwz4tCUKEg/18Kh2GfCUykuuLfWeyCrHA5ry3k+Ft8mvtmIFDjTprSBQolxDxkrCq0KLKn17K8EcZgwAtE/Duh/ZdlL3zk0k+vfyBxs26Yvl+Qc+04WQOQOvYAh6a16yUEZdjgIlUomvSIH1ynhQD3o+HEin0VBFPwN4Mfv6b0encNFqBWtcK3D2+95lPFx75UAG18DobL3UoP3Iwi+eiL72mXg8vBskICT7KrtbYSi89bmXoi672Sn4cmetDW1t85alVgtDfMIPguAI6hHUlKqlQkJTPo0rlvmAZCJpNanycA8d2mpEg3SzQXW5slsRpMS/K3tqGtN/TWu9vtLhKlNBC2SDyI1yFPvJpwi3spYt9STjrkO+m8t75t/2JvZLkr5QDffJfKXcdOreRsci9sz2uRxvOwhZrBSDCE+euhFi7VVYhwq2ulz5ZGHFYhj9qtC94qPN0JdGTgHLIaUDwEXHAtunx7CWNUx3e+qcjgEuzD0JgvUWnk9f8irlNXLsT9h7ZouGUQ/pj3zmhX9LaHfcdx/pGRwrIb+csBl1GyCipIyhRyVivP2+VwYkCWIw+FQmvuuLJrwBGystYGlHmBEdVAY0D9IVKVKgyQ8FeG8ftGcmmuqfxgiQ/KlDK3wOFCQCd16ybJC8OakMDBRoKeaKff6v7wdY1UvsGv7XAbuigt1uGv2jHdRKL1nYt7pmpjd4UxwitsahqEBTMOyDugNX4wFF4NGAAl/BqfR683pI9ULlk09KOQx2UtcQJJuc9VHkY2RUUvJhbN+HBC8zM7
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PAXPR07MB7806.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(4636009)(366004)(33656002)(66574015)(71200400001)(2906002)(86362001)(64756008)(76116006)(8676002)(66946007)(8936002)(4326008)(38070700005)(66556008)(52536014)(9686003)(7696005)(26005)(5660300002)(186003)(66476007)(6506007)(66446008)(83380400001)(316002)(30864003)(54906003)(6916009)(38100700002)(122000001)(55016003)(508600001)(82960400001)(20210929001); DIR:OUT; SFP:1102;
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: WOL0HSsxC1t14jC7QnPFgRbaDs8H2vonqPuwXHRHUWbMJrQoYZXAGtoQTXZ+aTV08aDy096/dlIgd82tv/dQONCrz2NQcBwodBYSaZcJEjOzo9zv+OM7Ugi14lILcW3k/6+/ZDZCG+Y4CpS3rUihpy/Mf903Ec3TQ+E6Zg9Mj03n7JTyydGznRRnOg+SFkaekKi3XVYI7koVXAcKMjKXa1IAqZ3lZTT/L+EF2HhSbJsPb3mgANsRySEoVbGsdKF6CoiFUhmBJjTGbjg1GVIWQReAeusTtoCngtE4qLwGyQw2kX2wNcXuDOTuKl8mpHr3erQF0FQbHkcd2T7yLs6XMmOveYEuM2h0ufwVQnoMJ2/aav3pKLyTrnPa+XsOYOOHFb8B7evjUxlAch2WFk1FtcME6f4U+vVLps2BkqdKqXCaGnf+pY8imXB9dBlcuZl6C59byKMO7GRJA966KbKoAC3t93eRnHcGhGwpgzL3z0sRm8Fip2yqOGM/nhiGjgLvnX6sx79WjoaFflG/zV0MkbjMB1Y3dC+m1e2yEa9UCAbf1O68eed4igl4oOGfGlP7E5ImluSsBYLTin8jrEp1tvrGJ8C4bnl2CCigkLSGSJJg56Z1gFsYqRjFFOgHpJVYtyIhppklKqiLzsLACnzJT5BdIbNmomsu/mLstiOLIOk+QF29uhstQm6i6/O6mvKm9YsSrHwPcuOi6SmoxVx/Nfm7EQUO4kGC5KiipML715GpU8ykpU5QG39FIYWqEgNTMb0bhX/xXEEkZQ/LYBnFlIwKFZW9If8Wp6CSWEXYdR8fgB5a1xWVioeACGJY2vzfIk4fMoigiM1wdaQfHJXx2RCVieTnjjIBmesmSVogWRrY8uSm3cfXKimApNTaUtjXxGb0x274DfLTQMXqa2kRURp3/t7C4WbF/3UBAr8s0nf3Zy5wCdmJIxz/Roq+UJ227E6RDuZrXoqnXV9xcmIvy2YD3vyAej2mf/ry5BFuj0jHdlgcvB3ysJEl7VFVmAjrwIj4kVyQjwHg0ZWPt4A6Kr4OPIO9pd5RkC24dkzq9qlarysl5qFFCICafkYGyD0PdDOu+ovhTIz+kXQp8d9ChMZ/iaJ3ZaqwkJR9rNal3ORjvBNjRKoNbWUDrRNLmLeMKLUvqhATM7fUvbyXprrneOi4n6TdTxAYsmcbKNC93mQ7IJchTF2HAo0qVEx9wh/OHaPPtEpWeAf6lW53IR/aiiCUJN5erYUq11i6PkZEhToEmtBUIxGkN+W3y4pGO0T8hOBmTOrJRYq9DhHcH8F8bMCNdvL+2rJLo6TEWey0MJmobIWr00/GF3xoxcpyD5+Fk3L0+JhpJeG8juvL9uVuN1RJS493KLqMMJthztdj8h3DWet4WPsmngjbnvwG+Ocb8RjcJ87CZW700VYt93GfZpC76aFospZXhl0wL+2+rkHY/sYjvlnb9tMCdct4E869m5FkBQmqMOpcIp9+/TK6m7tTp4YqowRf6+t01r0+FkgYjtjf4hFoOlOyiUtPhMKmDq4nYsVFgonCo7Mep3TFtWP+sCg93b9zHA5zVbuVkYI3Qf/k4epM9u/8pM4tN8nur+6AE66+od8FTfZ/7IxKdIwLcs4Y+9s9r7XTytwa9NpvpE7bwh+7sgQF1VlyyCJ+0LpsLRrX7xDvYC1BhtmfgnX55AdVmseLFiSvhtJDpOALUnCLxUcmMNb2Yp5yGUYI
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nokia-bell-labs.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PAXPR07MB7806.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a4c2fcd0-7cd9-4b59-b159-08d9e5b4c767
X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Feb 2022 18:58:05.1190 (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: kLEu1iA3NN7K4pD3b6JNDcd5a7qUE0Y9SNUNPz56Ddi/DNX6p+/VrQ954H4vSTrpjFVg3o1BsL2QUIpGqdKfXqJwwgoq/puyRGt+Lx6h3P3jG4Oq5I6na+WDsLXP+Apk
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM5PR0701MB2450
Archived-At: <https://mailarchive.ietf.org/arch/msg/alto/xZt1JIyzy1Ai5o8l74Dea_saPlg>
Subject: Re: [alto] Benjamin Kaduk's Discuss on draft-ietf-alto-unified-props-new-21: (with DISCUSS and COMMENT) - DISCUSS + COMMENTS on section
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, 01 Feb 2022 18:58:24 -0000
Hi Benjamin, Thanks a lot, no worries. Please find my responses inline. Best regards, Sabine >-----Original Message----- >From: Benjamin Kaduk <kaduk@mit.edu> >Sent: Friday, January 28, 2022 5:23 AM >To: Randriamasy, Sabine (Nokia - FR/Paris-Saclay) ><sabine.randriamasy@nokia-bell-labs.com> >Cc: The IESG <iesg@ietf.org>; draft-ietf-alto-unified-props-new@ietf.org; >alto-chairs@ietf.org; alto@ietf.org; Vijay Gurbani <vijay.gurbani@gmail.com> >Subject: Re: Benjamin Kaduk's Discuss on draft-ietf-alto-unified-props-new- >21: (with DISCUSS and COMMENT) - DISCUSS + COMMENTS on section > >Hi Sabine, all, > >Thanks for posting the -22, and my apologies for not having responded earlier >to the thread -- there were a lot of things going on for me. >I'm happy to say that the changes have addressed my discuss points, and I will >go update my ballot position shortly. > >A few remaining notes inline... > >On Tue, Jan 25, 2022 at 01:34:06PM +0000, Randriamasy, Sabine (Nokia - >FR/Paris-Saclay) wrote: >> Hello Benjamin, >> >> We have posted a revision that addresses the DISCUSS points raised in your >review, together with the related COMMENTS on section 10. >> Please see inline the e-mail below for the proposed updates. >> The other comments will be addressed in a next revision. >> Again thank you for your thorough review and guidance. >> Best regards, >> Sabine and co-authors >> >> >> >-----Original Message----- >> >From: Randriamasy, Sabine (Nokia - FR/Paris-Saclay) >> ><sabine.randriamasy@nokia-bell-labs.com> >> >Sent: Tuesday, December 21, 2021 3:23 PM >> >To: Benjamin Kaduk <kaduk@mit.edu>; The IESG <iesg@ietf.org> >> >Cc: draft-ietf-alto-unified-props-new@ietf.org; alto-chairs@ietf.org; >> >alto@ietf.org; Vijay Gurbani <vijay.gurbani@gmail.com> >> >Subject: RE: Benjamin Kaduk's Discuss on draft-ietf-alto-unified-props- >new-21: >> >(with DISCUSS and COMMENT) - DISCUSS + COMMENTS on section >> > >> >Hello Benjamin, >> > >> >Thank you very much for your thorough review and guidance. >> >The present e-mail addresses your DISCUSS points and COMMENTS on >> >section 10, as they relate to a DISCUSS point. >> >Please see the responses inline. >> >The other COMMENTS and NITS will be addressed in a separate e-mail. >> > >> >Best regards, >> >Sabine, Jensen, Kai, Richard >> > >> >>-----Original Message----- >> >>From: Benjamin Kaduk via Datatracker <noreply@ietf.org> >> >>Sent: Thursday, December 2, 2021 12:00 AM >> >>To: The IESG <iesg@ietf.org> >> >>Cc: draft-ietf-alto-unified-props-new@ietf.org; >> >>alto-chairs@ietf.org; alto@ietf.org; Vijay Gurbani >> >><vijay.gurbani@gmail.com>; vijay.gurbani@gmail.com >> >>Subject: Benjamin Kaduk's Discuss on draft-ietf-alto-unified-props-new- >21: >> >>(with DISCUSS and COMMENT) >> >> >> >>Benjamin Kaduk has entered the following ballot position for >> >>draft-ietf-alto-unified-props-new-21: Discuss >> >> >> >>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/blog/handling-iesg-ballot-positions/ >> >>for more information about how to handle DISCUSS and COMMENT >> >positions. >> >> >> >> >> >>The document, along with other ballot positions, can be found here: >> >>https://datatracker.ietf.org/doc/draft-ietf-alto-unified-props-new/ >> >> >> >> >> >> >> >>-------------------------------------------------------------------- >> >>-- >> >>DISCUSS: >> >>-------------------------------------------------------------------- >> >>-- >> >> >> >>(1) Section 8.6 seems to have some conflicting requirements. The >> >>filtered property map response "MUST include all the inherited >> >>property values for the requested entities and all the entities >> >>which are able to inherit property values from the requested >> >>entities." We then go on to say that to do this, the server MAY >> >>follow three rules, that themselves include SHOULD-level guidance, >> >>but don't say how the MUST is achieved if the SHOULDs or MAY are >> >>ignored. I was expecting to see a construction of the form "SHOULD do X, >but if not, MUST do Y". >> >> >> >[ [SR] ] >> >Indeed the requirements need to set clear rules. We propose to update >> >the text as follows: >> >OLD >> >... >> >To achieve this goal, the ALTO server MAY follow three rules: >> > >> > * If a property for a requested entity is inherited from another >> > entity not included in the request, the response SHOULD include >> > this property for the requested entity. For example, A full >> > property map may skip a property P for an entity A (e.g., >> > ipv4:192.0.2.0/31) if P can be derived using inheritance from >> > another entity B (e.g., ipv4:192.0.2.0/30). A filtered property >> > map request may include only A but not B. In such a case, the >> > property P SHOULD be included in the response for A. >> > >> > * If there are entities covered by a requested entity but having >> > different values for the requested properties, the response SHOULD >> > include all those entities and the different property values for >> > them. For example, considering a request for property P of entity >> > A (e.g., ipv4:192.0.2.0/31), if P has value v1 for >> > A1=ipv4:192.0.2.0/32 and v2 for A2=ipv4:192.0.2.1/32, then, the >> > response SHOULD include A1 and A2. >> > >> > * If an entity identifier in the response is already covered by >> > other entities identifiers in the same response, it SHOULD be >> > removed from the response, for the sake of compactness. In the >> > previous example, the entity A = ipv4:192.0.2.0/31 SHOULD be >> > removed because A1 and A2 cover all the addresses in A. >> >NEW >> >... >> >To achieve this goal, the ALTO server MUST follow two rules: >> > >> > * If a property for a requested entity is inherited from another >> > entity not included in the request, the response MUST include >> > this property for the requested entity. For example, A full >> > property map may skip a property P for an entity A (e.g., >> > ipv4:192.0.2.0/31) if P can be derived using inheritance from >> > another entity B (e.g., ipv4:192.0.2.0/30). A filtered property >> > map request may include only A but not B. In such a case, the >> > property P MUST be included in the response for A. >> > >> > * If there are entities covered by a requested entity but having >> > different values for the requested properties, the response MUST >> > include all those entities and the different property values for >> > them. For example, considering a request for property P of entity >> > A (e.g., ipv4:192.0.2.0/31), if P has value v1 for >> > A1=ipv4:192.0.2.0/32 and v2 for A2=ipv4:192.0.2.1/32, then, the >> > response MUST include A1 and A2. >> > >> >For the sake of response compactness, the ALTO Server SHOULD obey the >> >following rule: >> > >> > * If an entity identifier in the response is already covered by >> > other entities identifiers in the same response, it SHOULD be >> > removed from the response. In the >> > previous example, the entity A = ipv4:192.0.2.0/31 SHOULD be >> > removed because A1 and A2 cover all the addresses in A. >> > >> > >> >>(2) Many of the examples in Sections 10.X do not seem to match up >> >>with the prose that describes them and the previous data tables that >> >>they are intended to illustrate (see COMMENT). We should make sure >> >>that the examples are internally consistent. >> >> >> >[ [SR] ] Please see our responses on the "COMMENTS on SECTION 10" >section. >> > >> >>(3) Section 4.6.2 says: >> >> >> >> * Last, the entity domain types "asn" and "countrycode" defined in >> >> [I-D.ietf-alto-cdni-request-routing-alto] do not have a defining >> >> information resource. Indeed, the entity identifiers in these two >> >> entity domain types are already standardized in documents that the >> >> Client can use. >> >> >> >>But earlier we said that "the defining information resource of a >> >>resource- specific entity domain D is unique", but this seems to be >> >>saying that the defining information resource of domains of the "asn" >> >>and "contrycode" type are *not* unique, by virtue of not existing at >> >>all. How can we rectify these two statements? >> >> >> >[ [SR] ] To rectify this, we propose to update the text in Section >> >4.6.1 as >> >follows: >> >--- parag 2 >> >OLD >> >"...This concept applies to resource-specific entity domains. ..." >> >NEW >> >"...A defining information resource is defined for resource-specific >> >entity domains. It does not exist for entity domains that are not >> >resource-specific such as "ipv4" or "ipv6". Neither does it exist for >> >entity domains that are covering entity identifiers already defined >> >in other standardization documents, at it is the case for country >> >code identifiers standardized in [ISO3166-1] or AS numbers allocated by >the IANA. ..." >> > >> >--- parag 3, >> >OLD >> > "the defining information resource of a resource-specific entity >> >domain D is unique... " >> >NEW >> >"the defining information resource of a resource-specific entity >> >domain D, when it exists, is unique... " >> >> >> >>-------------------------------------------------------------------- >> >>-- >> >>COMMENT:[ [SR] ] on SECTION 10 >> >>-------------------------------------------------------------------- >> >>-- >> >> >> >>Section 10.x >> >> >> >>I am not really an HTTP expert, but the content-lengths in these >> >>examples seem to be based on byte counts with Unix-style line >> >>separators, whereas (per >> >>draft-ietf-httpbis-messaging) my understanding is that the values >> >>should be computed with CRLF as the line separator. >> >> >> >[ [SR] ] In this document, all the examples use Unix-style line >> >separators. But in neither draft-ietf-httpbis-messaging-19 nor >> >RFC7230, did we find any statement saying that the content-length >> >value should be computed with CRLF as EoL. CRLF is only used to >> >separate different HTTP header field lines and chunked data (Sec 2.1 >> >and 7.1 of draft-ietf-httpbis-messaging-19), not the message body. > >Looking at this more carefully, I now think that what you describe here is >correct. Sorry for making the extra work for you due to my misunderstanding! > [ [SR] ] No problem >> >If you think the content-length computation is not clear, how about >> >we add the following sentence at the beginning of Sec 10: >> >NEW >> >In this document, the HTTP message bodies of all the examples use >> >Unix-style line-ending character (%x0A) as the line separator. > >This is okay to include, but in light of the above, not needed. If you want to >remove it again, I do not object. [ [SR] ] Then, we will consider removing the sentence > >> >>Section 10.2 >> >> >> >> Beyond "pid", the examples in this section use four additional >> >> properties for Internet address domains, "ISP", "ASN", "country" and >> >> "state", with the following values: >> >> >> >>Are these property names, types, or both? >> >>Should we use "countrycode" that is defined by >> >>draft-ietf-alto-cdni-request- routing-alto, rather than the very >> >>similar >> >sounding "country"? >> >> >> >[ [SR] ] yes, we can use "countrycode" and will update the examples >> >accordingly OLD >> > Beyond "pid", the examples in this section use four additional >> > properties for Internet address domains, "ISP", "ASN", "country" and >> > "state", with the following values: >> >NEW >> > Beyond "pid", the examples in this section use four additional >> > property types for entities of domain type "ipv4": "countrycode", >> >"ASN", "ISP", and "state". >> >These properties are assumed to be resource-agnostic so their name is >> >identical to their type. >> >The entities have the following values: > >Excellent, thank you for this and for propagating the change through the >whole section. (Doing a quick search for the string "country" in the -22, I see it >appears in §10.6, but I have lost track of whether a >country->countrycode replacement would be appropriat there or not.) > [ [SR] ] We will double check, thanks >The rest of this all looks good. > >Many thanks, > >Ben > >> >>Section 10.3 >> >> >> >> The following IRD defines ALTO Server information resources that are >> >> relevant to the Entity Property Service. It provides two property >> >> maps: one for the "ISP" and "ASN" properties, and another one for the >> >> "country" and "state" properties. [...] >> >> >> >>I may be misreading things, but I could only find the former of >> >>these two. I should be looking for resources that have the >> >>"application/alto- >> >>propmap+json" media-type and do not accept parameters, right? >> >> >> >[ [SR] ] thanks for catching this. This text is inherited from previous >versions. >> >We will update as follows: >> >OLD >> >The following IRD defines ALTO Server information resources that are >> > relevant to the Entity Property Service. It provides two property >> > maps: one for the "ISP" and "ASN" properties, and another one for the >> > "country" and "state" properties. [...] NEW The following IRD >> >defines ALTO Server information resources that are >> > relevant to the Entity Property Service. It provides a property >> > map for the "ISP" and "ASN" properties. >> > >> >> The server provides several filtered property maps. The first >> >> returns all four properties, and the second returns only the "pid" >> >> property for the default network map. >> >> >> >>Does it also return the "pid" property for the alt-network-map? >> >> >> >[ [SR] ] Yes, thanks. >> >OLD >> >... the second returns only the "pid" property for the default network >map. >> >NEW >> >... the second returns only the "pid" property for the default >> >network map and the "alt-network-map". >> > >> >> The filtered property maps for the "ISP", "ASN", "country" and >> >> "state" properties do not depend on the default network map (it does >> >> not have a "uses" capability), because the definitions of those >> >> >> >>I only see "ISP" showing up in the ia-property-map and the >> >>iacs-property-map, both of which list "uses" for the default-network- >map. >> >> >> >[ [SR] ] Indeed, thanks. >> >We will remove member "uses": [ "default-network-map", "alt-network- >map" >> >] from "ia-property-map" and "iacs-property-map" >> > >> >> Note that for legacy clients, the ALTO server provides an Endpoint >> >> Property Service for the "pid" property defined on the endpoints of >> >> the default network map. >> >> >> >>Also the alt-network-map? >> >> >> >[ [SR] ] yes, indeed >> >OLD >> >... the "pid" property defined on the endpoints of the default network >map. >> >NEW >> >... the "pid" property defined on the endpoints of the default >> >network map and the "alt-network-map". >> > >> >>I think there are a couple other property maps in the returned IRD >> >>that are not mentioned in the prose at all (not sure if they need to be). >> >> >> >[ [SR] ] Our intent is that the examples mentioned in the prose helps >> >understanding the other ones. >> > >> >>Section 10.4 >> >> >> >> Note that, to be compact, the response does not include the entity >> >> "ipv4:192.0.2.0", because values of all those properties for this >> >> entity are inherited from other entities. >> >> >> >>Is this really the single IP address, equivalent to 192.0.2.0/32? I >> >>don't see why it's special enough to get called out, as opposed to >> >>the other addresses in 192.0.2.0/27. >> >> >> >[ [SR] ] this is a typo here. "ipv4:192.0.2.0" in this paragraph >> >should be "ipv4:192.0.2.1". >> >OLD >> >Note that, to be compact, the response does not include the entity >> > "ipv4:192.0.2.0", because values of all those properties for this >> > entity are inherited from other entities. >> >NEW >> >Note that, to be compact, the response does not include the entity >> > "ipv4:192.0.2.1", because values of all those properties for this >> > entity are inherited from other entities. >> > >> >> The same rule >> >> applies to the entities "ipv4:192.0.3.0/28" and "ipv4:192.0.3.0/28". >> >> >> >>Should one of these be 192.0.3.16/28? >> >> >> >[ [SR] ] yes indeed, thanks. >> >OLD >> >applies to the entities "ipv4:192.0.3.0/28" and "ipv4:192.0.3.0/28". >> >NEW >> >applies to the entities "ipv4:192.0.3.0/28" and "ipv4:192.0.3.16/28". >> > >> > >> >>Section 10.5 >> >> >> >> Note that the value of "state" for "ipv4:192.0.2.0" is the only >> >> explicitly defined property; the other values are all derived by the >> >> inheritance rules for Internet address entities. >> >> >> >>I think the .2.1 is explicitly defined and .2.0 is inherited... >> >> >> >[ [SR] ] SR, yes indeed, thanks >> >OLD >> >Note that the value of "state" for "ipv4:192.0.2.0" is the only >> > explicitly defined property; the other values are all derived by the >> > inheritance rules for Internet address entities. >> >NEW >> >Note that the value of "state" for "ipv4:192.0.2.1" is the only >> > explicitly defined property; the other values are all derived by the >> > inheritance rules for Internet address entities. >> > >> >> "property-map": { >> >> "ipv4:192.0.2.0": >> >> {".ISP": "BitsRus", ".ASN": "65543", ".state": "PA"}, >> >> "ipv4:192.0.2.1": >> >> {".ISP": "BitsRus", ".ASN": "65543", ".state": "NJ"}, >> >> "ipv4:192.0.2.17": >> >> {".ISP": "BitsRus", ".ASN": "65543", ".state": "CT"} >> >> >> >>...and my reading of the table in §10.2 would have .2.0 as NJ and .2.1 as >PA. >> >> >> >[ [SR] ] Indeed, thanks, we will correct >> > >> >>Section 10.6 >> >> >> >> "ipv4:192.0.2.0": {".state": "PA"}, >> >> >> >>As above, I think this has to be .2.1. >> >> >> >[ [SR] ] yes, thanks. We will correct >> > >> >> "ipv4:192.0.3.0/28": {".ASN": "65543", >> >> ".state": "TX"}, >> >> "ipv4:192.0.3.16/28": {".ASN": "65543", >> >> ".state": "MN"} >> >> >> >>These ASNs should be 65544. >> >> >> >[ [SR] ] yes, thanks. We will correct >> >> >> >> >>
- Re: [alto] Benjamin Kaduk's Discuss on draft-ietf… Randriamasy, Sabine (Nokia - FR/Paris-Saclay)
- Re: [alto] Benjamin Kaduk's Discuss on draft-ietf… Qin Wu
- Re: [alto] Benjamin Kaduk's Discuss on draft-ietf… Randriamasy, Sabine (Nokia - FR/Paris-Saclay)
- Re: [alto] Benjamin Kaduk's Discuss on draft-ietf… Benjamin Kaduk
- Re: [alto] Benjamin Kaduk's Discuss on draft-ietf… Randriamasy, Sabine (Nokia - FR/Paris-Saclay)