RE: AD review for draft-ietf-rtgwg-multihomed-prefix-lfa-06

Shraddha Hegde <shraddha@juniper.net> Wed, 19 September 2018 08:50 UTC

Return-Path: <shraddha@juniper.net>
X-Original-To: rtgwg@ietfa.amsl.com
Delivered-To: rtgwg@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 98CAB130DDE; Wed, 19 Sep 2018 01:50:31 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.71
X-Spam-Level:
X-Spam-Status: No, score=-2.71 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_LOW=-0.7, SPF_PASS=-0.001, T_DKIMWL_WL_HIGH=-0.01, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=juniper.net
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 sc3ftLKGlQIc; Wed, 19 Sep 2018 01:50:29 -0700 (PDT)
Received: from mx0a-00273201.pphosted.com (mx0a-00273201.pphosted.com [208.84.65.16]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 992CF130E58; Wed, 19 Sep 2018 01:50:29 -0700 (PDT)
Received: from pps.filterd (m0108159.ppops.net [127.0.0.1]) by mx0a-00273201.pphosted.com (8.16.0.22/8.16.0.22) with SMTP id w8J8nP8K019642; Wed, 19 Sep 2018 01:50:28 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=juniper.net; h=from : to : cc : subject : date : message-id : references : in-reply-to : content-type : content-transfer-encoding : mime-version; s=PPS1017; bh=IcRUDplWU1rX8HuUSI4lis4CYhac6StggKKIDoNB+BI=; b=m/MZNynqrN04/Xgu3Zyl/WdSN/sqQE7MlkIh6HNhsM6Vj8dv6t+ehlrmrwRYkhVp9VwF s/bM3UlO45X/5nbJHjF8aHyaXv0aVbDP05pyxC8hLE07TYdbvw4t9OWv5PID1BqlOa7S Xz8cIhbruRbXgYT1S1qX7aWGdXRc7cMLASzcGyh0vmZPEb/CXEOGc6+NsM7lK4nFXidX uIUV6gSeAW89m6UkrTtvvuA8OLiAgmEdiFCS5uAXCqxCmyRLKw1foKadjgucpa/AP0H2 lblS7ODgtOtyIXEjFO1SC94mlAigirJynyF513d/vvOAXME5U+YfNSivjfWkVh0r242U 7A==
Received: from nam05-dm3-obe.outbound.protection.outlook.com (mail-dm3nam05lp0117.outbound.protection.outlook.com [216.32.181.117]) by mx0a-00273201.pphosted.com with ESMTP id 2mkf95rcmt-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT); Wed, 19 Sep 2018 01:50:27 -0700
Received: from BYAPR05MB3943.namprd05.prod.outlook.com (52.135.195.146) by BYAPR05MB4648.namprd05.prod.outlook.com (52.135.233.74) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1164.12; Wed, 19 Sep 2018 08:50:24 +0000
Received: from BYAPR05MB3943.namprd05.prod.outlook.com ([fe80::a8b9:8235:ecf7:76f3]) by BYAPR05MB3943.namprd05.prod.outlook.com ([fe80::a8b9:8235:ecf7:76f3%6]) with mapi id 15.20.1164.017; Wed, 19 Sep 2018 08:50:24 +0000
From: Shraddha Hegde <shraddha@juniper.net>
To: Shraddha Hegde <shraddha@juniper.net>, Pushpasis Sarkar <pushpasis.ietf@gmail.com>, "Vigoureux, Martin (Nokia - FR)" <martin.vigoureux@nokia.com>
CC: "draft-ietf-rtgwg-multihomed-prefix-lfa@ietf.org" <draft-ietf-rtgwg-multihomed-prefix-lfa@ietf.org>, rtgwg-chairs <rtgwg-chairs@ietf.org>, Stewart Bryant <stewart.bryant@gmail.com>, RTGWG <rtgwg@ietf.org>
Subject: RE: AD review for draft-ietf-rtgwg-multihomed-prefix-lfa-06
Thread-Topic: AD review for draft-ietf-rtgwg-multihomed-prefix-lfa-06
Thread-Index: AQHTxhbah9G6prsDZEa8qmJ4CxL69qRny+KAgIYKvYCAAYCkAIAAK7DAgAjcG0A=
Date: Wed, 19 Sep 2018 08:50:23 +0000
Message-ID: <BYAPR05MB39439CEB3E275F8F903B8ED8D51C0@BYAPR05MB3943.namprd05.prod.outlook.com>
References: <2ccba50e-2032-bad6-b91d-cb583bd8cac6@nokia.com> <CAEFuwkixedEdM9t4iBytMBd=xNPmmuHxh=F8hmsGZ3Th12+BzA@mail.gmail.com> <0989548c-7dd3-712b-2f24-7ee2e2c3827c@nokia.com> <CAEFuwkiNgGLGP+UBVxVKJ0OYnCfbkO+7N_phJ=_CPSrF5e56HA@mail.gmail.com> <BYAPR05MB3943D6EFCA2CA393352CE2A3D51A0@BYAPR05MB3943.namprd05.prod.outlook.com>
In-Reply-To: <BYAPR05MB3943D6EFCA2CA393352CE2A3D51A0@BYAPR05MB3943.namprd05.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
dlp-product: dlpe-windows
dlp-version: 11.0.400.15
dlp-reaction: no-action
x-originating-ip: [116.197.184.14]
x-ms-publictraffictype: Email
x-microsoft-exchange-diagnostics: 1; BYAPR05MB4648; 6:Z/1R32MixC85EJskLrSiRBDW+6BjDGct+lg4JJmFC/ZvuGWJYxK9RQFaCKUWY/zmxfz80FOnUQjg09j9smACBCLapVlv5Dp1cIRkQWU0TTE2nrkKJqJfZaPe0l074+WstSaiPROAy0fUcoNFzSCcUzZ6yzVCaSf5ysFJDjWJG0o/aTYHp2DGTlDxemaVZdXjdJmV8RiqNFdUHQDWX/mDLO8NsmNDQVVOZ14GV0yP7QEarZepXx/YFRcIMNB3VPsC71j3lqZkUWy5Nh+9IXZPpo+K4HEjkilwBo96GFBY1CVxYPZWXLHQNCmrKmicwF+sj21BwdfxOPqLIFK4n5qcs+S++kU9aAuox5TSBs1wEeLoK9ll2xndzOZ6hNPAiadwxDhTybhLGjV7dsNBks2faIRi8OfTEZ6EZj0BRIbwFAC5POO5UBD3RXZElGgzxL3YfKRwpGRkff0jpUwVNXi2ww==; 5:mMFLLZm36PoQLyldWHJLAAgeGNyIzOxl1O7XwFP1TP+bXpadY0hqXbTBtWbuaF9MHEp73kgHbHt7R4PbgUR28rfGfuVEOflHauvuaomZP4lJqVrJx76Hl+Ux5DftEX5tALbZqWJuwwUJiuqvd7zkEEaA7Tl1e4I6YHKl0NpRKoU=; 7:7LKMBAsOGz1HlXRsppgpLoIbTOLTLLrDCWwnwd3z2uBu4nMaLwx2M6rjqEPDrkYfkB6HpjWKPB81eZXY30jlvsxK2ptQv4TRr9B/kVRITc9j+cxXuB09BWr/L3KFh0g9j00PcvdrujCauRjRoWpBqfKc5dRo5FVyMq+FzJjHXm6uTMzD1RupPEsIxT4qaFTfvT55HmA3GCfzSVHyDejePXoNZGbC0UVuaS7Fn3V6i6/azANKzBnkctI0TH79+txh
x-ms-exchange-antispam-srfa-diagnostics: SOS;SOR;
x-forefront-antispam-report: SFV:SKI; SCL:-1; SFV:NSPM; SFS:(10019020)(136003)(396003)(376002)(366004)(346002)(39860400002)(51914003)(13464003)(199004)(189003)(26005)(5250100002)(6346003)(68736007)(106356001)(54906003)(8936002)(1941001)(105586002)(4326008)(110136005)(316002)(76176011)(99286004)(93886005)(6506007)(53546011)(39060400002)(2906002)(102836004)(97736004)(25786009)(7696005)(478600001)(2900100001)(74316002)(55016002)(256004)(14444005)(5024004)(7736002)(86362001)(6436002)(305945005)(6116002)(3846002)(5660300001)(33656002)(229853002)(14454004)(8676002)(81156014)(476003)(6246003)(486006)(11346002)(446003)(9686003)(66066001)(53936002)(81166006); DIR:OUT; SFP:1102; SCL:1; SRVR:BYAPR05MB4648; H:BYAPR05MB3943.namprd05.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1;
x-ms-office365-filtering-correlation-id: d3d3563b-90ca-4b9c-9159-08d61e0cf080
x-ms-office365-filtering-ht: Tenant
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989299)(5600074)(711020)(4618075)(2017052603328)(7153060)(7193020); SRVR:BYAPR05MB4648;
x-ms-traffictypediagnostic: BYAPR05MB4648:
x-microsoft-antispam-prvs: <BYAPR05MB464857041E8853BF2EF7CC9BD51C0@BYAPR05MB4648.namprd05.prod.outlook.com>
x-exchange-antispam-report-test: UriScan:(138986009662008)(85827821059158)(109105607167333)(82608151540597)(195916259791689);
x-ms-exchange-senderadcheck: 1
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(8121501046)(5005006)(3231355)(944501410)(52105095)(3002001)(10201501046)(93006095)(93001095)(6055026)(149027)(150027)(6041310)(20161123560045)(20161123558120)(20161123562045)(20161123564045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(201708071742011)(7699050)(76991041); SRVR:BYAPR05MB4648; BCL:0; PCL:0; RULEID:; SRVR:BYAPR05MB4648;
x-forefront-prvs: 0800C0C167
received-spf: None (protection.outlook.com: juniper.net does not designate permitted sender hosts)
x-microsoft-antispam-message-info: QM/PFFDYUc3QFo/8Vi3oi028Fia7mdDrIA76s+S7oorzOIH48KwVD12jJYfbgOxFoRjfcw3kDF6Hi1e3sl8px1McGlvqNJi2u0wszTJtMRHbqbo2M85XOnx0UTAoefFZ06xS6tPD0AaMzZBn5xNw9nog1ujArNCt3p0wf6eU2bgD/gEoZLN2/8Cu7I1vBKSncXeNwKy4mRkr+dAWzmT2C8rbO7DmwpObYVto7VCKwl0cGlsIC1lyD+EGSshXH8uZLyTIjWEZcGcslDUhlyNL9AX8CxjKqxAYzFmmZBYuE/KKpw9nLiRkyWZ9cOF8NYHkwS5PlnZDNLvWTnN2azkAX1JcDG9O4UWXP56a8cBZLPo=
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: juniper.net
X-MS-Exchange-CrossTenant-Network-Message-Id: d3d3563b-90ca-4b9c-9159-08d61e0cf080
X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Sep 2018 08:50:24.1391 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: bea78b3c-4cdb-4130-854a-1d193232e5f4
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR05MB4648
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:, , definitions=2018-09-19_03:, , signatures=0
X-Proofpoint-Spam-Details: rule=outbound_spam_notspam policy=outbound_spam score=0 priorityscore=1501 malwarescore=0 suspectscore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1015 lowpriorityscore=0 mlxscore=0 impostorscore=0 mlxlogscore=999 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1807170000 definitions=main-1809190092
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtgwg/pL24u44o9dvSsCR2JsVc7DLiJp8>
X-BeenThere: rtgwg@ietf.org
X-Mailman-Version: 2.1.29
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: <https://mailarchive.ietf.org/arch/browse/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: Wed, 19 Sep 2018 08:50:32 -0000

Martin,

I have fixed the typos and also added more description to clarify a few items.

Pls see details

Section 4.2 (and subsections) is (are) a bit difficult to 
read/understand because of the typos but also because of the way it's 
written.

Section 4.2.1.
Do you mean ECMP FRR rather than simply ECMP (as section 4.2.3. seems to 
suggest)?
If so, please take this into account while addressing typos listed below.
[Pushpasis] 4.2.1 is for ECMP. ECMPs do not count as alternates. These rules cover all scenarios but related to alternates only. 
But, I will let Shraddha confirm that.. 
<shraddha> Fixed the text to read "ECMP FRR"

Sections 4.2.2., 4.2.3., and 4.2.5, seem to be linked to 4.2.1.. 
Wouldn't it be better to switch 4.2.4. and 4.2.5.? Alternatively can't 
these three sections in fact be subsections of 4.2.1 ?
[Pushpasis] Shraddha can you take look and let us know if that is okay. 
<shraddha> Thanks for the suggestion. Chaged the paraphasing sections to subsections of 4.2.1

Although Sections 4.2.2., 4.2.3., and 4.2.5 seem to paraphrase 4.2.1., I 
read one sentence which does not appear in the pseudo algorithm:
    If there are two ASBRs with different type2 cost, the higher cost
    ASBR is pruned.
So I am not sure to understand when this condition/action comes into 
play. Could you clarify?
[Pushpasis] Again will let Shraddha comment on it.
<Shraddha> Added more description to "type 1 type 2 cost" section.
Pls check if it looks better. 

Section 4.2.4
It is not clear which inequalities will apply in that case.
<Shraddha> Added statement to clarify the inequalities to be used.

In the same way as above, Section 4.2.5 seems to say a more than Step 5 
of the pseudo algorithm. Could you clarify when the extra conditions it 
describes come into play? Or said differently, shouldn't step 5 be 
reworked to be more complete? If you do so, please rework that step 
incorporating the types of changes/rephrasing I have suggested for the 
other steps (see typos below).
<Shraddha> Updated step 5. pls check.


    1a. if primary ASBR and alternate ASBR are intra area
        non-backbone path go to step 2.
do you mean "belong to" rather than "are"?
<shraddha>if primary ASBR and alternate ASBR are intra area
                  and belong to non-backbone go to step 2.

   2. If cost type (type1/type2) advertised by alternate
      ASBR same as primary
Do you mean:
   2. Compare cost types (type1/type2) advertised by alternate ASBR and
      by the primary ASBR
[Pushpasis] Shraddha, please take  a look. 
<shraddha> Fixed


   3. If cost type is type1
             3a. If cost is same, program ECMP and return.
             3b. else go to step 5.
Do you mean:
   3. If cost types are type1, compare costs advertised by alternate ASBR
      and by the primary ASBR
             3a. If costs are the same then program ECMP and return.
             3b. else go to step 5.
[Pushpasis] Shraddha, please take  a look. 
<Shraddha> Fixed


   4  If cost type is type 2
             4a. If cost is different, skip alternate ASBR and
                     consider next ASBR.
             4b. If type2 cost is same, proceed to step 4c to compare
                     compare type 1 cost.
             4c. If type1 cost is also same program ECMP and return.
             4d. If type 1 cost is different go to step 5.
Do you mean:
   4  If cost types are type2, compare costs advertised by alternate ASBR
      and by the primary ASBR
             4a. If costs are different, skip alternate ASBR and
                     consider next ASBR.
             4b. If cost are the same, proceed to step 4c to compare
                     compare type1 costs.
             4c. If type1 costs are also same program ECMP and return.
             4d. If type1 costs are different go to step 5.
[Pushpasis] Shraddha, please take  a look. 
<shraddha> Fixed

    While selecting alternate ASBR for loop evaluation for LFA, these
    rules should be applied and ensured that the alternate neighbor does
    not loop the traffic back.
I'm not sure about the meaning of the latter part of that sentence ("and 
ensured ...")
[Pushpasis] Shraddha, please take  a look. I think it means.. "... these rules should be applied and to ensure that the alternate neighbor does not loop ..." 
<shraddha> fixed the text




I have posted -07 version.Pls take a look and let me know if you have further comments.

Rgds
Shraddha

-----Original Message-----
From: Shraddha Hegde <shraddha@juniper.net> 
Sent: Thursday, September 13, 2018 11:02 PM
To: Pushpasis Sarkar <pushpasis.ietf@gmail.com>om>; Vigoureux, Martin (Nokia - FR) <martin.vigoureux@nokia.com>
Cc: draft-ietf-rtgwg-multihomed-prefix-lfa@ietf.org; rtgwg-chairs <rtgwg-chairs@ietf.org>rg>; Stewart Bryant <stewart.bryant@gmail.com>om>; RTGWG <rtgwg@ietf.org>
Subject: RE: AD review for draft-ietf-rtgwg-multihomed-prefix-lfa-06

Martin,

Apologise missing the e-mail from Pushpasis.
I'll look at the comments and update the draft in a day or two.

Rgds
Shraddha

-----Original Message-----
From: Pushpasis Sarkar <pushpasis.ietf@gmail.com>
Sent: Thursday, September 13, 2018 8:25 PM
To: Vigoureux, Martin (Nokia - FR) <martin.vigoureux@nokia.com>
Cc: draft-ietf-rtgwg-multihomed-prefix-lfa@ietf.org; rtgwg-chairs <rtgwg-chairs@ietf.org>rg>; Stewart Bryant <stewart.bryant@gmail.com>om>; RTGWG <rtgwg@ietf.org>
Subject: Re: AD review for draft-ietf-rtgwg-multihomed-prefix-lfa-06

Hi Martin,

I am extremely sorry. I forgot to follow up with co-authors on this. Buried with day job. I will try to close it with co-authors at the earliest possible.

Best regards,
-Pushpasis

On Wed, Sep 12, 2018 at 9:33 PM Martin Vigoureux <martin.vigoureux@nokia.com>
wrote:

> Authors,
>
> am I right in thinking that the ball is still in your camp and you 
> need Shraddha to look at some of the comments?
> The draft has been in Revised I-D Needed for 5 months now.
>
> -m
>
> Le 2018-06-19 à 11:00, Pushpasis Sarkar a écrit :
> > Hi Martin,
> >
> > Once again sorry for the delay. Please find answers to some of your 
> > points inline.
> >
> > Hi Shraddha,
> >
> > Please find attached the XML draft for the next revision with 
> > changes taken care by Uma and myself. Please add your changes and 
> > reply back on the comments on OSPF sections that I have requested 
> > you to take look at