Re: [Gen-art] Gen-art LC review: draft-ietf-pals-redundancy-spe-02

"BRUNGARD, DEBORAH A" <db3546@att.com> Sat, 17 October 2015 16:12 UTC

Return-Path: <db3546@att.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id EA7111B29B5; Sat, 17 Oct 2015 09:12:18 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.265
X-Spam-Level:
X-Spam-Status: No, score=-2.265 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, IP_NOT_FRIENDLY=0.334, RCVD_IN_DNSWL_LOW=-0.7] 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 DNgcKogeQdq4; Sat, 17 Oct 2015 09:12:16 -0700 (PDT)
Received: from mx0a-00191d01.pphosted.com (mx0a-00191d01.pphosted.com [67.231.149.140]) (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 7701F1B29B1; Sat, 17 Oct 2015 09:12:16 -0700 (PDT)
Received: from pps.filterd (m0053301.ppops.net [127.0.0.1]) by mx0a-00191d01.pphosted.com (8.15.0.59/8.15.0.59) with SMTP id t9HG9D7G027368; Sat, 17 Oct 2015 12:12:15 -0400
Received: from alpi155.enaf.aldc.att.com (sbcsmtp7.sbc.com [144.160.229.24]) by mx0a-00191d01.pphosted.com with ESMTP id 1xkgyp583r-1 (version=TLSv1/SSLv3 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Sat, 17 Oct 2015 12:12:14 -0400
Received: from enaf.aldc.att.com (localhost [127.0.0.1]) by alpi155.enaf.aldc.att.com (8.14.5/8.14.5) with ESMTP id t9HGCDLR004807; Sat, 17 Oct 2015 12:12:13 -0400
Received: from mlpi409.sfdc.sbc.com (mlpi409.sfdc.sbc.com [130.9.128.241]) by alpi155.enaf.aldc.att.com (8.14.5/8.14.5) with ESMTP id t9HGC4lR004763 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO); Sat, 17 Oct 2015 12:12:07 -0400
Received: from MISOUT7MSGHUBAC.ITServices.sbc.com (MISOUT7MSGHUBAC.itservices.sbc.com [130.9.129.147]) by mlpi409.sfdc.sbc.com (RSA Interceptor); Sat, 17 Oct 2015 16:11:49 GMT
Received: from MISOUT7MSGUSRDE.ITServices.sbc.com ([169.254.5.90]) by MISOUT7MSGHUBAC.ITServices.sbc.com ([130.9.129.147]) with mapi id 14.03.0248.002; Sat, 17 Oct 2015 12:11:49 -0400
From: "BRUNGARD, DEBORAH A" <db3546@att.com>
To: "Andrew G. Malis" <agmalis@gmail.com>, Robert Sparks <rjsparks@nostrum.com>
Thread-Topic: Gen-art LC review: draft-ietf-pals-redundancy-spe-02
Thread-Index: AQHRCFoB1F2sibcDrECvNq2HgCJ5Op5vpKQAgAAwBPA=
Date: Sat, 17 Oct 2015 16:11:48 +0000
Message-ID: <F64C10EAA68C8044B33656FA214632C8527548C0@MISOUT7MSGUSRDE.ITServices.sbc.com>
References: <56216C90.4060606@nostrum.com> <CAA=duU33DfGeTPKDYW-vwL0o4BQZrhcdsPHWou2Rtc5eiY=8XA@mail.gmail.com>
In-Reply-To: <CAA=duU33DfGeTPKDYW-vwL0o4BQZrhcdsPHWou2Rtc5eiY=8XA@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [135.70.191.252]
Content-Type: multipart/alternative; boundary="_000_F64C10EAA68C8044B33656FA214632C8527548C0MISOUT7MSGUSRDE_"
MIME-Version: 1.0
X-RSA-Inspected: yes
X-RSA-Classifications: public
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:, , definitions=2015-10-17_11:, , signatures=0
X-Proofpoint-Spam-Details: rule=outbound_policy_notspam policy=outbound_policy score=0 spamscore=0 suspectscore=0 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1507310000 definitions=main-1510170293
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/CigI8DRog0SY5KVxKyfOoW7IILg>
Cc: "draft-ietf-pals-redundancy-spe.all@ietf.org" <draft-ietf-pals-redundancy-spe.all@ietf.org>, General Area Review Team <gen-art@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>, "pals@ietf.org" <pals@ietf.org>
Subject: Re: [Gen-art] Gen-art LC review: draft-ietf-pals-redundancy-spe-02
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
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 Oct 2015 16:12:19 -0000

Hi Robert,

Thanks for your review. As Andy explained, in the Routing Area, there are multiple points in the process where folks are made aware/reminded of IPR and we follow RFC6702 suggestions. And for folks not participating actively in the Working Group, they still have the opportunity at IETF Last Call (which has a pointer to IPR) to discuss/express concern.

If you would like to discuss/recommend additional considerations for the IETF to follow on IPR, it would be good to change the subject line as I don’t think the current subject line is much of an attention-grabber☺

I’ll let the authors answer your questions on the draft itself.

Again, thanks for your careful review-
Deborah


From: Andrew G. Malis [mailto:agmalis@gmail.com]
Sent: Saturday, October 17, 2015 4:52 AM
To: Robert Sparks
Cc: General Area Review Team; ietf@ietf.org; pals@ietf.org; draft-ietf-pals-redundancy-spe.all@ietf.org
Subject: Re: Gen-art LC review: draft-ietf-pals-redundancy-spe-02

Robert,

Thanks for your review of the draft.

As PALS WG co-chair, I would like to address one point in your review:

There is a process issue that the IESG should pay attention to. The shepherd writeup says this: "There is one IPR declaration (1911) raised in November 2012 against an early version of the draft. There was no discussion in the WG related to this." That happens sometimes, but it's much better to have a real indication that the group considered the disclosure and explicitly decided not to change directions.

The IPR declaration is against the original individual draft, draft-dong-pwe3-redundancy-spe. The IPR declaration was from a company that was not represented as an author on the draft, and offered free licencing with reciprocity. At the time, no concerns were raised by either the authors or from anyone in the WG in response to the declaration. This, IMHO, is normal operating procedure when IPR declarations are made, especially by non-author entities and early in the process. The usual concern is when declarations come in late in the process, especially from an author company. Neither was the case here, it was still an individual draft. And of course, the declaration was clear for all to see during both WG adoption and WG LC polls.

Thanks,
Andy


On Fri, Oct 16, 2015 at 11:30 PM, Robert Sparks <rjsparks@nostrum.com<mailto:rjsparks@nostrum.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-pals-redundancy-spe-02
Reviewer: Robert Sparks
Review Date: 16 Oct 2015
IETF LC End Date: 19 Oct 2015
IESG Telechat date: 22 Oct 2015

Summary: Almost ready for publication as PS but with issues that need to be discussed/addressed

This document is hard to read. It is more acronym-laden than it
needs to be.

-----
There is a process issue that the IESG should pay attention to.
The shepherd writeup says this:

  "There is one IPR declaration (1911) raised in November 2012 against
   an early version of the draft.  There was no discussion in the WG
   related to this."

That happens sometimes, but it's much better to have a real indication
that the group considered the disclosure and explicitly decided not to
change directions.
-----

The last sentence of the 2nd paragraph (declaring multi-homing on both
sides of an S-PE out of scope) should be moved earlier in the document.
The introduction and perhaps even the abstract can be clearer about
what _is_ in scope.

It needs to be clearer where the normative description of behavior is.
I think you're intending it to be the first part of section 3. I have
not worked through the references enough to ensure that it is complete.

The third paragraph starts off "In general, ...". Are there any
specific cases where the requirements that follow do not hold? If so,
there needs to be more description. If not, please delete "In general,".

Are sections 3.1 and 3.2 supposed to be only examples? Would the
specification of the protocol be complete if they were deleted? If not,
something needs to be moved up into the main part of section 3.
For instance, is the SHOULD at the end of 3.1 a requirement placed by
this document, or is it restating a requirement from somewhere else?
Similarly, please inspect the SHOULD in the second paragraph of 3.2.

I also suggest moving 3.1 and 3.2 into their own section, clearly
labeling them as examples.

Is it worth more explanation in the document why you've added the
MUST NOT in the first paragraph of section 3?

The security considerations section only points off to other documents.
Most of those just point to each other. Chasing it back, there's some
meat in the security considerations section of 4447, and some in 5085,
but it's a real chase to find what's relevant.  Please consider calling
out what an implementer needs to consider explicitly here.