RE: AD Review of draft-ietf-rtgwg-lfa-manageability

<stephane.litkowski@orange.com> Fri, 12 June 2015 12:55 UTC

Return-Path: <stephane.litkowski@orange.com>
X-Original-To: rtgwg@ietfa.amsl.com
Delivered-To: rtgwg@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id C4D171A9125 for <rtgwg@ietfa.amsl.com>; Fri, 12 Jun 2015 05:55:24 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.597
X-Spam-Level:
X-Spam-Status: No, score=-2.597 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, NORMAL_HTTP_TO_IP=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001, UNPARSEABLE_RELAY=0.001] autolearn=ham
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 YFjxdWkpcxD5 for <rtgwg@ietfa.amsl.com>; Fri, 12 Jun 2015 05:55:21 -0700 (PDT)
Received: from relais-inet.francetelecom.com (relais-ias92.francetelecom.com [193.251.215.92]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id B53361A9121 for <rtgwg@ietf.org>; Fri, 12 Jun 2015 05:55:20 -0700 (PDT)
Received: from omfedm06.si.francetelecom.fr (unknown [xx.xx.xx.2]) by omfedm12.si.francetelecom.fr (ESMTP service) with ESMTP id BF7FE18C800; Fri, 12 Jun 2015 14:55:18 +0200 (CEST)
Received: from Exchangemail-eme2.itn.ftgroup (unknown [10.114.31.69]) by omfedm06.si.francetelecom.fr (ESMTP service) with ESMTP id 9867227C064; Fri, 12 Jun 2015 14:55:18 +0200 (CEST)
Received: from OPEXCLILMA4.corporate.adroot.infra.ftgroup ([fe80::65de:2f08:41e6:ebbe]) by OPEXCLILMA2.corporate.adroot.infra.ftgroup ([fe80::bc1c:ad2f:eda3:8c3d%19]) with mapi id 14.03.0235.001; Fri, 12 Jun 2015 14:55:18 +0200
From: stephane.litkowski@orange.com
To: Alia Atlas <akatlas@gmail.com>, "rtgwg@ietf.org" <rtgwg@ietf.org>, "draft-ietf-rtgwg-lfa-manageability@tools.ietf.org" <draft-ietf-rtgwg-lfa-manageability@tools.ietf.org>
Subject: RE: AD Review of draft-ietf-rtgwg-lfa-manageability
Thread-Topic: AD Review of draft-ietf-rtgwg-lfa-manageability
Thread-Index: AQHQnw+hi+e9HiM5802G1Gm47Uj0nJ2o3Hrg
Date: Fri, 12 Jun 2015 12:55:17 +0000
Message-ID: <13533_1434113718_557AD6B6_13533_1650_1_9E32478DFA9976438E7A22F69B08FF9216674F12@OPEXCLILMA4.corporate.adroot.infra.ftgroup>
References: <CAG4d1rd1+v5PQLGquh6ufgRCx3c5iRZodwDsmbjuT_0j6-j0dw@mail.gmail.com> <CAG4d1re0++G9rfcoKfa=Uq4O_JZGKRY7dJaVKH7vkLxvxed0Qg@mail.gmail.com> <CAG4d1rcLc+r268beuL+4iHTLzS=L3x1wX20+eBsW-ZocVwxZEw@mail.gmail.com> <CAG4d1rfcRR0JUwfq-KbKqFAyZ7g5MBMHHZdp4O-Sh2u9PD6ygA@mail.gmail.com>
In-Reply-To: <CAG4d1rfcRR0JUwfq-KbKqFAyZ7g5MBMHHZdp4O-Sh2u9PD6ygA@mail.gmail.com>
Accept-Language: fr-FR, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.168.234.1]
Content-Type: multipart/alternative; boundary="_000_9E32478DFA9976438E7A22F69B08FF9216674F12OPEXCLILMA4corp_"
MIME-Version: 1.0
X-PMX-Version: 6.2.1.2478543, Antispam-Engine: 2.7.2.2107409, Antispam-Data: 2015.6.2.75418
Archived-At: <http://mailarchive.ietf.org/arch/msg/rtgwg/jxUZntji8_fkNhSUC6wr47u48v8>
X-BeenThere: rtgwg@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Routing Area Working Group <rtgwg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtgwg>, <mailto:rtgwg-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/rtgwg/>
List-Post: <mailto:rtgwg@ietf.org>
List-Help: <mailto:rtgwg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtgwg>, <mailto:rtgwg-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 12 Jun 2015 12:55:24 -0000

