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".
>