Re: [Gen-art] Genart last call review of draft-ietf-anima-stable-connectivity-07
Toerless Eckert <tte@cs.fau.de> Wed, 17 January 2018 02:36 UTC
Return-Path: <eckert@i4.informatik.uni-erlangen.de>
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 308C712EC37; Tue, 16 Jan 2018 18:36:12 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.473
X-Spam-Level:
X-Spam-Status: No, score=-2.473 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, FAKE_REPLY_C=1.486, HEADER_FROM_DIFFERENT_DOMAINS=0.25, RCVD_IN_DNSWL_MED=-2.3, T_RP_MATCHES_RCVD=-0.01, URIBL_BLOCKED=0.001] 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 J-bPy82fcej6; Tue, 16 Jan 2018 18:36:10 -0800 (PST)
Received: from faui40.informatik.uni-erlangen.de (faui40.informatik.uni-erlangen.de [IPv6:2001:638:a000:4134::ffff:40]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D785812EC34; Tue, 16 Jan 2018 18:36:09 -0800 (PST)
Received: from faui40p.informatik.uni-erlangen.de (faui40p.informatik.uni-erlangen.de [IPv6:2001:638:a000:4134::ffff:77]) by faui40.informatik.uni-erlangen.de (Postfix) with ESMTP id A131758C5A1; Wed, 17 Jan 2018 03:36:05 +0100 (CET)
Received: by faui40p.informatik.uni-erlangen.de (Postfix, from userid 10463) id 881A3B0D79D; Wed, 17 Jan 2018 03:36:05 +0100 (CET)
Date: Wed, 17 Jan 2018 03:36:05 +0100
From: Toerless Eckert <tte@cs.fau.de>
To: Alissa Cooper <alissa@cooperw.in>
Cc: Matthew Miller <linuxwolf+ietf@outer-planes.net>, gen-art <gen-art@ietf.org>, anima@ietf.org, draft-ietf-anima-stable-connectivity.all@ietf.org
Message-ID: <20180117023605.GA26656@faui40p.informatik.uni-erlangen.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <2AAF0CB9-E081-40EB-ACC5-C752708BFCD1@cooperw.in>
User-Agent: Mutt/1.5.21 (2010-09-15)
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/Aj-w-CnTmbCbhYtNgOxR7ZxW8QU>
Subject: Re: [Gen-art] Genart last call review of draft-ietf-anima-stable-connectivity-07
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.22
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: Wed, 17 Jan 2018 02:36:12 -0000
Thanks a lot Matthew, Alissa I just posted -08: https://tools.ietf.org/id/draft-ietf-anima-stable-connectivity-08.txt All nits fixed. Diff here: http://tools.ietf.org/tools/rfcdiff/rfcdiff.pyht?url1=https://raw.githubusercontent.com/anima-wg/autonomic-control-plane/9bff109281e8b3c22522c3144cbf0f13e5000498/draft-ietf-anima-stable-connectivity/draft-ietf-anima-stable-connectivity-08.txt&url2=https://raw.githubusercontent.com/anima-wg/autonomic-control-plane/14d5f9b66b8318bc160cee74ad152c0b926b4042/draft-ietf-anima-stable-connectivity/draft-ietf-anima-stable-connectivity-08.txt > """ > If the secure ACP was extendable via untrusted routers then it would > be a lot more verify the secure domain assertion. > """ Was fixed to "...more difficult to verify..." via earlier fix for another reviewer, so it does not show up in diff above. > > * In Section 2.2. "Stable Connectivity for Distributed Network/OAM", > > second paragraph; the word "applicable" should be "applied" in the > > fragment "... how ell it applicable to a ...". Actually fixed to "how well it _is_ applicable" Hope this resolves your concerns. Cheers Toerless On Thu, Jan 11, 2018 at 09:36:32AM -0500, Alissa Cooper wrote: > Matt, thanks for your review. I have entered a No Objection ballot and asked for your nits to be addressed. > > Alissa > > > On Nov 26, 2017, at 3:48 PM, Matthew Miller <linuxwolf+ietf@outer-planes.net> wrote: > > > > Reviewer: Matthew Miller > > Review result: Ready with Nits > > > > 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 > > > > <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>. > > > > Document: draft-ietf-anima-stable-connectivity-07 > > Reviewer: Matthew A. Miller > > Review Date: 2017-11-26 > > IETF LC End Date: 2017-11-26 > > IESG Telechat date: N/A > > > > Summary: > > > > This document is ready to be published as informational, but there are > > a number of nits that ought be addressed before final publication. > > > > Overall, the document is > > > > Major issues: NONE > > > > Minor issues: NONE > > > > Nits/editorial comments: > > > > * Throughout, there is a mix of the term "data plane" and "data-plane". > > One form should be chosen and the rest corrected to match. > > > > * In Section 1.2. "Data Communications Networks (DCNs)", first > > paragraph; "'th" should be "'s" (or just "s"): "In the late > > 1990'th and ..." should be "In the late 1990s and ...". > > > > * In Section 1.2. "Data Communications Networks (DCNs)", first > > paragraph; the word "where" should be "were" in the fragment > > "These where (and still are) ...". > > > > * In Section 1.2. "Data Communications Networks (DCNs)", first > > paragraph; there is a missing "a" between "are" and "separate" in > > the fragment "they are separate network entirely". > > > > * In Section 2.1.1. "Simple Connectivity for Non-ACP capable NMS > > Hosts", first paragraph; there is a mismatch in plurality in the > > third sentence; instead of: > > > > """ > > They acts as the default router to those NMS hosts and provide them > > with IPv6 connectivity into the ACP. > > """ > > > > it should be: > > > > """ > > They act as the default routers to those NMS hosts and provide them > > with IPv6 connectivity into the ACP. > > """ > > > > * In Section 2.1.3. "Simultaneous ACP and Data Plane Connectivity", > > last paragraph; the following sentence seems to be missing a word > > or two: > > > > """ > > If the secure ACP was extendable via untrusted routers then it would > > be a lot more verify the secure domain assertion. > > """ > > > > * In Section 2.1.4. "IPv4-only NMS Hosts", second paragraph after the > > ordered list; the word "thought" should be "through" in the fragment > > "... be reachable thought the IPv6/IPv4 ...". > > > > * In Section 2.1.5. "Path Selection Policies", fourth paragraph from > > the section's end; there is an extra "of" between "shaping" and "at": > > > > """ > > Traffic policing and/or shaping of at the ACP edge in the NOC can be > > used to throttle applications such as software download into the ACP. > > """ > > > > * In Section 2.2. "Stable Connectivity for Distributed Network/OAM", > > second paragraph; there is a missing "to" between "start" and > > "provide" in the fragment "... tried to start provide common ...". > > > > * In Section 2.2. "Stable Connectivity for Distributed Network/OAM", > > second paragraph; the word "applicable" should be "applied" in the > > fragment "... how ell it applicable to a ...". > > > > * In Section 3.1. "No IPv4 for ACP", third paragraph; the word "as" > > should be "to" in the fragment "... from a native transport as just > > a service on the edge." > > > > * In Section 3.1. "No IPv4 for ACP", last paragraph; the word "type" > > should be "types" in the fragment "In other type of networks as > > well, ...". > > > > * In Section 3.1. "No IPv4 for ACP", last paragraph; the word "support" > > should be "supported" in the fragment "... family will be support so > > all use...". > > > > > > _______________________________________________ > > Gen-art mailing list > > Gen-art@ietf.org > > https://www.ietf.org/mailman/listinfo/gen-art
- [Gen-art] Genart last call review of draft-ietf-a… Matthew Miller
- Re: [Gen-art] Genart last call review of draft-ie… Alissa Cooper
- Re: [Gen-art] Genart last call review of draft-ie… Toerless Eckert