Re: [alto] Adam Roach's No Objection on draft-ietf-alto-cost-calendar-09: (with COMMENT)

"Randriamasy, Sabine (Nokia - FR/Paris-Saclay)" <sabine.randriamasy@nokia-bell-labs.com> Fri, 25 January 2019 16:06 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 52A7B13101F; Fri, 25 Jan 2019 08:06:52 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.454
X-Spam-Level:
X-Spam-Status: No, score=-6.454 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-4.553, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, 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 2KWDtHCmw5HA; Fri, 25 Jan 2019 08:06:49 -0800 (PST)
Received: from EUR02-VE1-obe.outbound.protection.outlook.com (mail-eopbgr20090.outbound.protection.outlook.com [40.107.2.90]) (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 B18F9130F9A; Fri, 25 Jan 2019 08:06:39 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nokia.onmicrosoft.com; s=selector2-nokia-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=qg7FHsrvOITmUtH4XA95BMiYCHBLcTDzME5BNEh/nPY=; b=UwxNcHvWhGsgA0Dk4TgobTMzVZikMrZ3WivqdelTHIQe33FSVXXY59SNDKsGlSNISPuoIaSsB8vicUm0qALvqNWcolkw57mPZqTDyxtnzl1LtEVI13gGNhUIvqKFM6vqVlGXkci1Uj9YpLMURvH9/eaND7J87cFPJIN7XiHuzUs=
Received: from AM4PR07MB3236.eurprd07.prod.outlook.com (10.171.189.13) by AM4PR07MB3217.eurprd07.prod.outlook.com (10.171.188.158) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1580.6; Fri, 25 Jan 2019 16:06:37 +0000
Received: from AM4PR07MB3236.eurprd07.prod.outlook.com ([fe80::5c0b:728b:7ba3:9d74]) by AM4PR07MB3236.eurprd07.prod.outlook.com ([fe80::5c0b:728b:7ba3:9d74%5]) with mapi id 15.20.1558.016; Fri, 25 Jan 2019 16:06:37 +0000
From: "Randriamasy, Sabine (Nokia - FR/Paris-Saclay)" <sabine.randriamasy@nokia-bell-labs.com>
To: Adam Roach <adam@nostrum.com>, The IESG <iesg@ietf.org>
CC: "draft-ietf-alto-cost-calendar@ietf.org" <draft-ietf-alto-cost-calendar@ietf.org>, "Gurbani, Vijay (Nokia - US/Naperville)" <vijay.gurbani@nokia.onmicrosoft.com>, "alto-chairs@ietf.org" <alto-chairs@ietf.org>, "Gurbani, Vijay (Nokia - US/Naperville)" <vijay.gurbani@nokia.onmicrosoft.com>, "alto@ietf.org" <alto@ietf.org>
Thread-Topic: Adam Roach's No Objection on draft-ietf-alto-cost-calendar-09: (with COMMENT)
Thread-Index: AQHUh44Cwk630YuXGUeUJ3PuYuNPFKXAaEiA
Date: Fri, 25 Jan 2019 16:06:37 +0000
Message-ID: <AM4PR07MB3236DD059F50035D0F5C8F21959B0@AM4PR07MB3236.eurprd07.prod.outlook.com>
References: <154345970393.13521.17177728478340801020.idtracker@ietfa.amsl.com>
In-Reply-To: <154345970393.13521.17177728478340801020.idtracker@ietfa.amsl.com>
Accept-Language: fr-FR, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=sabine.randriamasy@nokia-bell-labs.com;
x-originating-ip: [135.245.212.89]
x-ms-publictraffictype: Email
x-microsoft-exchange-diagnostics: 1; AM4PR07MB3217; 6:AXLn4ej/yvlTw//tmu2ARomVn0JZoOP5ubFLcKADFq4l81brD08S9gHiECerXwW0jPv7Nv6jbUmayFE1GotFKTaRvT0paCKs0JivxhBvfeqFJE+yKi/4cOifI1e8njzR1UuAGCiXHEOP4LJeIIOxXOH0af23Nx3dgTZfxm2XMAbge/yd8ydwpKc5h+47tzLVxAnKUbc054EmR03kyjlrTXS/tP9c2fHqzp+am63VXWW9VtHoKOEora+TbiEkX55eie6sOe0lD/xt+QPBgtJH9mWv8jD8vJ47WlkEShbMwVfZN6HKxuEzwIsN14sQK0MyPaBFXUE2bDxzfu6URBhl+fQCTr7t84GhH2jOZvd4kqIUvn5Zvq4Ktdv54GkLRTPO1LDBloy486fnwwmTg8B5iMPx7lJM0OA0eHxhis1USoDkR3RIJKsESCdKS9louhKT1dMgAZnNPbJhnIg0231n+Q==; 5:ssJsipvNGRIwXsKBtsw3rgl29ppDEwsyajD4g5I+YWCcSj4W6q/w9Dc7Fsw/9A0kPvZrwDKz1igsEhaYqhHzt3KVCzHpV5xm9dBvbuAZVDKhbrS+KUX7f7YM94Z9eISh7KWEIMlD3la5l2vm2ziL2rLg1rX+bgwFxiv2vZ/rsu+NPFnYi2iidMuVxEVxfR1aNv6pKkipznZSVUUqD1s+Yw==; 7:R85xTcJCqrvR93zkapyf295dIciMI8HvhTLOHfzcumau3xFSTi48vYC9dpBNyR5KgxMCsHn0+/xoFlDm3nVrDrW1VWt57JOgSyXzNi/KJgKRnht50454N5oporyxf4eWrlit8EG0aQKSWP8//MmZeA==
x-ms-exchange-antispam-srfa-diagnostics: SOS;
x-ms-office365-filtering-correlation-id: 9b956615-d050-401e-ebd8-08d682df15a0
x-ms-office365-filtering-ht: Tenant
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600110)(711020)(4605077)(4618075)(2017052603328)(7193020); SRVR:AM4PR07MB3217;
x-ms-traffictypediagnostic: AM4PR07MB3217:
x-microsoft-antispam-prvs: <AM4PR07MB32178EEBF9984CA77DBDDFE5959B0@AM4PR07MB3217.eurprd07.prod.outlook.com>
x-forefront-prvs: 0928072091
x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(136003)(366004)(39860400002)(376002)(346002)(396003)(54164003)(199004)(189003)(13464003)(14444005)(99286004)(97736004)(6506007)(53546011)(102836004)(7696005)(316002)(76176011)(26005)(256004)(8676002)(8936002)(81166006)(186003)(305945005)(7736002)(478600001)(81156014)(14454004)(2906002)(68736007)(966005)(110136005)(54906003)(446003)(476003)(11346002)(66066001)(6436002)(229853002)(74316002)(105586002)(486006)(53936002)(86362001)(33656002)(71200400001)(71190400001)(55016002)(3846002)(6246003)(4326008)(6116002)(25786009)(6306002)(9686003)(106356001); DIR:OUT; SFP:1102; SCL:1; SRVR:AM4PR07MB3217; H:AM4PR07MB3236.eurprd07.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:0; MX:1;
received-spf: None (protection.outlook.com: nokia-bell-labs.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam-message-info: fmOsAL/HQSUiBr7ebxOmX77k969vUtCAT/UHns7wOH/H82JqkXujmJqAiw51KYlYJvBq6SEzH3qfTPSaWT6pdATEa9qwPIFVdTBoD9Y8mKRL93MPlVKxRs/ueavLuuMYNH8NxUsLt+I85DPTwoYQsR6zqrC+i7QsgTg2s+5ZTh/vzM//cYwfv9Lq3Fu7nS2+YDeH7A0pwiVALQEQErqVTQCaDBHTAyDqmpiIi5I5t94Z9DeedEw5d6V6T1F2kdY1Xzmr2Eht51QAbFyxxoLVv7odKRGafEY1XeW2rHtV67LpZvIdlBDy0RwZ3K9DRDEm8Oxi0p4pE2/sFacszTDK1jfdY8DQOHLRIsrQvEJEnF0Lkrau+MnjPrf46ErtbiJwEXEArZ0TiXjLBtVbuMJY7NmypACCSIZ4l0TcS4FpM1g=
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: nokia-bell-labs.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 9b956615-d050-401e-ebd8-08d682df15a0
X-MS-Exchange-CrossTenant-originalarrivaltime: 25 Jan 2019 16:06:37.1199 (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-Transport-CrossTenantHeadersStamped: AM4PR07MB3217
Archived-At: <https://mailarchive.ietf.org/arch/msg/alto/ZM52ML9vzo53XXf-x-YIS0e0XgY>
Subject: Re: [alto] Adam Roach's No Objection on draft-ietf-alto-cost-calendar-09: (with COMMENT)
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: Fri, 25 Jan 2019 16:06:57 -0000

Hi Adam,

Thanks a lot for your feedback. The JSON examples and ipv6 address ranges will be corrected.
Otherwise, see my answer to one of your comments inline,
All the best for 2019,
Sabine


-----Original Message-----
From: Adam Roach <adam@nostrum.com>; 
Sent: Thursday, November 29, 2018 3:48 AM
To: The IESG <iesg@ietf.org>;
Cc: draft-ietf-alto-cost-calendar@ietf.org; Gurbani, Vijay (Nokia - US/Naperville) <vijay.gurbani@nokia.onmicrosoft.com>;; alto-chairs@ietf.org; Gurbani, Vijay (Nokia - US/Naperville) <vijay.gurbani@nokia.onmicrosoft.com>;; alto@ietf.org
Subject: Adam Roach's No Objection on draft-ietf-alto-cost-calendar-09: (with COMMENT)

Adam Roach has entered the following ballot position for
draft-ietf-alto-cost-calendar-09: No Objection

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/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-alto-cost-calendar/



----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------


Thanks to everyone who worked on creating this extension. It seems useful, and the document is easy to read. I have a handful of suggestions for improvement.

---------------------------------------------------------------------------

General:

I found a surprising number of JSON errors in this document by casual examination.  Those errors I found are called out below, but it is extremely likely that I have missed some issues. Please run the JSON examples in this document through a formal validation prior to publication.

With a reasonably modern version of nodejs, validation can be as simple as something like:


node -e 'console.log(JSON.stringify(JSON.parse(`
  {
    "meta" : {
      "cost-type" : {"cost-mode" : "numerical",
                     "cost-metric" : "routingcost"},
      "calendar-response-attributes" : [
        {"calendar-start-time" : Mon, 30 Jun 2014 00:00:00 GMT,
         "time-interval-size" : "1 hour",
         "number-of-intervals" : 24,
         "repeated": 4
        }
      ],
    }
    "endpoint-cost-map" : {
      "ipv4:192.0.2.2": {
        "ipv4:192.0.2.89"    : [v1, v2, ... v24],
        "ipv4:198.51.100.34" : [v1, v2, ... v24],
        "ipv4:203.0.113.45"  : [v1, v2, ... v24],
        "ipv6:2000::1:2345:6789:abcd" : [v1, v2, ... v24]
      }
    }
  }
`)));'

Running this example shows the first error in this example, involving the date. After fixing that error, you will find the missing comma. And so on.

---------------------------------------------------------------------------

General:

Please expand "ALTO" in the title, in the Abstract, and on first use in the body of the document.
[[SR]] DONE

---------------------------------------------------------------------------

§6:

>  [RFC7285] ensures the availability of such a solution in its  Section 
> 8.3.5.  "Authentication and Encryption", which specifies that  "ALTO 
> server implementations as well as ALTO client implementations  MUST 
> support the "https" URI scheme [RFC2818] and Transport Layer  Security 
> (TLS) [RFC5246]".

Based on this guidance, I would encourage updating all instances of "http://" in this document to instead be "https://". I count four such instances.
[[SR]] DONE

---------------------------------------------------------------------------

§1:

>  In this draft an "ALTO Cost Calendar" is specified by information  
> resources capabilities that are applicable to time-sensitive ALTO  
> metrics.  An ALTO Cost Calendar exposes ALTO Cost Values in JSON

Please cite RFC 8259.
[[SR]] DONE

---------------------------------------------------------------------------

§4.1.3:

>        "calendar-response-attributes" : [
>          "calendar-start-time" : Tue, 1 Jul 2014 13:00:00 GMT,
>          "time-interval-size" : "2 hour",

Nit: The value for calendar-start-time needs to be enclosed in quotation marks.

       "cost-map" : {
         "PID1": { "PID1": [v1,v2, ... v12],
                   "PID2": [v1,v2, ... v12],
                   "PID3": [v1,v2, ... v12] },
         "PID2": { "PID1": [v1,v2, ... v12],
                   "PID2": [v1,v2, ... v12],
                   "PID3": [v1,v2, ... v12] }
       }

I don't quite follow this closely enough to understand what is intended here, but the naked 'v1,v2' in the arrays are not valid JSON. Assuming these are literal values, they need to each be enclosed in quotation marks.

Both of these comments apply to the examples given in §4.2.3 and §4.2.3 as well.

[[SR]] The purpose is actually to lighten the reading. Would the following addition to paragraph 3 of section 4.1.3 be OK ?
"The Server returns Calendars with arrays of 12 numbers. To lighten the text, the arrays in the provided example are symbolized by expression "[v1,v2, ... v12]" that is otherwise not valid in JSON. The same type of symbolization is used in the example Server responses."


---------------------------------------------------------------------------

§4.2.3:

>       "ipv6:2000::1:2345:6789:abcd"

Please use an address from the range reserved by RFC 3489.

This comment applies to the example given in §4.2.4 as well.

>   }
>   "endpoint-cost-map" : {

There is a missing comma after the closing brace.

---------------------------------------------------------------------------

§4.2.4:

>     "calendar-response-attributes" : [
>       {"cost-type-name : num-routingcost"

Nit: This line is missing two quotation marks and a comma. It should be:

        {"cost-type-name" : "num-routingcost",