Re: [Teas] Mail regarding draft-tsaad-ccamp-rsvpte-bidir-lsp-fastreroute

"Rakesh Gandhi (rgandhi)" <rgandhi@cisco.com> Wed, 07 January 2015 02:45 UTC

Return-Path: <rgandhi@cisco.com>
X-Original-To: teas@ietfa.amsl.com
Delivered-To: teas@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id B338F1A890D for <teas@ietfa.amsl.com>; Tue, 6 Jan 2015 18:45:26 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.511
X-Spam-Level:
X-Spam-Status: No, score=-14.511 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01, USER_IN_DEF_DKIM_WL=-7.5] 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 jqx3YyjRie7x for <teas@ietfa.amsl.com>; Tue, 6 Jan 2015 18:45:23 -0800 (PST)
Received: from alln-iport-5.cisco.com (alln-iport-5.cisco.com [173.37.142.92]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 195BE1A8900 for <teas@ietf.org>; Tue, 6 Jan 2015 18:45:23 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=11751; q=dns/txt; s=iport; t=1420598723; x=1421808323; h=from:to:cc:subject:date:message-id:in-reply-to: content-id:content-transfer-encoding:mime-version; bh=X6/5oWBV/p4okFJOdub15Q5IWuD6c5SIZjtiLhR6o0I=; b=E3N1+quGKeivPCr7hyS5P/gT7BsUYFZbvVB0a/umEPzkhY4QW4bknBAF pgqU8rKiIVyGAplyUw6TF8P3AJYk+/fwrZMEnE87wEYpvXymkmV6OGoxw 9P2PKCYj9SbKF8dy9ZbJrGnOUc+DuJch1dPRcjVQOp5LhYYk8erwkzFWj Y=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: Ak4FAEmdrFStJV2Q/2dsb2JhbABcgwZSWATGSAqFcwKBEBYBAQEBAX2EDAEBAQQBAQE3MQMLDAIEAQgRAwEBAQEeCSIGBgsUCQgCBA4FiBgDEQ2/Pw2DWgEBAQEBAQEBAQEBAQEBAQEBAQEBARMEBI1KgUgQAgFPBwaEIwWFLIh5hUCBdYFEgQ2CaoI0hVaCIoM5IoNub4FFfgEBAQ
X-IronPort-AV: E=Sophos;i="5.07,711,1413244800"; d="scan'208";a="110934278"
Received: from rcdn-core-8.cisco.com ([173.37.93.144]) by alln-iport-5.cisco.com with ESMTP; 07 Jan 2015 02:45:20 +0000
Received: from xhc-rcd-x13.cisco.com (xhc-rcd-x13.cisco.com [173.37.183.87]) by rcdn-core-8.cisco.com (8.14.5/8.14.5) with ESMTP id t072jKZB013170 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=FAIL); Wed, 7 Jan 2015 02:45:20 GMT
Received: from xmb-aln-x07.cisco.com ([169.254.2.59]) by xhc-rcd-x13.cisco.com ([173.37.183.87]) with mapi id 14.03.0195.001; Tue, 6 Jan 2015 20:45:20 -0600
From: "Rakesh Gandhi (rgandhi)" <rgandhi@cisco.com>
To: Gregory Mirsky <gregory.mirsky@ericsson.com>
Thread-Topic: [Teas] Mail regarding draft-tsaad-ccamp-rsvpte-bidir-lsp-fastreroute
Thread-Index: AdAfw4yPXwLBG3MFToqlKPshDlwAggAFdR6AABXMfIAABlTzgAJ11UUAAAxUN4D//7OagA==
Date: Wed, 07 Jan 2015 02:45:20 +0000
Message-ID: <D0D2066F.4978F%rgandhi@cisco.com>
In-Reply-To: <7347100B5761DC41A166AC17F22DF1121B8CBBCF@eusaamb103.ericsson.se>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/14.3.5.130515
x-originating-ip: [10.82.226.73]
Content-Type: text/plain; charset="us-ascii"
Content-ID: <CE75C9182EE6D5418C26C45783F18C5C@emea.cisco.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Archived-At: http://mailarchive.ietf.org/arch/msg/teas/Mr54vEpXzXhn2w6cSW7-UagR27Q
Cc: "Mike Taillon (mtaillon)" <mtaillon@cisco.com>, Manav Bhatia <manav@ionosnetworks.com>, "frederic.jounay@orange.ch" <frederic.jounay@orange.ch>, TEAS WG <teas@ietf.org>, "Lizhong com>" <lizho.jin@gmail.com>, "Tarek Saad (tsaad)" <tsaad@cisco.com>, "Zafar Ali (zali)" <zali@cisco.com>
Subject: Re: [Teas] Mail regarding draft-tsaad-ccamp-rsvpte-bidir-lsp-fastreroute
X-BeenThere: teas@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Traffic Engineering Architecture and Signaling working group discussion list <teas.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/teas>, <mailto:teas-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/teas/>
List-Post: <mailto:teas@ietf.org>
List-Help: <mailto:teas-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/teas>, <mailto:teas-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 07 Jan 2015 02:45:27 -0000

