Re: [Gen-art] Review of draft-ietf-ccamp-flexible-grid-ospf-ext-07

Daniele Ceccarelli <daniele.ceccarelli@ericsson.com> Tue, 07 February 2017 09:37 UTC

Return-Path: <daniele.ceccarelli@ericsson.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 76B501295EA; Tue, 7 Feb 2017 01:37:00 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.219
X-Spam-Level:
X-Spam-Status: No, score=-4.219 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, 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=ericsson.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 Y6AdllDks58B; Tue, 7 Feb 2017 01:36:58 -0800 (PST)
Received: from sesbmg23.ericsson.net (sesbmg23.ericsson.net [193.180.251.37]) (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 3554C127058; Tue, 7 Feb 2017 01:36:57 -0800 (PST)
X-AuditID: c1b4fb25-4bbfb70000006c88-d7-589995350d22
Received: from ESESSHC003.ericsson.se (Unknown_Domain [153.88.183.27]) by (Symantec Mail Security) with SMTP id 6B.D4.27784.53599985; Tue, 7 Feb 2017 10:36:55 +0100 (CET)
Received: from EUR01-HE1-obe.outbound.protection.outlook.com (153.88.183.145) by oa.msg.ericsson.com (153.88.183.27) with Microsoft SMTP Server (TLS) id 14.3.319.2; Tue, 7 Feb 2017 10:36:53 +0100
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ericsson.onmicrosoft.com; s=selector1-ericsson-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=kZaYWAl40rP3n3/bUUfi0wLpE1XqIQDW8UlOj3+SVyU=; b=TinfBy0/czOCPRM0tPaClf0Lv1iO6P8i0+MhbF85PxU9n2Wcs7S0iaWW+Y2QtuUK6mw61x6uCJI/LwWUU2Ze/vrGxTmx95Gr2bmvhvFzpnqmmKMb+GMR43zxHcJrv43ZnyM+ATPrDUIa+iHbt7uwLM3psIhiDwO1dLvLyASI5jE=
Received: from AM2PR07MB0994.eurprd07.prod.outlook.com (10.162.37.152) by AM2PR07MB0993.eurprd07.prod.outlook.com (10.162.37.151) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.888.5; Tue, 7 Feb 2017 09:36:51 +0000
Received: from AM2PR07MB0994.eurprd07.prod.outlook.com ([10.162.37.152]) by AM2PR07MB0994.eurprd07.prod.outlook.com ([10.162.37.152]) with mapi id 15.01.0888.025; Tue, 7 Feb 2017 09:36:51 +0000
From: Daniele Ceccarelli <daniele.ceccarelli@ericsson.com>
To: Pete Resnick <presnick@qti.qualcomm.com>
Thread-Topic: [Gen-art] Review of draft-ietf-ccamp-flexible-grid-ospf-ext-07
Thread-Index: AQHSfnRknsTURaNSRU6G5sP6JBParKFb3HyAgAAVk9CAAFB/AIABAp2A
Date: Tue, 07 Feb 2017 09:36:51 +0000
Message-ID: <AM2PR07MB0994A0EE3F25F2BDF4FA20D4F0430@AM2PR07MB0994.eurprd07.prod.outlook.com>
References: <148616413998.4157.2652633501511666882.idtracker@ietfa.amsl.com> <2ACBB603-E990-4C86-BE83-4FCC741C254B@piuha.net> <AM2PR07MB0994EB1B4B526A4D7A385E8EF0400@AM2PR07MB0994.eurprd07.prod.outlook.com> <98F6A1A1-2444-4782-99A6-3F5532FE1F63@qti.qualcomm.com>
In-Reply-To: <98F6A1A1-2444-4782-99A6-3F5532FE1F63@qti.qualcomm.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=daniele.ceccarelli@ericsson.com;
x-originating-ip: [151.0.200.100]
x-ms-office365-filtering-correlation-id: 89430744-6d54-4be3-76e9-08d44f3cd89c
x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001);SRVR:AM2PR07MB0993;
x-microsoft-exchange-diagnostics: 1; AM2PR07MB0993; 7:0TVbv4OBIJIOxY/wkMj88yoBi/co9Sv26VhsCevf+F0KHR7IYR/nZiqxabH4VwCYCoy25fM0fqcWxwN2QVh7fWGKSFtx4mBQdNxg7wNqKYT2VlSfjFMj4bNzguB7LAjoo6F3kxbNTwJe7GiWOgr+kT/mbXTzB2jJpHDCNkrDty4Jn5EhumLvBQY1FdwcU7P4b1wcwuzDlZsV5Nv05/8MPwcb+ny340cffoR0bPrrit8wyOGAA01rQVl1SkLlb6XioCiCqk/5xztibE9l9qd+HtG7Q0BpRC8LrVOzmzJphKoLEzIcjR41kPN+xcw09tj/vIP/jbI3qSf7wxsgcZvkgRCOC0ZpMK+tWHXSOgOfz8DIYhQBLtZIw1DT+WMXZFOTXjqDWmf8r9T5J+fobQZcH82tzlafm4mbiYxbQld5gk5vKt4p/svQlFDYQdzEUvl7eBWrhhFJnzNye8Gg3tthE0bdc/c8HI8l4j8Iv/6naS+QsC50aMd4OjN0V4O5eQ9lhHYXYPFtLXU22tG7LO4pXA==
x-microsoft-antispam-prvs: <AM2PR07MB099336B339C9009A6FD040C8F0430@AM2PR07MB0993.eurprd07.prod.outlook.com>
x-exchange-antispam-report-test: UriScan:(37575265505322)(21748063052155);
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(6040375)(601004)(2401047)(2017020603029)(20170203043)(8121501046)(5005006)(3002001)(10201501046)(6041248)(20161123564025)(20161123555025)(20161123560025)(20161123558025)(20161123562025)(6072148); SRVR:AM2PR07MB0993; BCL:0; PCL:0; RULEID:; SRVR:AM2PR07MB0993;
x-forefront-prvs: 0211965D06
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(7916002)(39450400003)(24454002)(199003)(189002)(13464003)(51914003)(51444003)(93886004)(2950100002)(230783001)(122556002)(6916009)(6246003)(2900100001)(229853002)(66066001)(3660700001)(53546003)(54906002)(92566002)(74316002)(68736007)(110136004)(38730400002)(6306002)(53936002)(54896002)(9686003)(5660300001)(86362001)(8676002)(8936002)(77096006)(4326007)(3280700002)(105586002)(76176999)(25786008)(106116001)(54356999)(7696004)(6506006)(236005)(790700001)(102836003)(2906002)(7906003)(3846002)(7736002)(55016002)(6116002)(189998001)(33656002)(966004)(106356001)(606005)(81166006)(101416001)(81156014)(97736004)(6436002)(99286003)(50986999); DIR:OUT; SFP:1101; SCL:1; SRVR:AM2PR07MB0993; H:AM2PR07MB0994.eurprd07.prod.outlook.com; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en;
received-spf: None (protection.outlook.com: ericsson.com does not designate permitted sender hosts)
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: multipart/alternative; boundary="_000_AM2PR07MB0994A0EE3F25F2BDF4FA20D4F0430AM2PR07MB0994eurp_"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-originalarrivaltime: 07 Feb 2017 09:36:51.5597 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 92e84ceb-fbfd-47ab-be52-080c6b87953f
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM2PR07MB0993
X-OriginatorOrg: ericsson.com
X-Brightmail-Tracker: H4sIAAAAAAAAA02SfUhTURjGOffe3V2l0WlqvmlBrsI0tQ9LZ5oUSEga6F+uMG3mTefXZJuV UbIwy2bWiC2dmZoZptX8oI9lFjYsKSyzNGwoFq5AMTCdyPIr513gf7/3fR6e85zDYUhhJ8+L keWoWEWONEtEu1IGyTPvwFC9QbLD1LBObK0coMRVc0ZC3D82RYl/tVRT4vJX92lxX/9Fcj8d XVdnJ6Kf1JXR0bX6XyiOPOoakcpmyU6xiu2Rx13T1U2v6NyHZnTm49QDWo3KXyINcmEA74aa FgOpQa6MEBsRzHaN8LihC8G3XhtyDBQuJeFtY7VT0RFQardR3PAGge1aJa1BDEPjvWA1xzpy 3XEA6G4/X84lcREBVl0V5RDccAx0fG3lc6ZYsD/vITk+CPrC7uVSFN4M7QudhCNTgBPhsiae O+sqAXN/KnkOjwuOgr8N88t+hNfCzPuHhINJ7AkWazXBXQ5DXTuXD9gDRkcWeJw/BZqKTE6P D3xqrHDyYdB3VKH//Nt0k89xNwXFdsxxJgxbSiiOD8Gtge+EoxzgpwQM6PudQeuhvrnMKbyg YfRF83KSELNQ/6gIcS/hBUN9V5AW+VWsKM6xHIxDdykHC/AaeGewLjGztPeDprbtnMUHdCU/ +BxvhaLK2/yV+xrEb0QeSlaZkp22KziIVchOKJXynKAcVtWKlv7W68ezW0zoy/gBM8IMEq0S 5F4vlwh50lPK/GwzAoYUuQvOaw0SoSBVmn+WVciTFXlZrNKMvBlK5CkIaRhOEOI0qYrNZNlc VvFfJRgXLzWit2WGBoR8jpv2HZEXHpOoLQUxSRlVdy603ZBDseXDhsmw1aO9s/4TzzZZ8hYn Ijsyrs4sxl8KmJ8zT5/eGBlh8tXrA8fcf6q1lgfDPQlwLlU2aasJLn2f2J4ctsXl3p6TSfvc tM22imxJQUP9eMxgljz8+uCRcE2UUTVU6yOeEVHKdOlOf1KhlP4DFhUDpVcDAAA=
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/rC77_dZlHctlOtYiyqUmTYE7S38>
Cc: "draft-ietf-ccamp-flexible-grid-ospf-ext.all@ietf.org" <draft-ietf-ccamp-flexible-grid-ospf-ext.all@ietf.org>, "gen-art@ietf.org" <gen-art@ietf.org>, "ccamp@ietf.org" <ccamp@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>
Subject: Re: [Gen-art] Review of draft-ietf-ccamp-flexible-grid-ospf-ext-07
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 07 Feb 2017 09:37:00 -0000

