Re: [Gen-art] Gen-ART Last Call review of draft-ietf-idr-deprecate-30-31-129-00

Job Snijders <job@ntt.net> Sat, 17 December 2016 12:10 UTC

Return-Path: <job@ntt.net>
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 486BE129A32 for <gen-art@ietfa.amsl.com>; Sat, 17 Dec 2016 04:10:16 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.828
X-Spam-Level:
X-Spam-Status: No, score=-2.828 tagged_above=-999 required=5 tests=[RCVD_IN_DNSWL_LOW=-0.7, RP_MATCHES_RCVD=-3.1, SPF_SOFTFAIL=0.972] autolearn=ham autolearn_force=no
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 icCntmLIwsqQ for <gen-art@ietfa.amsl.com>; Sat, 17 Dec 2016 04:10:15 -0800 (PST)
Received: from mail3.mlpsca01.us.to.gin.ntt.net (mail3.mlpsca01.us.to.gin.ntt.net [IPv6:2001:418:3ff:3::22]) (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 007B01296BB for <gen-art@ietf.org>; Sat, 17 Dec 2016 04:10:14 -0800 (PST)
Received: by mail3.mlpsca01.us.to.gin.ntt.net with esmtpsa (TLSv1.2:DHE-RSA-AES256-GCM-SHA384:256) (Exim 4.84_2) (envelope-from <job@ntt.net>) id 1cIDoZ-000DvN-KA (job@us.ntt.net); Sat, 17 Dec 2016 12:10:12 +0000
Date: Sat, 17 Dec 2016 13:10:05 +0100
From: Job Snijders <job@ntt.net>
To: Meral Shirazipour <meral.shirazipour@ericsson.com>
Message-ID: <20161217121005.GL34914@Vurt.local>
References: <ABCAA4EF18F17B4FB619EA93DEF7939A4E828D6C@eusaamb107.ericsson.se>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <ABCAA4EF18F17B4FB619EA93DEF7939A4E828D6C@eusaamb107.ericsson.se>
X-Clacks-Overhead: GNU Terry Pratchett
User-Agent: Mutt/1.7.1 (2016-10-04)
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/GqPuf5jThtS_rCPpS_Kmgkqr2NU>
Cc: "draft-ietf-idr-deprecate-30-31-129.all@tools.ietf.org" <draft-ietf-idr-deprecate-30-31-129.all@tools.ietf.org>, "gen-art@ietf.org" <gen-art@ietf.org>
Subject: Re: [Gen-art] Gen-ART Last Call review of draft-ietf-idr-deprecate-30-31-129-00
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: Sat, 17 Dec 2016 12:10:16 -0000

Dear Meral,

A new version of the draft has been published to address your comments.
Please review the diff here:

    https://www.ietf.org/rfcdiff?url2=draft-ietf-idr-deprecate-30-31-129-01

On Fri, Dec 16, 2016 at 12:35:40AM +0000, Meral Shirazipour wrote:
> Summary:
> This draft is ready to be published as Standards Track RFC but I have some comments.
> 
> Major issues:
> 
> Minor issues:
> -[Page 2]:
> "This has led to  deployment problems for new technologies such as
> Large BGP Communities [I-D.ietf-idr-large-community]."
> 
> Not sure if draft-ietf-idr-large-community explains the issues caused
> by the use of these attribute values ? If yes it would be good to
> point to that section.  If not it would be good to add short summary
> here.

A short summary has been added.

> Nits/editorial comments:
> -[Page 2], Section 4 please refer to latest version of
> [I-D.ietf-idr-large-community].

I mistakenly linked to -08 (while -11 is the latest version), this will
be corrected before publication.

> -[Page 2]:
> "The squatting of values 30, 31, 129, 241, 242 and 243 has been
> confirmed by the involved vendors or through source code review."
> 
> A bit confusing, somehow it seems 30 is already deprecated as per:
> https://www.ietf.org/assignments/bgp-parameters/bgp-parameters.xml
> with reference to [draft-ietf-idr-large-community].
> The switch between used of 30 to  use of 32 happened between v04 and
> v05 of [draft-ietf-idr-large-community]?
> Perhaps https://www.ietf.org/assignments/bgp-parameters/bgp-parameters.xml
> should be updated to point to this draft instead?

Yes, I expect IANA to remove the reference to Large Communities behind
30 (because Large no longer has anything to do with value 30, and indeed
we want to prevent confusion), and reference this document, if approved
for publication.

Kind regards,

Job