Re: [Gen-art] Gen-ART Last Call review of draft-ietf-rtgwg-rlfa-node-protection-09

Pushpasis Sarkar <pushpasis.ietf@gmail.com> Fri, 30 December 2016 02:10 UTC

Return-Path: <pushpasis.ietf@gmail.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 85B58129A3B for <gen-art@ietfa.amsl.com>; Thu, 29 Dec 2016 18:10:36 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 0
X-Spam-Level:
X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, GUARANTEED_100_PERCENT=2.699, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001] autolearn=no autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.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 P_DWfp6H6K7q for <gen-art@ietfa.amsl.com>; Thu, 29 Dec 2016 18:10:35 -0800 (PST)
Received: from mail-yw0-x22c.google.com (mail-yw0-x22c.google.com [IPv6:2607:f8b0:4002:c05::22c]) (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 D20FD129A37 for <gen-art@ietf.org>; Thu, 29 Dec 2016 18:10:34 -0800 (PST)
Received: by mail-yw0-x22c.google.com with SMTP id r204so218297321ywb.0 for <gen-art@ietf.org>; Thu, 29 Dec 2016 18:10:34 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=vmLPkRsZr9uBJOo7NuTUSZssmwrVqMT0X/17H1cnX7c=; b=nWEbB15RNi7efOsKfej5Iqbl/8g7gHlvUzReU503E243yB1EoH4qSynPn56jIxZqFM 2KLDNYyKx0hHuojsDh8rfMaOnCxYY7QLEjj2Vzl5A6kx5cEXEA/UuJ1i4lOwBr0ewVgw lcaujVvYCoLBTTlE+DTA4dZ6FI0AB1WCyoPoZCThvBYT+S6sngFxVF7A7SwztENiT7XX XNhDCIQ0sTX8GuMTrJYWmi6s7z8hUg/TwXejIeUM9CwIIUBnFOq/Et87Fl8M/4EYjj8y apwbfyJWcGZ5EWgXoPHGwxgQ/yUhK8ngAmvqKUvy3b5VMQXwQcBGIcvLSxrfvn0ivw1L xsVg==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=vmLPkRsZr9uBJOo7NuTUSZssmwrVqMT0X/17H1cnX7c=; b=KyRUr3KXen1rlzX7xNzyq7xApnr+ecBKn09TFJyKVoNLMUccPpKYhu+aZEyLpg7rWH FhnlhIlnCKarCPpPZ1ZZHH0zBBDYRKksT9d3eKOJ7Ec6ufMnfLoLqBBo3eTpbK95QGSE tUUdnaC5gf2EM1I05BTHqLv6BjKSe+ju6PCVHjCB09T+7NQeAvgH130U7xPeM5boBW02 q/QNWHQ94fWtJhxu9FNizd62zQY76aBWLnunQesVKfCQTsHy6zKiweMsPDjF66AJHT67 0XW450Yn8is0ul3O5FFwF9fCvnywjjDX8KC4T+KfKXhkcmsbdjwytFZkbNgY5Hhpb617 EZQg==
X-Gm-Message-State: AIkVDXKGADutSjQP1dE3A8r7QYxlewV7iyZ9DMMLShyDlPPXo/XkqUJ5zdtg37mt/4EaM7chJpGgmmLSo7RXFA==
X-Received: by 10.129.182.96 with SMTP id h32mr28916347ywk.165.1483063834030; Thu, 29 Dec 2016 18:10:34 -0800 (PST)
MIME-Version: 1.0
Received: by 10.37.216.82 with HTTP; Thu, 29 Dec 2016 18:10:33 -0800 (PST)
In-Reply-To: <ABCAA4EF18F17B4FB619EA93DEF7939A4E842DE0@eusaamb107.ericsson.se>
References: <ABCAA4EF18F17B4FB619EA93DEF7939A4E84241B@eusaamb107.ericsson.se> <CAEFuwkgmiWR7=Xj6Tdf=-vn3-n4prr1GRAY+ZGup4UuetujSdg@mail.gmail.com> <ABCAA4EF18F17B4FB619EA93DEF7939A4E842DE0@eusaamb107.ericsson.se>
From: Pushpasis Sarkar <pushpasis.ietf@gmail.com>
Date: Fri, 30 Dec 2016 07:40:33 +0530
Message-ID: <CAEFuwkjn3dkaOW+n4fwv_Y2epgcizd5YdWg6_Xy79_PQX61tFw@mail.gmail.com>
To: Meral Shirazipour <meral.shirazipour@ericsson.com>
Content-Type: multipart/alternative; boundary="f403045e678801b8160544d6b5f4"
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/JnFG1ukHt451LzkdWk6QmIcY7zA>
Cc: "gen-art@ietf.org" <gen-art@ietf.org>, "draft-ietf-rtgwg-rlfa-node-protection.all@tools.ietf.org" <draft-ietf-rtgwg-rlfa-node-protection.all@tools.ietf.org>
Subject: Re: [Gen-art] Gen-ART Last Call review of draft-ietf-rtgwg-rlfa-node-protection-09
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: Fri, 30 Dec 2016 02:10:36 -0000

Hi Meral,

I have addressed all your comments and uploaded version 10. Thanks once
again for the detailed review. Please let me know if you have any further
comments.

Thanks and regards,
-Pushpasis

On Thu, Dec 29, 2016 at 10:33 PM, Meral Shirazipour <
meral.shirazipour@ericsson.com> wrote:

> Hi,
>
>   Many thanks for the response.
>
>
>
> Best Regards,
>
> Meral
>
>
>
> *From:* Pushpasis Sarkar [mailto:pushpasis.ietf@gmail.com]
> *Sent:* Thursday, December 29, 2016 12:20 AM
> *To:* Meral Shirazipour <meral.shirazipour@ericsson.com>
> *Cc:* draft-ietf-rtgwg-rlfa-node-protection.all@tools.ietf.org;
> gen-art@ietf.org
> *Subject:* Re: Gen-ART Last Call review of draft-ietf-rtgwg-rlfa-node-
> protection-09
>
>
>
> Hi Meral,
>
>
>
> First, many many thanks to you for the detailed review comments. :)
>
>
>
> Next, please find few replies inline. Rest of the comments has been
> accepted as is, and will be addressed soon.
>
>
>
> Thanks
>
> -Pushpasis
>
>
>
>
>
> On Wed, Dec 28, 2016 at 11:03 PM, Meral Shirazipour <
> meral.shirazipour@ericsson.com> wrote:
>
> I am the assigned Gen-ART reviewer for this draft. The General Area Review
> Team (Gen-ART) reviews all IETF documents being processed by the IESG for
> the IETF Chair.  Please treat these comments just like any other last call
> comments. For more information, please see the FAQ at <
> http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
>
>
>
>
>
> Document: draft-ietf-rtgwg-rlfa-node-protection-09
>
> Reviewer: Meral Shirazipour
>
> Review Date: 2016-12-28
>
> IETF LC End Date:   2017-01-04 (extension 2017-01-11)
>
> IESG Telechat date: 2017-01-05 (extended)
>
>
>
>
>
> Summary:
>
> This draft is ready to be published as Standards Track RFC but I have
> comments.
>
>
>
> Major issues:
>
> Minor issues:
>
> Nits/editorial comments:
>
>
>
> -Please spell out Loop-Free Alternate (LFA), Shortest Path First (SPF) at
> first use.
>
>
>
> -typo in draft header:
>
> "Internet-Draft   R-LFA Node-Protection and Manageabilty    December 2016"
>
>                 "Manageabilty"--should be-->"Manageability"
>
>
>
> -Same typo in Section 3 header: "Manageabilty of Remote-LFA Alternate
> Paths"
>
>                 "Manageabilty"--should be-->"Manageability"
>
>
>
> -[Page 6] Section 2.2.3.
>
> "on any of the shortest path", path -->"paths"
>
>
>
> -[Page 6] Section 2.2.3.
>
> "from the node Y to primary nexthop E":
>
> "one ECMP path from the node Y":
>
> -->the example in this draft did not use the letter Y for any nodes. Would
> it be clearer to say node Y is defined in Section 2.2.4?
>
>
>
> -Few occurences of "w.r.t to ", the "to" is redundant.
>
>
>
> -[Page 1], "can be utilised "--->"can be utilized"
>
>
>
> -[Page 9], Section 2.3
>
> "Sections Section 2.3.1 and Section 2.3.2 shows "--->"Section 2.3.1 and
> Section 2.3.2 show"
>
>
>
> -[Page 11], "To determine wether"--->"To determine whether"
>
>
>
> -[Page 11], "primary nexthop node"-->"primary nexthop nodes"
>
>
>
> -[Page 13], "choose only the ones that does"--->"choose only the ones that
> do"
>
>
>
> -[Page 13], "not gaurantee"--->"not guarantee"
>
>
>
> -[Page 14], "Figure 7: Toplogy with multiple ECMP primary
> nexthops"--->"Figure 7: Topology with multiple ECMP primary nexthops"
>
>
>
> -[Page 14], "node-proecting"--->"node-protecting"
>
>
>
> -[Page 15], "paths tp PQ-node R2"--->"paths to PQ-node R2"
>
>
>
> -[Page 16], "gaurantees node-protection"--->"guarantees node-protection"
>
>
>
> -[Page 17],  "above example above"--->"above example"
>
>
>
> -[Page 17], "also allow user"--->"also allow the user"
>
>
>
> -[Page 18], "the the computing"---->"the computing"
>
>
>
> -[Page 18], "in section Section 2.3.2."---->"in Section 2.3.2." (2
> occurrences)
>
>
>
> -[Page 18], "in section Section 2.3 the"---->"in Section 2.3 the"
>
>
>
> -[Page 18], "i.e from "---->"i.e. from "
>
>
>
> -[Page 18], "two Remote-LFA alternate path"--->"two Remote-LFA alternate
> paths"
>
>
>
> -[Page 19], "the approach proposed"----->"the proposed approach "
>
>
>
> -[Page 19], "is needed keep "---->"is needed to keep"
>
>
>
> -[Page 19], "entire toplogy"---->"entire topology"
>
>
>
> -General: Is there any proof or extensive simulation that has proved that
> the mechanism proposed works for various network topologies and not only
> the one shown in the examples?
>
>
>
> [Pushpasis] These mechanisms has been implemented (by atleast one vendor,
> to my knowledge) and has been extensively tested in few of the operator's
> network. As is the case with L-FA (RFC 5286) and R-LFA(RFC 7490)
> specifications, these mechanisms either provide you with a backup
> alternate, or does not. But if a backup is verified using these mechanism,
> it is 100% guaranteed to be loop-free. As you might be already be aware of,
> R-LFA is more suited towards ring-like network topologies. None of these
> mechanisms always provide 100% backup coverage (i.e. they do not always let
> you discover all the possible/feasible backup paths). But depending on
> actual topologies, they can provide coverage somewhere between 100%-86%.
> You may want to refer to RFC 7490 section 9. Please let me know if you need
> any further information on this.
>
>
>
> Thanks
>
> -Pushpasis
>
>
>
>
>
>
>
> Best Regards,
>
> Meral
>
> ---
>
> Meral Shirazipour
>
> Ericsson Research
>
> www.ericsson.com
>
>
>