Re: Gen-ART Telechat review of draft-ietf-isis-node-admin-tag-10
Pushpasis Sarkar <pushpasis.ietf@gmail.com> Tue, 10 May 2016 03:30 UTC
Return-Path: <pushpasis.ietf@gmail.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id E7E6B12D1BB; Mon, 9 May 2016 20:30:59 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.7
X-Spam-Level:
X-Spam-Status: No, score=-2.7 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.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 l1IOGkgFdn1Q; Mon, 9 May 2016 20:30:58 -0700 (PDT)
Received: from mail-pf0-x22d.google.com (mail-pf0-x22d.google.com [IPv6:2607:f8b0:400e:c00::22d]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id CBD2812D195; Mon, 9 May 2016 20:30:57 -0700 (PDT)
Received: by mail-pf0-x22d.google.com with SMTP id 206so288130pfu.0; Mon, 09 May 2016 20:30:57 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=subject:to:references:cc:from:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding; bh=1bQHdfmoeFdkwmaTKkXjuJ3c2UaKJSRv7DV4jXJXWcI=; b=JNxVKBljHrclSNPh1uPBfusdDFLRfStFXw5Qi76z81O3q6O17eRLmuiYzzaxQjNmkR 56w+93PnZ9nKzoMBseeUbtuumobZjtNSvl1tDRPIK5nBEX2zSNVywzH9ndum+mtzIZgK mtbIe17WxKMSpUc4Zkuo+8hEMJXSqnbt+rABgcKub/o13HCssbgKl6Oj2aOkjeFj7PJv RPmEIRclTL9yTMP61ZCI9QkJzTTykHrAKV9uI8sAE6zj8lSaliAcPozN0im7s1bIxgLg moZ9dzv3hCZdU9VEil2KkviwK0T3Y0glwckmW3qwWDXBMPlC2RYqVxM/NO0xcKjCNYUv aYoQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:subject:to:references:cc:from:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding; bh=1bQHdfmoeFdkwmaTKkXjuJ3c2UaKJSRv7DV4jXJXWcI=; b=PaLfTql4P23uz+8yD33LieLBqe6+yioqqpTM4gbJZjCXMg7MwcTFUqNyzXBwIEQoN4 tpkkW8iEGbmwe8XEWlIsVCUnBes/94gPFbtIL5rj4k2oDwoutC+SqsagZQ11/UVlmblv E8hlCkKRYbK5XLpVCkOHV5mF0LRaOCgsxfrNQcJGL7XhRsGiYUb5U9TfOJzdivxwGGTh 014tqFa/zA4EZIiC7fpQALDY47OnsQBrwoOlQeBhLyIZfrFBXnC1/P9rfWfvDUCkZgGw p1io2ZLozJfK1kv824cFkuhgYMa8TkD8e46u1Nd0EAjrXBpVx9D8/u2YDW7QCjR2e4uu SDtQ==
X-Gm-Message-State: AOPr4FUbHgKeRo8/oPZ/Fs1q6GgQHeeSSGSsALK7rb0C26rVmfVy87C9NRt+geGeNPpxMw==
X-Received: by 10.98.109.197 with SMTP id i188mr55066754pfc.88.1462851057422; Mon, 09 May 2016 20:30:57 -0700 (PDT)
Received: from Pushpasiss-MacBook-Pro.local ([122.166.94.67]) by smtp.gmail.com with ESMTPSA id g84sm44776pfj.42.2016.05.09.20.30.55 (version=TLSv1/SSLv3 cipher=OTHER); Mon, 09 May 2016 20:30:56 -0700 (PDT)
Subject: Re: Gen-ART Telechat review of draft-ietf-isis-node-admin-tag-10
To: Peter Yee <peter@akayla.com>, draft-ietf-isis-node-admin-tag.all@ietf.org
References: <027101d1a57f$25508a80$6ff19f80$@akayla.com>
From: Pushpasis Sarkar <pushpasis.ietf@gmail.com>
Message-ID: <f03d2ea8-1a41-65bf-2bdd-d3c6f02308bc@gmail.com>
Date: Tue, 10 May 2016 09:00:54 +0530
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:45.0) Gecko/20100101 Thunderbird/45.0
MIME-Version: 1.0
In-Reply-To: <027101d1a57f$25508a80$6ff19f80$@akayla.com>
Content-Type: text/plain; charset="windows-1252"; format="flowed"
Content-Transfer-Encoding: 7bit
Archived-At: <http://mailarchive.ietf.org/arch/msg/ietf/UGepBlgMUpqIlJnFrNK7KQAI_po>
Cc: gen-art@ietf.org, ietf@ietf.org
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 10 May 2016 03:31:00 -0000
Hi Peter, I have uploaded version-11 addressing all the comments below. Please let me know if you have any more comments. Thanks and Regards, -Pushpasis On 5/4/16 2:32 AM, Peter Yee 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 wait for direction from your document shepherd or AD > before posting a new version of the draft. For background on Gen-ART, > please see the FAQ at > <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq> > > Document: draft-ietf-isis-node-admin-tag-10 > Reviewer: Peter Yee > Review Date: May 3, 2016 > IETF LC End Date: April 29, 2016 > IESG Telechat date: May 5, 2016 > > Summary: This draft is basically ready for publication as a Standards Track > RFC, but has an issue and some nits that should be fixed/considered before > publication. [Ready with issues] > > This draft defines a means to carry additional per-node administrative tags > with the IS-IS protocol. These tags can be used along with local policy to > simplify the management of routing and path selection. This specification > gives informative examples of such tag usage but does not otherwise > prescribe the meaning of the tags. > > Major issues: None > > Minor issues: > > Page 5, section 4.2, 2nd paragraph, 1st sentence: The sentence states: > "Being part of the Router CAPABILITY TLV, the node administrative tag > sub-TLV MUST be reasonably small and stable." If you're going to make this > a MUST, you've got to at least give a definition of "reasonably small" and > perhaps even "stable" in the context of this specification. As it stands, > there's no test for whether the MUST is enforceable or understandable > between parties. > > Nits: > > General: > > The use of capitalization of Node administrative tag varies throughout the > document. It seems clear that you mean for it to be written as "Node > Administrative Tag" when referring to the name of sub-TLV. For other uses, > I would suggest using "node administrative tag" (all lower case) > consistently. Use that to replace "Node administrative tag". > > Specific: > > Page 3, 1st paragraph after the labeled items at the top, 3rd sentence: > change the first "TLV" to "sub-TLV". > > Page 3, Section 2, 2nd paragraph: append a comma after "another". > > Page 3, Section 3, 1st paragraph, 1st sentence: change "a" before "IS-IS" to > "an". > > Page 3, Section 3, 1st paragraph, 2nd sentence: change "Capablity" to > "CAPABILITY". > > Page 3, Section 3, 1st paragraph, 3rd sentence: change "Operator" to "An > operator". Change "diiferent" to "different". > > Page 3, Section 3, 2nd paragraph, 1st sentence: insert "the" before "Node". > > Page 3, Section 3, 2nd paragraph, 2nd sentence: change "topology specific" > to "topology-specific". (That is to say, swap the space for a hyphen.) > > Page 4, Section 3.1, Value definition: insert "node" before > "administrative". > > Page 4, Section 4.1, 1st sentence: change "Node" to "node". (See general > nit.) > > Page 4, Section 4.1, 3rd sentence: delete the space after the slash and > before regulations. > > Page 5, 3rd full paragraph, 4th sentence: delete the spaces after closing > parenthesis and the terminating period. > > Page 6, Section 4.3, 1st paragraph, 2nd sentence: change "Node" to "node". > (See general nit.) Change the first "TLVs" to "sub-TLVs". > > Page 6, Section 4.3, 2nd paragraph, 2nd sentence: delete the space before > the comma. > > Page 6, Section 5, 1st paragraph, 1st sentence: change "Node" to "node". > (See general nit.) > > Page 6, Section 5, 1st paragraph, 4th sentence: change "Following" to "The > following". > > Page 6, Section 5, 1st paragraph, 5th sentence: change "section-3" to > "section 3". > > Page 7, Section 6, 2nd paragraph, the comma should be rejoined with the > closing parenthesis. > > Page 7, Section 7, 1st paragraph, 3rd sentence: change "is" to "are". > > Page 7, Section 8, 1st paragraph, 2nd sentence: delete "The" before "YANG". > With the change in the rest of the sentence, "The" becomes superfluous. > > Page 8, Section 9: change "Capabality" to "CAPABILITY". >
- Gen-ART Telechat review of draft-ietf-isis-node-a… Peter Yee
- Re: [Gen-art] Gen-ART Telechat review of draft-ie… Jari Arkko
- Re: Gen-ART Telechat review of draft-ietf-isis-no… Pushpasis Sarkar
- Re: Gen-ART Telechat review of draft-ietf-isis-no… Pushpasis Sarkar