Hi Gregory,

Thank you for the context. Good stuff.

May be we could state in the draft something along the line of:
"PSC (RFC6378) is applicable to FRR (RFC4090) for local protection
switchover in both directions in order to minimize traffic disruptions. In
the absence of such mechanism, control plane restoration proposed is
required in order to avoid soft state time out".

Would that work?

thanks,
Rakesh



On 2015-01-06 9:18 PM, "Gregory Mirsky" <gregory.mirsky@ericsson.com>
wrote:

>Hi Rakesh,
>thank you for your interest in my comments.
>I can note that during work on MPLS-TP we, as union of MPLS and CCAMP
>WGs, left question of RSVP FRR applicability undocumented. Perhaps we can
>find mailing list discussions where some of us agreed that local
>protection of uni-directional and bi-directional associated LSPs in
>MPLS-TP can follow RFC 4090 but question of local protection of
>bi-directional co-routed LSPs requires further discussion. And there
>already were references to use ASSOCIATION object to support segment
>protection if node protection desired. That would allow monitoring of
>both working and protecting paths and switchover coordination by using
>RFC 6378.
>Of course, proposed mechanism is workable as control plane restoration
>mechanism but I'm not convinced that is what should be standardized as
>local protection mechanism.
>
>	Regards,
>		Greg
>
>-----Original Message-----
>From: Rakesh Gandhi (rgandhi) [mailto:rgandhi@cisco.com]
>Sent: Tuesday, January 06, 2015 5:26 PM
>To: Gregory Mirsky
>Cc: TEAS WG; Tarek Saad (tsaad); Lizhong com>; Mike Taillon (mtaillon);
>Zafar Ali (zali); Manav Bhatia; frederic.jounay@orange.ch
>Subject: Re: [Teas] Mail regarding
>draft-tsaad-ccamp-rsvpte-bidir-lsp-fastreroute
>
>Hi Gregory,
>
>Thank you for your review comments.
>
>Can u please elaborate how does segment protection using ASSOCIATION
>object solve the highlighted issue ?  I am thinking such mechanisms would
>apply to FRR as well.
>
>Thanks,
>Rakesh
>
>
>On 2014-12-25 2:51 AM, "Gregory Mirsky" <gregory.mirsky@ericsson.com>
>wrote:
>
>>Dear All,
>>I think that node protection mode for bidirectional co-routed LSP FRR
>>is cumbersome and, at best, likely cause out-of-order delivery and more
>>service disruption (switching reverse direction on R5 from R4 onto R3
>>triggered by tunneled RSVP Path message). I believe that extension of
>>RFC
>>4090 for bidirectional co-routed LSP FRR is viable only for link
>>protection mode and node protection must be referred to segment
>>protection and use of RSVP ASSOCIATION object (RFC 4872 and RFC 6689).
>>Without such change in the scope of the draft I cannot support its
>>adoption by the WG.
>>
>>	Regards,
>>		Greg
>>
>>-----Original Message-----
>>From: Teas [mailto:teas-bounces@ietf.org] On Behalf Of Loa Andersson
>>Sent: Wednesday, December 24, 2014 8:50 PM
>>To: Rakesh Gandhi (rgandhi); Nobo Akiya (nobo);
>>draft-tsaad-ccamp-rsvpte-bidir-lsp-fastreroute@tools.ietf.org
>>Cc: TEAS WG
>>Subject: Re: [Teas] Mail regarding
>>draft-tsaad-ccamp-rsvpte-bidir-lsp-fastreroute
>>
>>Working Group,
>>
>>I was almost through my review of this document, saw Nobo's review and
>>found that he had found everything I found and a bit more :).
>>
>>With that and with Rakesh's statement that the comments will be
>>addressed I souppor the adoption of the document as a wg doc.
>>
>>/Loa
>>
>>On 2014-12-25 07:25, Rakesh Gandhi (rgandhi) wrote:
>>> Thank you Nobo for your detailed review comments.
>>>
>>> We will address these in the next revision.
>>>
>>> Happy holidays.
>>>
>>> Thanks,
>>> Rakesh
>>>
>>>
>>> From: "Nobo Akiya (nobo)" <nobo@cisco.com <mailto:nobo@cisco.com>>
>>> Date: Wednesday, 24 December, 2014 5:45 PM
>>> To: "draft-tsaad-ccamp-rsvpte-bidir-lsp-fastreroute@tools.ietf.org
>>> <mailto:draft-tsaad-ccamp-rsvpte-bidir-lsp-fastreroute@tools.ietf.org>"
>>> <draft-tsaad-ccamp-rsvpte-bidir-lsp-fastreroute@tools.ietf.org
>>> <mailto:draft-tsaad-ccamp-rsvpte-bidir-lsp-fastreroute@tools.ietf.org
>>> >
>>> >
>>> Cc: TEAS WG <teas@ietf.org <mailto:teas@ietf.org>>
>>> Subject: Mail regarding
>>>draft-tsaad-ccamp-rsvpte-bidir-lsp-fastreroute
>>> Resent-From: <draft-alias-bounces@tools.ietf.org
>>> <mailto:draft-alias-bounces@tools.ietf.org>>
>>> Resent-To: "frederic.jounay@orange.ch
>>><mailto:frederic.jounay@orange.ch>" <frederic.jounay@orange.ch
>>><mailto:frederic.jounay@orange.ch>>, "Lizhong com>"
>>> <lizho.jin@gmail.com <mailto:lizho.jin@gmail.com>>,
>>><manav@ionosnetworks.com <mailto:manav@ionosnetworks.com>>,
>>>"=SMTP:mtaillon@cisco. com"
>>> <mtaillon@cisco.com <mailto:mtaillon@cisco.com>>, Rakesh Gandhi
>>><rgandhi@cisco.com <mailto:rgandhi@cisco.com>>, <tsaad@cisco.com
>>><mailto:tsaad@cisco.com>>, Zafar Ali <zali@cisco.com
>>><mailto:zali@cisco.com>>
>>> Resent-Date: Wednesday, 24 December, 2014 5:46 PM
>>>
>>> Hi Authors,
>>>
>>> Happy Holidays!
>>>
>>> I read draft-tsaad-ccamp-rsvpte-bidir-lsp-fastreroute-05 (as part of
>>>its WG adoption poll), and had some questions/comments. Please find
>>>them below.
>>>
>>> 1) Section 4.4.1 - The rules for the BYPASS_ASSIGNMENT subobject may
>>> be easier to understand if "list" is used.
>>>
>>> [OLD]
>>>
>>>     The BYPASS_ASSIGNMENT subobject is added in the RECORD_ROUTE
>>> object
>>>
>>>     prior to adding the node's IP address in the node-ID subobject.
>>> A
>>>
>>>     node MUST NOT add a BYPASS_ASSIGNMENT subobject without also
>>> adding a
>>>
>>>     Node-ID subobject.  A node MUST NOT add a BYPASS_ASSIGNMENT
>>> subobject
>>>
>>>     without also adding an IPv4 or IPv6 subobject.
>>>
>>> [NEW]
>>>
>>>     When the BYPASS_ASSIGNMENT subobject is added in the RECORD_ROUTE
>>>
>>>     object:
>>>
>>>     o  The BYPASS_ASSIGNMENT subobject MUST be added prior to the
>>> node-ID
>>>
>>>        subobject describing the node's IP address.
>>>
>>>     o  The Node-ID subobject MUST also be added.
>>>
>>>     o  The IPv4 or IPv6 subobject MUST also be added.
>>>
>>> 2) Section 4.4.1 - First sentence of the third paragraph was quite
>>> difficult to read/understand. Perhaps adding some commas and
>>> quotations may improve its readability.
>>>
>>> [OLD]
>>>
>>>     The upstream PLR (downstream MP) that detects a BYPASS_ASSIGNMENT
>>>
>>>     subobject whose bypass tunnel and the node-ID subobject when used
>>> as
>>>
>>>     a bypass tunnel source terminates locally assigns the matching
>>>
>>>     bidirectional bypass tunnel in the reverse direction, and
>>> forwards
>>>
>>>     the RSVP Path message downstream.
>>>
>>> [NEW]
>>>
>>>     The upstream PLR (downstream MP) that detects a BYPASS_ASSIGNMENT
>>>
>>>     subobject, whose bypass tunnel and the node-ID subobject when
>>> used as
>>>
>>>     a "bypass tunnel source" terminates locally, assigns the matching
>>>
>>>     bidirectional bypass tunnel in the reverse direction, and
>>> forwards
>>>
>>>     the RSVP Path message downstream.
>>>
>>> 3) Section 4.4.1 - I have a slight concern regarding below statement.
>>>
>>> [snip]
>>>
>>>     In the absence of BYPASS_ASSIGNMENT subobject, the upstream PLR
>>> does
>>>
>>>     not assign a bypass tunnel in the reverse direction.  This allows
>>> the
>>>
>>>     downstream PLR to always initiate the bypass assignment and
>>> upstream
>>>
>>>     PLR to simply reflect the bypass assignment.
>>>
>>>     In the case of upstream PLR receiving multiple BYPASS_ASSIGNMENT
>>>
>>>     subobjects from multiple downstream PLRs, the decision of
>>> selecting a
>>>
>>>     bypass tunnel in the reverse direction can be based on local
>>> policy,
>>>
>>>     for example, prefer link protection versus node protection bypass
>>>
>>>     tunnel, or prefer the most upstream versus least upstream node
>>>
>>>     protection bypass tunnel.
>>>
>>> [snip]
>>>
>>> When LSRs within a bidirectional LSP employ different policies, as
>>> described above, then we can result in pockets where there are no
>>> protections.
>>>
>>> For example:
>>>
>>> Node A uses node protection.
>>>
>>> Node B and C use link protection.
>>>
>>>      +---------+
>>>
>>>     /       +-+ \
>>>
>>>    /       /   \ \
>>>
>>> /       /     v v
>>>
>>> A ----- B ----- C
>>>
>>>           ^     /
>>>
>>>            \   /
>>>
>>>             +-+
>>>
>>> The result of above is that, in the reverse direction, there will not
>>> be any protection between nodes B and A.
>>>
>>> Another example:
>>>
>>> Node A and C use node protection.
>>>
>>> Node B use link protection.
>>>
>>>      +---------+
>>>
>>>     /       +-+ \
>>>
>>>    /       /   \ \
>>>
>>> /       /     v v
>>>
>>> A ----- B ----- C
>>>
>>> ^              /
>>>
>>>    \            /
>>>
>>>     +----------+
>>>
>>> The result of above is that, in the reverse direction, there will not
>>> be any protection between nodes B and A.
>>>
>>> In both cases, if there is a breakage of the link connecting nodes A
>>> and B, then node C receives the RSVP PATH message. If node C follows
>>> procedures described in Section 6.2, then it will not find a matching
>>> bypass tunnel.
>>>
>>> If I'm off track, then I apologize for the noise. If above is indeed
>>> a problem, then this limitation should at least be described in the
>>> document, followed by a recommendation of all LSRs within a
>>> bidirectional tunnel employing the same policy.
>>>
>>> It may be possible to (be brave and) address this but it will require
>>> more procedures. I'll leave the decision and further investigation to
>>> the authors.
>>>
>>> 4) Section 4.4.1 - Should there be SHOULD or MUST in this paragraph?
>>>
>>> [snip]
>>>
>>>     In the absence of BYPASS_ASSIGNMENT subobject, the upstream PLR
>>> does
>>>
>>>     not assign a bypass tunnel in the reverse direction.  This allows
>>> the
>>>
>>>     downstream PLR to always initiate the bypass assignment and
>>> upstream
>>>
>>>     PLR to simply reflect the bypass assignment.
>>>
>>> [snip]
>>>
>>> I'm guessing that there should be a SHOULD ...
>>>
>>> 5) Section 6.2 - "If" and "If not" in below text do not correspond to
>>> each other (thus a bit confusing).
>>>
>>> [snip]
>>>
>>>        - If found, checks whether the primary LSP traffic and
>>> signaling
>>>
>>>          are already rerouted over the found bypass tunnel.  If not,
>>> PRR
>>>
>>>          R5 activates FRR reroute procedures to direct traffic and
>>>
>>>          RSVP Resv over the found bypass tunnel T2 in the
>>>
>>>          reverse direction.
>>>
>>> [snip]
>>>
>>> If we write above with a psuedocode, then it will be:
>>>
>>>      if (bypass tunnel found) {      // "If found" is this line
>>>
>>>          check if already rerouted
>>>
>>>          if (not already rerouted) { // "If not" is this line
>>>
>>>               active FRR reroute
>>>
>>>          }
>>>
>>>      }
>>>
>>> It  might make the texts easier to read/understand if you can
>>> rephrase it a bit?
>>>
>>> Thanks!
>>>
>>> -Nobo
>>>
>>>
>>>
>>> _______________________________________________
>>> Teas mailing list
>>> Teas@ietf.org
>>> https://www.ietf.org/mailman/listinfo/teas
>>>
>>
>>--
>>
>>
>>Loa Andersson                        email: loa@mail01.huawei.com
>>Senior MPLS Expert                          loa@pi.nu
>>Huawei Technologies (consultant)     phone: +46 739 81 21 64
>>
>>_______________________________________________
>>Teas mailing list
>>Teas@ietf.org
>>https://www.ietf.org/mailman/listinfo/teas
>