Re: [dhcwg] Opsdir last call review of draft-ietf-dhc-dhcpv6-yang-24
"Liushucheng (Will LIU, Strategy & Industry Development)" <liushucheng@huawei.com> Thu, 16 December 2021 07:06 UTC
Return-Path: <liushucheng@huawei.com>
X-Original-To: dhcwg@ietfa.amsl.com
Delivered-To: dhcwg@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 121D13A0C0C; Wed, 15 Dec 2021 23:06:02 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.897
X-Spam-Level:
X-Spam-Status: No, score=-1.897 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
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 6LLESLE986Td; Wed, 15 Dec 2021 23:05:57 -0800 (PST)
Received: from frasgout.his.huawei.com (frasgout.his.huawei.com [185.176.79.56]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 5301F3A0C0A; Wed, 15 Dec 2021 23:05:57 -0800 (PST)
Received: from fraeml707-chm.china.huawei.com (unknown [172.18.147.200]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4JF34x5Smfz67yyT; Thu, 16 Dec 2021 15:04:17 +0800 (CST)
Received: from dggpemm100004.china.huawei.com (7.185.36.189) by fraeml707-chm.china.huawei.com (10.206.15.35) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.20; Thu, 16 Dec 2021 08:05:48 +0100
Received: from dggpemm500002.china.huawei.com (7.185.36.229) by dggpemm100004.china.huawei.com (7.185.36.189) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.20; Thu, 16 Dec 2021 15:05:46 +0800
Received: from dggpemm500002.china.huawei.com ([7.185.36.229]) by dggpemm500002.china.huawei.com ([7.185.36.229]) with mapi id 15.01.2308.020; Thu, 16 Dec 2021 15:05:45 +0800
From: "Liushucheng (Will LIU, Strategy & Industry Development)" <liushucheng@huawei.com>
To: "ianfarrer@gmx.com" <ianfarrer@gmx.com>
CC: "ops-dir@ietf.org" <ops-dir@ietf.org>, "dhcwg@ietf.org" <dhcwg@ietf.org>, "draft-ietf-dhc-dhcpv6-yang.all@ietf.org" <draft-ietf-dhc-dhcpv6-yang.all@ietf.org>, "last-call@ietf.org" <last-call@ietf.org>
Thread-Topic: Opsdir last call review of draft-ietf-dhc-dhcpv6-yang-24
Thread-Index: AQHX3RWeh2vvx1pYkUGAESy7hwY8Naw0hDmA
Date: Thu, 16 Dec 2021 07:05:45 +0000
Message-ID: <91989b7006674302acccea2d964bce65@huawei.com>
References: <163722823173.27432.7026766423118638736@ietfa.amsl.com> <A698BBD7-DD36-485C-84DE-4949317322A8@gmx.com>
In-Reply-To: <A698BBD7-DD36-485C-84DE-4949317322A8@gmx.com>
Accept-Language: zh-CN, en-US
Content-Language: zh-CN
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.110.113.187]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Archived-At: <https://mailarchive.ietf.org/arch/msg/dhcwg/Sm2B2jxC2ePIh8u_44M-DcnvGpg>
Subject: Re: [dhcwg] Opsdir last call review of draft-ietf-dhc-dhcpv6-yang-24
X-BeenThere: dhcwg@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Dynamic Host Configuration <dhcwg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/dhcwg>, <mailto:dhcwg-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/dhcwg/>
List-Post: <mailto:dhcwg@ietf.org>
List-Help: <mailto:dhcwg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/dhcwg>, <mailto:dhcwg-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 16 Dec 2021 07:06:03 -0000
Dear Ian, Sorry for being late. Thanks for your reply. I had a long discussion on with several people to try to provide some content example. My reply inline with [Will]. > -----Original Message----- > From: ianfarrer@gmx.com [mailto:ianfarrer@gmx.com] > Sent: Friday, November 19, 2021 3:18 PM > To: Liushucheng (Will LIU, Strategy & Industry Development) > <liushucheng@huawei.com> > Cc: ops-dir@ietf.org; dhcwg@ietf.org; draft-ietf-dhc-dhcpv6-yang.all@ietf.org; > last-call@ietf.org > Subject: Re: Opsdir last call review of draft-ietf-dhc-dhcpv6-yang-24 > > Hi Will > > Many thanks for your review and comments. Please see inline below. > > Best regards, > Ian > > > On 18. Nov 2021, at 10:37, Will LIU via Datatracker <noreply@ietf.org> > wrote: > > > > Reviewer: Will LIU > > Review result: Has Nits > > > > Hi all, > > > > I have reviewed draft-ietf-dhc-dhcpv6-yang-24 as part of the > > Operational directorate's ongoing effort to review all IETF documents > > being processed by the IESG. These comments were written with the > > intent of improving the operational aspects of the IETF drafts. > > Comments that are not addressed in last call may be included in AD > > reviews during the IESG review. Document editors and WG chairs should > > treat these comments just like any other last call comments. > > > > “This document describes YANG data modules for the configuration and > > management of DHCPv6 (Dynamic Host Configuration Protocol for IPv6 > > RFC8415) servers, relays, and clients.” > > > > My overall view of the document is 'Has Nits'. > > > > ** Technical ** > > > > Page54, > > 1. The DHCPv6 server may be bound to an interface to specify the > > DHCPv6 address pool of the corresponding interface. > > [if - Interface configuration and binding the server function to specific > interfaces/addresses is not covered in the ietf-dhc6-server module. There is a > lot of variance in the way that individual implementations configure this (e.g. > server/router/BNG), and also the class-selection logic that defines how an > individual request’s address/prefix pool and options are selected that this best > left to the implementor to define implementation specific YANG to cover these > areas. > > Examples of vendor specific configuration modules for the server’s base > configuration and class-selector logic are given in Appendixes C & D. > ] [Will] Agree this is not covered in the module and a lot of variance exist. After checking the examples in Appendix, there is no example about DHCPv6 server bounding to an interface to specific interfaces/addresses, may I suggest add a short example like below? augment /if:interfaces/if:interface { description "Extend interface attribute."; container dhcpv6-server-attribute { description "Configure DHCPv6 server interface function."; choice server-mode { description "Set dhcpv6 server mode in the Interface."; case automatic-enable { description "Enable/disable dhcpv6 server automatic mode."; leaf server-automatic-anable { type boolean; default "false"; description "Enable/disable dhcpv6 server automatic mode."; } } case pool-name { description "Pool name."; leaf pool-name { type string { length "1..31"; } description "Pool name."; } } } leaf is-allow-hint { type boolean; default "false"; description "Enable/disable to allow the server to assign prefix according to the client expectation."; } leaf preference { type uint8 { range "0..255"; } default "0"; description "The server priority in the advertise packet."; } leaf is-rapid { type boolean; default "false"; description "Enable/disable to support for fast allocation."; } leaf is-unicast { type boolean; default "false"; description "Enable/disable to specify the unicast communication between client and server in the address renewal process."; } } > > > 2. The key of container address-pools and container prefix-pools in > > the DHCPv6 server may be changed to pool-name. > > [if - ‘pool-id’ was originally typed as being uint32 with the intention that it > was just a unique ID number for the pool. The type got changed to string due > to a recent review comment, But I think the intended use remains the same - it > is a unique identifier for the pool, whether the user choses a numeric or a > string based identifier as suits their requirements. > > My feeling is that pool-id is a suitable name for this function, but would be > happy to change if there is a good reason to use pool-name. > ] > [Will] Agree to change 'pool-id’ to string. Regards, | 致礼! Will LIU | 刘树成 > > > > ** Editorial ** > > > > No. > > > > Regards, > > Will (Shucheng LIU) > > > >
- [dhcwg] Opsdir last call review of draft-ietf-dhc… Will LIU via Datatracker
- Re: [dhcwg] Opsdir last call review of draft-ietf… ianfarrer
- Re: [dhcwg] Opsdir last call review of draft-ietf… Liushucheng (Will LIU, Strategy & Industry Development)
- Re: [dhcwg] Opsdir last call review of draft-ietf… Ms. Li HUANG
- Re: [dhcwg] Opsdir last call review of draft-ietf… ianfarrer