[Gen-art] Gen-ART Last Call review of draft-ietf-ospf-two-part-metric-05
Brian E Carpenter <brian.e.carpenter@gmail.com> Sun, 07 August 2016 01:39 UTC
Return-Path: <brian.e.carpenter@gmail.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 9DF3612B061; Sat, 6 Aug 2016 18:39:02 -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 waUvBA7W4XFe; Sat, 6 Aug 2016 18:39:00 -0700 (PDT)
Received: from mail-pa0-x233.google.com (mail-pa0-x233.google.com [IPv6:2607:f8b0:400e:c03::233]) (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 60C1112B043; Sat, 6 Aug 2016 18:38:57 -0700 (PDT)
Received: by mail-pa0-x233.google.com with SMTP id fi15so103368536pac.1; Sat, 06 Aug 2016 18:38:57 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:subject:to:organization:message-id:date:user-agent :mime-version:content-transfer-encoding; bh=xnoidCUYbdcNg5WdBjFnKUBseWZ9EXHp0ecDSlMGEn4=; b=1DoHRCmO/7o3fUs2qjFNULpNrqi33dD5JX19xgyL9wxVoGoxzZ0DvulzO6ze3c+x3+ UiNf0LYX13FiM62fTEj+8pYubdb84+50A/V0N945l4v+pyoAiD3S5wRAm1UGlgOChsBv TxuKB83quYY8dPppoM03llM8isXZ/TtrwuphcbO49japMvGys2D6A4agP6m5oU6kH7VY CaIA8vZOXbStyG5ClE5Xb6pWMMd3emuTSIiP9C5GrkXNfhpbsA8e3/OUxWppLs3oFnWa mxTjouCrESYYpZsXKCZhXBJuE1Wdr5HUZi8mVQEENWoq576iLoSKU7PnyDleLmxeTWcn 1SVA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:subject:to:organization:message-id:date :user-agent:mime-version:content-transfer-encoding; bh=xnoidCUYbdcNg5WdBjFnKUBseWZ9EXHp0ecDSlMGEn4=; b=UV1sBcCn9yhK1ZkFk9OmsCAlftc8+AFECe2kkugkTb+wO2Iogv9u8pc57TEhdEgy4A 4vlMKj8upIFEkGkkAHJo75qQEdTP8PvtAf9+GkvwrRvdDAwc8LjXA8A4aigdAcuNtVuq cSHIAjrxNEE+0fMjaw0M2f5AHLeRDCRwG8DdCneLSmEguNmbJG1JnN2z25LIimkyFJ2g rwuL7UK6keALOzbWEh9A+4eCORHD6XKxw9nesihr9U3n56XqqwaPjl1/eYTXE/CdAunj OUOwkDWDVCxMoF/B/qnuHFnp3hwrxBNKCiAIvQxiwwCKoBN7qshhWQnscP9v7arTCgdj a7gA==
X-Gm-Message-State: AEkoousIJALeDwZLf5DOK+VFWO2CryvZG+uuWnI91Fr3ST2xJmmp79B+HCCqjR1Cym/8Yg==
X-Received: by 10.66.8.163 with SMTP id s3mr50492079paa.142.1470533936738; Sat, 06 Aug 2016 18:38:56 -0700 (PDT)
Received: from [192.168.178.23] ([118.148.115.173]) by smtp.gmail.com with ESMTPSA id s12sm37750022pfj.73.2016.08.06.18.38.54 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sat, 06 Aug 2016 18:38:55 -0700 (PDT)
From: Brian E Carpenter <brian.e.carpenter@gmail.com>
To: draft-ietf-ospf-two-part-metric.all@ietf.org, General Area Review Team <gen-art@ietf.org>
Organization: University of Auckland
Message-ID: <4ee8e895-a939-d747-a82f-fb7c8696b36e@gmail.com>
Date: Sun, 07 Aug 2016 13:38:52 +1200
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/mPoAOW3scdwiq7CqAfPUdruLyus>
Subject: [Gen-art] Gen-ART Last Call review of draft-ietf-ospf-two-part-metric-05
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: Sun, 07 Aug 2016 01:39:02 -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-ospf-two-part-metric-05.txt Reviewer: Brian Carpenter Review Date: 2016-08-07 IETF LC End Date: 2016-08-15 IESG Telechat date: Summary: Almost ready -------- Major issues: ------------- > Updates: 2328, 5340 (if approved) If that is so, the text needs to explain what is changed in those two RFCs. Since this draft describes an "optional extension" to OSPF, it does not obviously update them. Is any text in those two RFCs made invalid by this draft? > 3.6. SPF Calculation > > During the first stage of shortest-path tree calculation for an area, > when a vertex V corresponding to a Network-LSA is added to the > shortest-path tree and its adjacent vertex W (joined by a link in V's > corresponding Network LSA), the cost from V to W, which is W's > network-to-router cost, is determined as follows: I can't parse that sentence. If we delete the subordinate clauses, we get When a vertex V is added to the shortest-path tree and its adjacent vertex W, the cost from V to W is determined as follows: What does that mean? What does "its" refer to? Is W adjacent to V, or is W adjacent to the existing tree? Is W added to the tree before V, or is V added before W? If I was coding this, I'd have no idea what to do. > 3.7. Backward Compatibility This calls for a Router Functional Capability Bit assignment under RFC 7770. The bit number should be given as (say) TBD1 not as 0. > 4. IANA Considerations The IANA considerations ask for four assignments. These should be specified as TBD1, TBD2, TBD3, TBD4 and the TBDs elsewhere in the text should be updated correspondingly. Also, please reference the relevant RFCs (7770 and whatever defines the Sub-TLV registries.) Finally, to put this on the standards track, I would really expect to see an Implementation Status section (RFC 7942). Has this been tested? Minor issues: ------------- Please check the three occurrences of lower-case "must" in Section 3. Should they be "MUST"? > 5. Security Considerations > > This document does not introduce new security risks. That's easy to say but hard to prove. Shouldn't you at least refer to the security considerations of OSPFv2 and OSPFv3? Also, does section 3.7 introduce a new risk whereby a rogue router could flap its Two-Part Metric bit on and off, causing all its OSPF peers to continually recalculate their routes? Nits: ----- > Requirements Language It's unusual to put this at the front. The normal place is after the Introduction. > This document may contain material from IETF Documents or IETF > Contributions published or made publicly available before November > 10, 2008. ... Why is this needed? What did you copy from an old document? > 0 OSPF Two-part Metric [TPM] The abbreviation TPM is defined but not used, so why bother? Also, s/[TPM]/(TPM)/ to avoid confusion with a reference. > routes w/o considering any network-to-router costs. Just say "without".
- Re: [Gen-art] Gen-ART Last Call review of draft-i… Brian E Carpenter
- Re: [Gen-art] Gen-ART Last Call review of draft-i… Acee Lindem (acee)
- Re: [Gen-art] Gen-ART Last Call review of draft-i… Brian E Carpenter
- Re: [Gen-art] Gen-ART Last Call review of draft-i… Jeffrey (Zhaohui) Zhang
- Re: [Gen-art] Gen-ART Last Call review of draft-i… Acee Lindem (acee)
- Re: [Gen-art] Gen-ART Last Call review of draft-i… Brian E Carpenter
- Re: [Gen-art] Gen-ART Last Call review of draft-i… Acee Lindem (acee)
- [Gen-art] Gen-ART Last Call review of draft-ietf-… Brian E Carpenter