Re: [Gen-art] Gen-ART LC Review of draft-ietf-pwe3-static-pw-status-10

Stewart Bryant <stbryant@cisco.com> Mon, 30 April 2012 17:26 UTC

Return-Path: <stbryant@cisco.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 34FCD21F86DB; Mon, 30 Apr 2012 10:26:03 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -110.483
X-Spam-Level:
X-Spam-Status: No, score=-110.483 tagged_above=-999 required=5 tests=[AWL=0.116, BAYES_00=-2.599, RCVD_IN_DNSWL_HI=-8, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id EeNv-2wzwxea; Mon, 30 Apr 2012 10:26:02 -0700 (PDT)
Received: from ams-iport-1.cisco.com (ams-iport-1.cisco.com [144.254.224.140]) by ietfa.amsl.com (Postfix) with ESMTP id E836021F8657; Mon, 30 Apr 2012 10:26:01 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=stbryant@cisco.com; l=1840; q=dns/txt; s=iport; t=1335806762; x=1337016362; h=message-id:date:from:reply-to:mime-version:to:cc:subject: references:in-reply-to:content-transfer-encoding; bh=baKfUvMNr17/OaV1zcWr69S32z51WzU4pWoHI/KqzIg=; b=XihZk4h+C69BOITTQ4YIIwUJ6YXdSDFJyfqkTlZPrjUkzmKndyVcLE8s MtM9GE4jX+yDlzzM+Gaq50m8g5GZ1J21C3XqrAvy8U/gqn61/uqviek86 tv6NURZbMnDICxlDz4lIpMHbDuXI3EaxyClQHyk+keZX6QipCByYhgkAJ A=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: AgAFAPXKnk+Q/khM/2dsb2JhbABEr0+DAIEHggkBAQEEEgECI0ABEAsUBAkWDwkDAgECAUUGDQEHAQEeh2sLmimDQhCcKpEiBJV+gRGNSIECZ4Jp
X-IronPort-AV: E=Sophos;i="4.75,505,1330905600"; d="scan'208";a="136639932"
Received: from ams-core-3.cisco.com ([144.254.72.76]) by ams-iport-1.cisco.com with ESMTP; 30 Apr 2012 17:25:42 +0000
Received: from cisco.com (mrwint.cisco.com [64.103.70.36]) by ams-core-3.cisco.com (8.14.3/8.14.3) with ESMTP id q3UHPgps021791; Mon, 30 Apr 2012 17:25:42 GMT
Received: from dhcp-128-107-166-59.cisco.com (localhost [127.0.0.1]) by cisco.com (8.14.4+Sun/8.8.8) with ESMTP id q3UHPddg007627; Mon, 30 Apr 2012 18:25:41 +0100 (BST)
Message-ID: <4F9ECB13.3040001@cisco.com>
Date: Mon, 30 Apr 2012 18:25:39 +0100
From: Stewart Bryant <stbryant@cisco.com>
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:11.0) Gecko/20120327 Thunderbird/11.0.1
MIME-Version: 1.0
To: Ben Campbell <ben@nostrum.com>
References: <ED4FC298-44C1-409D-9D0A-01CA75E14059@nostrum.com>
In-Reply-To: <ED4FC298-44C1-409D-9D0A-01CA75E14059@nostrum.com>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
Cc: "gen-art@ietf.org Review Team" <gen-art@ietf.org>, draft-ietf-pwe3-static-pw-status.all@tools.ietf.org, The IETF <ietf@ietf.org>
Subject: Re: [Gen-art] Gen-ART LC Review of draft-ietf-pwe3-static-pw-status-10
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
Reply-To: stbryant@cisco.com
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: <http://www.ietf.org/mail-archive/web/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: Mon, 30 Apr 2012 17:26:03 -0000

On 26/04/2012 23:55, Ben Campbell wrote:
> I am the assigned Gen-ART reviewer for this draft. For background on
> Gen-ART, please see the FAQ at
> <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
>
> Please resolve these comments along with any other Last Call comments
> you may receive.
>
> Document:  (Proposed RFC 6478) (was draft-ietf-pwe3-static-pw-status-10)
> Reviewer: Ben Campbell
> Review Date:  2012-04-26
> IETF LC End Date:  2012-04-30
>
> **** Note: This draft has previously been approved as RFC 6478, but I understand we are last calling it again due to some material changes in AUTH48. Therefore this is a review of the diff between revision 10 and the text at http://www.rfc-editor.org/authors/rfc6478.txt ****
>
>
> Summary:
>
> The edited text is essentially ready for publication, but I have a couple of minor issues that might should be considered first.
>
> Major issues:
>
> None
>
> Minor issues:
>
> -- 5.3, last paragraph:
>
> The last sentence changed from a non-normative statement to "This SHOULD cause the PE sending the PW status notification message with a PW status code equal to zero to stop sending and to continue normal operation."
>
> Is that really intended as a normative statement, or a statement of fact? I suspect it's the latter, but if the former, then it should be stated more of the form "If the sending PE receives ... it SHOULD stop ..."
>
> -- IANA considerations:
>
> Maybe I missed it, but I don't see a registration policy for adding things to the new registry. This wasn't an AUTH48 change, but it should probably be there.
Hi Ben

Thank you for your review.

The IANA policy is stated as IETF Review (end of first para in IANA)

The normative text is deliberate - this was part of the change that we 
needed to make.

- Stewart