Hi Pete,

We’re getting there.
Please see in line for further explanation.

Thanks a lot
Daniele

From: Pete Resnick [mailto:presnick@qti.qualcomm.com]
Sent: lunedì 6 febbraio 2017 18:38
To: Daniele Ceccarelli <daniele.ceccarelli@ericsson.com>
Cc: Jari Arkko <jari.arkko@piuha.net>; gen-art@ietf.org; draft-ietf-ccamp-flexible-grid-ospf-ext.all@ietf.org; ccamp@ietf.org; ietf@ietf.org
Subject: Re: [Gen-art] Review of draft-ietf-ccamp-flexible-grid-ospf-ext-07


On 6 Feb 2017, at 7:36, Daniele Ceccarelli wrote:

Hi Jari, Pete,

First of all thanks for the accurate review.

All the nits/editorial comments are correct and will be fixed.

Regarding the minor issue:

4.1.1:

The figure is a bit confusing: There might not exist a "Max Slot Width at Priority
7" if bit 7 is clear in the Priority field, correct?
Perhaps it would be better to just show that as "..." or "Max Slot Width at
Priority n". The thing is, it might only be a single value, followed by the
padding, so having the second value in there might be misleading. (Perhaps
similar constructs are used in other MPLS docs and people will understand. But
it took me a while to figure it out.)

