[I2nsf] Yangdoctors last call review of draft-ietf-i2nsf-sdn-ipsec-flow-protection-08
Martin Björklund via Datatracker <noreply@ietf.org> Mon, 24 August 2020 14:37 UTC
Return-Path: <noreply@ietf.org>
X-Original-To: i2nsf@ietf.org
Delivered-To: i2nsf@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 5C2D53A0E92; Mon, 24 Aug 2020 07:37:35 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Martin Björklund via Datatracker <noreply@ietf.org>
To: yang-doctors@ietf.org
Cc: last-call@ietf.org, draft-ietf-i2nsf-sdn-ipsec-flow-protection.all@ietf.org, i2nsf@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 7.14.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <159827985531.30993.17722282912726281276@ietfa.amsl.com>
Reply-To: Martin Björklund <mbj+ietf@4668.se>
Date: Mon, 24 Aug 2020 07:37:35 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/i2nsf/1ESs-bA_hfvox4RLP73iHxyXcAY>
Subject: [I2nsf] Yangdoctors last call review of draft-ietf-i2nsf-sdn-ipsec-flow-protection-08
X-BeenThere: i2nsf@ietf.org
X-Mailman-Version: 2.1.29
List-Id: "*I2NSF: Interface to Network Security Functions mailing list*" <i2nsf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/i2nsf>, <mailto:i2nsf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/i2nsf/>
List-Post: <mailto:i2nsf@ietf.org>
List-Help: <mailto:i2nsf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/i2nsf>, <mailto:i2nsf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 24 Aug 2020 14:37:35 -0000
Reviewer: Martin Björklund Review result: Ready with Nits I did an early YANG Doctor's review of this draft. Most of my comments then have been addressed in this version. Comments: o As I wrote in my early review, the RFC editor enforces a common format of YANG modules, so it is better to adhere to this format before sending the draft to the RFC editor. Use pyang -f yang --yang-line-length 69 <FILE> to get a consistent look-and-feel for your module. (You will have to manually re-flow description statements after this.) o There are some leafs that are optional in the model, but w/o a default value and w/o an explanation of what happens if that leaf is not set. You should find those and either make them mandatory, add a default value, or explain what it means when it isn't set. As an example, /ipsec-ike/pad/pad-entrypeer-authenticatin/pre-shared/secret is optional. I suspect that this leaf needs to be mandatory. Another example is the leaf espencap. /martin
- [I2nsf] Yangdoctors last call review of draft-iet… Martin Björklund via Datatracker
- Re: [I2nsf] [yang-doctors] Yangdoctors last call … Christian Hopps
- Re: [I2nsf] Yangdoctors last call review of draft… Gabriel Lopez
- Re: [I2nsf] [yang-doctors] Yangdoctors last call … Rob Wilton (rwilton)
- Re: [I2nsf] [yang-doctors] Yangdoctors last call … Rafa Marin-Lopez
- Re: [I2nsf] [yang-doctors] Yangdoctors last call … Christian Hopps
- Re: [I2nsf] [yang-doctors] Yangdoctors last call … Martin Björklund
- Re: [I2nsf] [yang-doctors] Yangdoctors last call … Martin Björklund
- Re: [I2nsf] [yang-doctors] Yangdoctors last call … Rob Wilton (rwilton)
- Re: [I2nsf] [Last-Call] [yang-doctors] Yangdoctor… tom petch
- Re: [I2nsf] [yang-doctors] Yangdoctors last call … Rafa Marin-Lopez
- Re: [I2nsf] [yang-doctors] Yangdoctors last call … Rob Wilton (rwilton)
- Re: [I2nsf] [Last-Call] [yang-doctors] Yangdoctor… tom petch
- Re: [I2nsf] [yang-doctors] Yangdoctors last call … Christian Hopps
- Re: [I2nsf] [yang-doctors] Yangdoctors last call … Martin Björklund
- Re: [I2nsf] [Last-Call] [yang-doctors] Yangdoctor… Christian Hopps
- Re: [I2nsf] [IPsec] [Last-Call] [yang-doctors] Ya… Lou Berger
- Re: [I2nsf] [Last-Call] [yang-doctors] Yangdoctor… Rafa Marin-Lopez
- Re: [I2nsf] [Last-Call] [yang-doctors] Yangdoctor… tom petch
- Re: [I2nsf] [Last-Call] [yang-doctors] Yangdoctor… Rafa Marin-Lopez
- Re: [I2nsf] [Last-Call] Yangdoctors last call rev… tom petch
- Re: [I2nsf] [yang-doctors] [IPsec] [Last-Call] Ya… Rob Wilton (rwilton)
- Re: [I2nsf] [yang-doctors] [IPsec] [Last-Call] Ya… Rafa Marin-Lopez
- Re: [I2nsf] [IPsec] [yang-doctors] [Last-Call] Ya… Christian Hopps
- Re: [I2nsf] [IPsec] [yang-doctors] [Last-Call] Ya… Rafa Marin-Lopez
- Re: [I2nsf] [IPsec] [yang-doctors] [Last-Call] Ya… Christian Hopps
- Re: [I2nsf] [IPsec] [yang-doctors] [Last-Call] Ya… Rafa Marin-Lopez
- Re: [I2nsf] [IPsec] [yang-doctors] [Last-Call] Ya… Christian Hopps