[Gen-art] Telechat Review of draft-ietf-p2psip-share-09

Matt Miller <mamille2@cisco.com> Thu, 03 November 2016 02:21 UTC

Return-Path: <mamille2@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 09F4312987D; Wed, 2 Nov 2016 19:21:07 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -16.018
X-Spam-Level:
X-Spam-Status: No, score=-16.018 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_HI=-5, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-1.497, SPF_PASS=-0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com
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 ZqZkgiA9xYuf; Wed, 2 Nov 2016 19:21:04 -0700 (PDT)
Received: from alln-iport-7.cisco.com (alln-iport-7.cisco.com [173.37.142.94]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 37DD2129871; Wed, 2 Nov 2016 19:21:03 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=2838; q=dns/txt; s=iport; t=1478139663; x=1479349263; h=to:from:subject:message-id:date:mime-version; bh=5Kl+IZUC120K3kebgbJgIkMS3qEooJBrIjBDNeKFJVw=; b=hxq8Nc0Dk1PelLJuTU6WtO4KskyAyECMXnhgfnO/NkI8s3S26ojHjRn6 7Q5h7fE5w6df0WcmMgcXm1DzqXcUTjg2xnLbiX81ae2lB/aHTtSlmsoaI PQwBhxP8Ki5Y5Ayww9jZqHFAlzEpD3mKplSl3iPBPl3jr11we5065X5VX Y=;
X-Files: signature.asc : 496
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0BkBQCmnhpY/4UNJK1dHAEBBAEBCgEBgywBAQEBAR9YKlIBjTaVRBsFAYEXlEuCCCiFeoIcPxQBAgEBAQEBAQFiKIULgQUuAmAMCAEBiFIOrSeNAAEiDoVygkgIhzeCZIJcBYEmAY4iilICg0mBeXGKB4FuToQhgxiGFI0YhAQeNlZMhHtUhhYoghMBAQE
X-IronPort-AV: E=Sophos;i="5.31,436,1473120000"; d="asc'?scan'208";a="343492076"
Received: from alln-core-11.cisco.com ([173.36.13.133]) by alln-iport-7.cisco.com with ESMTP/TLS/DHE-RSA-AES256-SHA; 03 Nov 2016 02:21:01 +0000
Received: from [64.101.72.62] ([64.101.72.62]) by alln-core-11.cisco.com (8.14.5/8.14.5) with ESMTP id uA32L1kj028716; Thu, 3 Nov 2016 02:21:01 GMT
To: gen-art@ietf.org, draft-ietf-p2psip-share.all@ietf.org
From: Matt Miller <mamille2@cisco.com>
Message-ID: <00bc7a56-831f-2436-e44e-9ae4e4a896cb@cisco.com>
Date: Wed, 02 Nov 2016 20:21:01 -0600
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:45.0) Gecko/20100101 Thunderbird/45.4.0
MIME-Version: 1.0
Content-Type: multipart/signed; micalg="pgp-sha512"; protocol="application/pgp-signature"; boundary="42dk6KnD703FVEPUKWREb6aCxSWuGB09d"
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/un7aEG4ZsSzt136pyyFRAxW3eFw>
Subject: [Gen-art] Telechat Review of draft-ietf-p2psip-share-09
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: Thu, 03 Nov 2016 02:21:07 -0000

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-p2psip-share-09
Reviewer: Matthew A. Miller
Review Date: 2016-11-02
IETF LC End Date: 2016-10-25
IESG Telechat date: 2016-11-03

Summary:

This document is ready to be published as a Standards Track
document once the nits are addressed.

Major issues: NONE

Minor issues: NONE

Nits/editorial comments:

* idnits reports a stale reference to I-D.ietf-p2psip-sip (should
  be RFC 7904).

* In 5.1. "Overview", the word "witch" should be "which".

* In 5.3. "Overlay Configuration Document Extension", there should
  be a space between "P2PSIP" and "[I-D.ietf-p2psip-sip]".

* In 6.2. "Revoking White Access", there should be a space between
  "see" and "[RFC6940]".

* In 6.4. "Operations of Storing Peers", a comma is missing between
  "peers" and "at" in the phrase "Storing peers at which Shared
  Resource and ACL are physically stored ...".


Non-issue comments:

* idnits is reporting weird spacing and "possible code", but that
  appears to be due to the Relax NG grammar.  In my opinion the nit
  can be safely ignored.


-- 
- m&m

Matt Miller
Cisco Systems, Inc.