it was difficult to express in the figure the fact that only some of the priorities are advertised. As you said it could be one, two or any number up to 8.
What about a single field (with ~ on the borders to indicate that it can have variable length) saying "Max Slot Width at Prio n" and then say that 16 bits are used for each prio and when an odd number of priorities is used the field is padded to line up with multiples of 32 bits?

Yes, I think that would make more sense to me.

[DC] OK

The discussion of Priority was very confusing for me. In the third sentence, do
you mean, "A bit is set (1) corresponding to each priority represented in the
sub-TLV, and clear (0) for each priority not represented in the sub-TLV"? I don't
understand the MUST/MUST NOT as you had it. And I don't understand the
last sentence at all. Are you trying to say, "The leftmost bit (priority level 0)
MUST be set, and priority level 0 MUST be advertised in the sub-TLV."?
Otherwise, I don't get it.

It means that if the priority field is set to e.g. 10010010 in the following you would find 4 fields indicating respectively: Max Slot Width at Priority 0, Max Slot Width at Priority 3, Max Slot Width at Priority 6 and since they are 3 a 16 bits padding to 32 bits.

OK, then I think my sentence is clearer for that: "A bit is set (1) corresponding to each priority represented in the sub-TLV, and clear (0) for each priority not represented in the sub-TLV."

[DC] Agree. We’ll use your text.

It also means that at least one priority must be advertised (i.e. priority 00000000 is not allowed).

I get that part ("At least one priority level MUST be advertised"). It's the end I don't understand: "that, unless overridden by local policy, SHALL be at priority level 0." What does that mean?

[DC] It means that if only one priority is supported it has to be priority 0. For any particular administrative purpose it could be possible to set it to a different value, but that shouldn’t be done.

I don't understand the MAY in the last sentence. Does that mean that I MAY
also set it to the highest possible nominal central frequency supported by the
link? I don't understand what that sentence is trying to tell me.

An example is provided in section 4.1.2, where the available range goes from -2 to +8 but the range supported by the link goes from -9 to +11. The sentence means that even if not available it could be possible to indicate also n=-9 to indicate the starting point of the range supported by the link.

" In this example, it is assumed that the lowest nominal central
frequency supported is n= -9 and the highest is n=11. Note they
cannot be used as a nominal central frequency for setting up a LSP,
but merely as the way to express the supported frequency range."

I'm ok with dropping the sentence.

I think dropping the sentence would make the most sense.

[DC] Ok for me

Thank you
Daniele

Thanks for considering my suggested changes.

pr

-----Original Message-----
From: Jari Arkko [mailto:jari.arkko@piuha.net]
Sent: lunedì 6 febbraio 2017 12:32
To: Pete Resnick <presnick@qti.qualcomm.com<mailto:presnick@qti.qualcomm.com>>
Cc: gen-art@ietf.org<mailto:gen-art@ietf.org>; draft-ietf-ccamp-flexible-grid-ospf-ext.all@ietf.org<mailto:draft-ietf-ccamp-flexible-grid-ospf-ext.all@ietf.org>;
ccamp@ietf.org<mailto:ccamp@ietf.org>; ietf@ietf.org<mailto:ietf@ietf.org>
Subject: Re: [Gen-art] Review of draft-ietf-ccamp-flexible-grid-ospf-ext-07

Thanks for your review, Pete. Authors, any comments?

Jari

--
Pete Resnick http://www.qualcomm.com/~presnick/<http://www.qualcomm.com/%7Epresnick/>
Qualcomm Technologies, Inc. - +1 (858)651-4478