Re: [Pce] Poll for adoption: draft-dhody-pce-stateful-pce-auto-bandwidth-09

"Adrian Farrel" <adrian@olddog.co.uk> Sat, 28 January 2017 17:04 UTC

Return-Path: <adrian@olddog.co.uk>
X-Original-To: pce@ietfa.amsl.com
Delivered-To: pce@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 763A212962F; Sat, 28 Jan 2017 09:04:55 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.62
X-Spam-Level:
X-Spam-Status: No, score=-2.62 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01] autolearn=ham autolearn_force=no
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 ESA-Xo5Z_XIM; Sat, 28 Jan 2017 09:04:53 -0800 (PST)
Received: from asmtp4.iomartmail.com (asmtp4.iomartmail.com [62.128.201.175]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 80CB8129623; Sat, 28 Jan 2017 09:04:50 -0800 (PST)
Received: from asmtp4.iomartmail.com (localhost.localdomain [127.0.0.1]) by asmtp4.iomartmail.com (8.13.8/8.13.8) with ESMTP id v0SH4emf032287; Sat, 28 Jan 2017 17:04:41 GMT
Received: from 950129200 ([176.241.250.4]) (authenticated bits=0) by asmtp4.iomartmail.com (8.13.8/8.13.8) with ESMTP id v0SH4bVJ032262 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NO); Sat, 28 Jan 2017 17:04:39 GMT
From: Adrian Farrel <adrian@olddog.co.uk>
To: 'Dhruv Dhody' <dhruv.dhody@huawei.com>, 'Jonathan Hardwick' <Jonathan.Hardwick@metaswitch.com>, pce@ietf.org
References: <BY2PR0201MB191078869D7F31052330053284600@BY2PR0201MB1910.namprd02.prod.outlook.com> <04d801d271c7$75a51770$60ef4650$@olddog.co.uk> <23CE718903A838468A8B325B80962F9B8CA6247C@blreml501-mbb>
In-Reply-To: <23CE718903A838468A8B325B80962F9B8CA6247C@blreml501-mbb>
Date: Sat, 28 Jan 2017 17:04:35 -0000
Message-ID: <082c01d27988$9c30e950$d492bbf0$@olddog.co.uk>
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
X-Mailer: Microsoft Outlook 14.0
Thread-Index: AQIr/eg1Q9mZk8qgpXnDhlnKP/1QeALisXMGAYJ7SLagd+dH4A==
Content-Language: en-gb
X-TM-AS-MML: disable
X-TM-AS-Product-Ver: IMSS-7.1.0.1679-8.1.0.1062-22852.001
X-TM-AS-Result: No--7.773-10.0-31-10
X-imss-scan-details: No--7.773-10.0-31-10
X-TMASE-MatchedRID: byfwvk+IcRmnykMun0J1wrmR+C0l9vjVC/ExpXrHizwn+p552csI1dcc ZZI4yBC+fNzR/By+p4ZWf7Zsl1bSmCaJdRfH0U3lXbTfocfAWb+zHFBZZzj/T2d6vNuG6CqyutV 8DW4JUoF2oJyUlMgfqPeJFmZGyHxmuSGcTVJ7rYh7SLbBwHbj8aTYf9v9flolyPRAwD/3aba0IV JxcWBFKBVH2g8usRSkB58MJbm1r5rXTQWfzhUW/Xpuli4Ij9S3JDAZBInjo2YifM7JMNHW6wLyt DvV39h+FqzzhFk7BYZf8868kEpnmStg+qZmq1X/QpxiLlDD9FUxmbT6wQT2aznKpbGL4ChVHACU cDvcWyAcl4Vrs6KMmHZN0waG0PCQ1EPqXk+jMV44jxClIfsQfn0tCKdnhB58uME6WhSqqOHUZxE AlFPo8/cUt5lc1lLg44eEqKw8IZeOZa9FQWt0qLDb3BkJ+xraDjYtTOOpxZWcXhxBnzOXvfpuMA 0FyK0N
Archived-At: <https://mailarchive.ietf.org/arch/msg/pce/zv4UvjPIeYf5zEnCmgjbzrdk_g8>
Cc: draft-ietf-pce-stateful-pce-auto-bandwidth@ietf.org, pce-chairs@ietf.org
Subject: Re: [Pce] Poll for adoption: draft-dhody-pce-stateful-pce-auto-bandwidth-09
X-BeenThere: pce@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
Reply-To: adrian@olddog.co.uk
List-Id: Path Computation Element <pce.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/pce>, <mailto:pce-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/pce/>
List-Post: <mailto:pce@ietf.org>
List-Help: <mailto:pce-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/pce>, <mailto:pce-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 28 Jan 2017 17:04:55 -0000

Hey Dhruv,

Thanks for the careful response.

I've snipped down to  couple of points.

Best,
Adrian

>> I note that this revision seems to have dropped the ability for a PCC
>> to report samples to the PCE and have the PCE do all of the work of
>> determining whether auth-bandwidth adjustments are needed/desirable
>> before performing the various path computations.

[snip]

> [Dhruv] You are right, we have not dropped the model, only decided to pursue
> the second model in a different draft “draft-gandhi-pce-pm-04” which deals
> with reporting parameters to the PCE including bandwidth Utilizations, as well
> as delay, jitter etc. This would be in-line with other RFCs in OSPF, IS-IS and
PCE
> which deals with these parameters together.  

OK. Understood. Maybe a simple note and pointer to the other I-D (Informative
reference) would be helpful and stop future reviewers asking similar questions.

>> ---
>> Did you consider allowing a different Adjustment-Interval for adjustment
>> up and adjustment down (with them defaulting to the same value)?
>
> [Dhruv] No, it is not supported by most of the vendors.

ROFL
It's not even a WG draft yet.

> They keep the same interval but allow setting of different values for the
> overflow and underflow threshold/count. Do you feel that this should
> be added? 

Not sure. If I was writing the code, I would certainly have used two different
timers because that is cheap and easy and flexible. Whether I would have exposed
both as configurable in the first release is debatable.

If I was deploying I think I would want to experiment with making changes more
sticky in one direction than in the other direction. How important that is might
depend on how expensive it is to increase b/w once it has been reduced, and how
likely it is that an increase might fail.

>>---
>> 4.2 has
>>
>> All thresholds in this document could be represented in both absolute
>> value and percentage, and could be used together.
>>
>> I looked but couldn't find. If absolute and percentage are both supplied
>> and imply different thresholds, which one is used? Could be as simple as
>> "the first threshold hit", but this could lead to some flaps that are 
>> not obvious to someone staring at either the up/down percentage 
>> thresholds or at the up/down absolute values.
>
> [Dhruv] In section 5.2.3 (and similarly 5.2.5) states – 
>
> An implementation MAY include both
> sub-TLVs for the absolute value and the percentage, in which case the
> bandwidth is adjusted when either of the adjustment threshold
>  conditions are met.
>
> Which is similar to what you state. 
> Do you suggest we add some guidelines to provide information on which
> threshold is met by logging? 
> Or do you feel strongly about this and we should use only one format? 

I think that this may be flexibility too far. I can't see a way that this would
break anything, but might lead to "unexpected" results. For example...

Up % = 1%
Down % = 10%
Up absolute = 1 GB
Down absolute = 1 MB

This would cause wibbles, but maybe that is what the operator has asked for.

But maybe nothing more to say.