[OPSAWG] Benjamin Kaduk's No Objection on draft-ietf-opsawg-nat-yang-16: (with COMMENT)
Benjamin Kaduk <kaduk@mit.edu> Tue, 25 September 2018 18:30 UTC
Return-Path: <kaduk@mit.edu>
X-Original-To: opsawg@ietf.org
Delivered-To: opsawg@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 0DE80128B14; Tue, 25 Sep 2018 11:30:42 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Benjamin Kaduk <kaduk@mit.edu>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-opsawg-nat-yang@ietf.org, Joe Clarke <jclarke@cisco.com>, opsawg-chairs@ietf.org, jclarke@cisco.com, opsawg@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.84.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <153790024204.5176.8102975803900099153.idtracker@ietfa.amsl.com>
Date: Tue, 25 Sep 2018 11:30:42 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/opsawg/mGxXfieWnF7qJwyO4XQ1zjqK72Y>
Subject: [OPSAWG] Benjamin Kaduk's No Objection on draft-ietf-opsawg-nat-yang-16: (with COMMENT)
X-BeenThere: opsawg@ietf.org
X-Mailman-Version: 2.1.29
List-Id: OPSA Working Group Mail List <opsawg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/opsawg>, <mailto:opsawg-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/opsawg/>
List-Post: <mailto:opsawg@ietf.org>
List-Help: <mailto:opsawg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/opsawg>, <mailto:opsawg-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 25 Sep 2018 18:30:42 -0000
Benjamin Kaduk has entered the following ballot position for draft-ietf-opsawg-nat-yang-16: No Objection When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-opsawg-nat-yang/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Thanks for the easy-to-read document! I just have a few comments and potential nits I noticed. It was somewhat interesting to me that basically everything is config rw, including ports and addresses that would normally be assigned internally by the NAT, but I don't see this as problematic. Section 2.1 Considerations about instructing explicit dynamic means (e.g., [RFC6887], [RFC6736], or [RFC8045]) are out of scope. [...] I'm having trouble parsing this; is it maybe "instructing by explicit dynamic means" or "explicit dynamic mappings"? Section 3 What's the relationship between hold-down-timeout and hold-down-max -- that is, if the maximum number of ports in the pool gets hit, to the oldest ports in the pool get ejected even if they haven't timed out, or what happens? I don't expect this to need to be in the document, but I'm curious what the use case for the all-algs-enable leaf is. I may be confused, but is the ordering relationship between low-threshold and high-threshold correct? From the description it would seem like we need low < high, but I'm reading the text as requiring low >= high. Also, the error-message for that "must" stanza talks about port numbers, not percentage thresholds. container connection-limits { [...] list limit-per-protocol { [...] leaf limit { type uint32; description "Rate-limit the number of protocol-specific mappings and sessions per instance."; This is a maximum, not a rate-limit, I think? Section A.6 EAMs may be enabled jointly with statefull NAT64. This example shows a NAT64 function that supports static mappings: nit: "stateful"
- [OPSAWG] Benjamin Kaduk's No Objection on draft-i… Benjamin Kaduk
- Re: [OPSAWG] Benjamin Kaduk's No Objection on dra… mohamed.boucadair
- Re: [OPSAWG] Benjamin Kaduk's No Objection on dra… Benjamin Kaduk
- Re: [OPSAWG] Benjamin Kaduk's No Objection on dra… mohamed.boucadair
- Re: [OPSAWG] Benjamin Kaduk's No Objection on dra… Benjamin Kaduk
- Re: [OPSAWG] Benjamin Kaduk's No Objection on dra… mohamed.boucadair
- Re: [OPSAWG] Benjamin Kaduk's No Objection on dra… mohamed.boucadair