Hi Alia,

Many thanks for your review, I will address them shortly and publish a new version.
Some comments inline.


Best Regards,

Stephane

From: rtgwg [mailto:rtgwg-bounces@ietf.org] On Behalf Of Alia Atlas
Sent: Thursday, June 04, 2015 23:44
To: rtgwg@ietf.org; draft-ietf-rtgwg-lfa-manageability@tools.ietf.org
Subject: AD Review of draft-ietf-rtgwg-lfa-manageability


As is customary, I have done my AD Review of this draft.   Thank you for a clearly written and well thought out draft.

I do have some minor concerns,  as below, but I am also letting this draft move to IETF Last Call while they are addressed.   I will need an updated draft by June 18, so the draft can go on the IESG telechat on June 25.

Minor comments:

0)      This draft has 6 authors.  Please prune down to 5 or assign an editor or two.

[SLI] Everyone listed worked hardly on the text as well as on specifications, I will put myself as editor to keep everyone ☺.

  1) In section 6.2.1, it says " When selecting the best alternate, the selection algorithm MUST
      consider all available alternates (connected or tunnel).  Especially,
      computation of PQ set ([I-D.ietf-rtgwg-remote-lfa]) SHOULD be
      performed before best alternate selection."

      Instead of "Especially" with a SHOULD - which implies that Remote LFA should always be run, could you change it to:  "For example with Remote LFA, computation of PQ set ...."?   I think the manageability concerns in this document are useful regardless of the fast-reroute technology and this is only a good example of an implementation ordering that is important.

[SLI] Fixed.

  2) In 6.2.4.1<http://6.2.4.1>: " attributes from PLR to alternate path are retrieved from the
      interface connected to the alternate."

      There can be multiple interfaces.  The correct behavior (union or  evaluate once per different interface) should be clearly described.  The similar issue exists for the alternate path and in 6.2.4.2, but there may be more or less freedom about controlling which path is taken.

[SLI] I need to discuss with my co authors on that.



3) In Sec 6.2.6, "Maintain a preference system between alternates based on number of
      SRLG violations : more violations = less preference."   The way that I've seen SRLGs used as a soft restriction is by giving each SRLG a value.  Then one can prefer the lower sum.  This allows different consideration and valuation of the SRLGs.  Of course, this can fall back to each SRLG has a value of 1.   Could you please loosen the assumption here about equally valuing the SRLGs?  I'd prefer to see both alternatives allowed - but that is <no-hat>technical opinion</no-hat> whereas loosening the assumption is about not accidentally forcing more limited behavior and removing the ability to implement more sophisticated mechanisms.

[SLI] Right, here is a new text proposal which is more open:

“

When SRLG protection is computed, and implementation SHOULD permit to :

                                                <list style="symbols">

                                                <t>Exclude alternates violating SRLG.</t>

                                                <t>Maintain a preference system between alternates based on SRLG violations. How the preference system is implemented is out of scope of this document but here are few examples :

                                                <list style="symbols">

                                                <t>Preference based on number of violation. In this case : the more violation = the less preferred.</t>

                                                <t>Preference based on violation cost. In this case, each SRLG violation has an associated cost. The lower violation cost sum is preferred.</t>

                                                </list>”

The path considerations mentioned in (2) still apply.

4) In Sec 6.2.7, you might be interested in the link/node-attribute drafts that are being finished.

[SLI] Could you give me the pointers of drafts you are thinking about ?



5) In Sec 6.2.8: "The bandwidth criteria of the policy framework SHOULD work in two
   ways"  Please expand to "at least two ways" - there are other strategies as well that might be reasonable and no standardization reason to rule them out.

[SLI] Agree, fixed

Nits:
   a) Introduction needs to be the first section. Terminology can follow.

[SLI] Fixed

  b) Remote LFA reference needs updating to RFC 7490.  I think,  given some of the details in this draft,  that it should be a normative reference.

[SLI] Fixed.



Thanks for the good work,
Alia

_________________________________________________________________________________________________________________________

Ce message et ses pieces jointes peuvent contenir des informations confidentielles ou privilegiees et ne doivent donc
pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce message par erreur, veuillez le signaler
a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration,
Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci.

This message and its attachments may contain confidential or privileged information that may be protected by law;
they should not be distributed, used or copied without authorisation.
If you have received this email in error, please notify the sender and delete this message and its attachments.
As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified.
Thank you.