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

Alia Atlas <akatlas@gmail.com> Thu, 04 June 2015 21:44 UTC

Return-Path: <akatlas@gmail.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 5472B1AC432 for <rtgwg@ietfa.amsl.com>; Thu, 4 Jun 2015 14:44:21 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -100.099
X-Spam-Level:
X-Spam-Status: No, score=-100.099 tagged_above=-999 required=5 tests=[BAYES_40=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, NORMAL_HTTP_TO_IP=0.001, SPF_PASS=-0.001, USER_IN_WHITELIST=-100] 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 rOyOA1nZnavl for <rtgwg@ietfa.amsl.com>; Thu, 4 Jun 2015 14:44:19 -0700 (PDT)
Received: from mail-ob0-x235.google.com (mail-ob0-x235.google.com [IPv6:2607:f8b0:4003:c01::235]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 7D8C01AC437 for <rtgwg@ietf.org>; Thu, 4 Jun 2015 14:44:17 -0700 (PDT)
Received: by obbqz1 with SMTP id qz1so23756918obb.3 for <rtgwg@ietf.org>; Thu, 04 Jun 2015 14:44:17 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=F8VqC2HS17fGB6/f2SI2LZBtQttgUGeBVUcPJTjLkUw=; b=KB12IEPJ0yLm3HP7FMv9tjqKMwq19ZHXbJyrjpFpkMfsAASV/Qy7QUq0VFirkqsMBY szhaYqtYZes1tgrnqR9hoBfh/Bb7hugTPPAx35ToFY3hXkHOsyDWsgX5efU23uyb6x7Z 4iJRXLO1MKJQcx6080lAzfwTfuXYN959pxbF8hkmbn55oqktugU4sKJkCVobHlfVTGz8 DJ656ep02WYrCR3ENzD58CbyqRnmmoqlrxtAbjcvOEO3d6GvJ1XS/eTOa/HI692fodBM mdkatun69xet1Z9HOS9TMroWR+T3+vn01glqNgryQxTau7GQrjvJPDtXx8lL24unJTVR LtPw==
MIME-Version: 1.0
X-Received: by 10.202.175.87 with SMTP id y84mr120118oie.22.1433454256880; Thu, 04 Jun 2015 14:44:16 -0700 (PDT)
Received: by 10.60.33.167 with HTTP; Thu, 4 Jun 2015 14:44:16 -0700 (PDT)
Received: by 10.60.33.167 with HTTP; Thu, 4 Jun 2015 14:44:16 -0700 (PDT)
In-Reply-To: <CAG4d1rcLc+r268beuL+4iHTLzS=L3x1wX20+eBsW-ZocVwxZEw@mail.gmail.com>
References: <CAG4d1rd1+v5PQLGquh6ufgRCx3c5iRZodwDsmbjuT_0j6-j0dw@mail.gmail.com> <CAG4d1re0++G9rfcoKfa=Uq4O_JZGKRY7dJaVKH7vkLxvxed0Qg@mail.gmail.com> <CAG4d1rcLc+r268beuL+4iHTLzS=L3x1wX20+eBsW-ZocVwxZEw@mail.gmail.com>
Date: Thu, 04 Jun 2015 17:44:16 -0400
Message-ID: <CAG4d1rfcRR0JUwfq-KbKqFAyZ7g5MBMHHZdp4O-Sh2u9PD6ygA@mail.gmail.com>
Subject: AD Review of draft-ietf-rtgwg-lfa-manageability
From: Alia Atlas <akatlas@gmail.com>
To: rtgwg@ietf.org, draft-ietf-rtgwg-lfa-manageability@tools.ietf.org
Content-Type: multipart/alternative; boundary="001a113ce638c880c00517b813c6"
Archived-At: <http://mailarchive.ietf.org/arch/msg/rtgwg/fM6GlDsvmZIm_5oqv7N4BwEozeA>
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: Thu, 04 Jun 2015 21:44:21 -0000

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.

  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.

  2) In 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.

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.

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.

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.

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

  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.

Thanks for the good work,
Alia