IETF Logo Mail Archive

Re: [netconf] Shepherd review on draft-ietf-netconf-netconf-client-server-26

tom petch <ietfc@btconnect.com> Wed, 04 January 2023 12:04 UTC

Return-Path: <ietfc@btconnect.com>
X-Original-To: netconf@ietfa.amsl.com
Delivered-To: netconf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 9D2A0C1516E8; Wed, 4 Jan 2023 04:04:58 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.901
X-Spam-Level:
X-Spam-Status: No, score=-1.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_MSPIKE_H2=-0.001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_PASS=-0.001] autolearn=unavailable autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=btconnect.onmicrosoft.com
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id UBlzLFehawJ6; Wed, 4 Jan 2023 04:04:57 -0800 (PST)
Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-he1eur01on2125.outbound.protection.outlook.com [40.107.13.125]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 40182C1516E5; Wed, 4 Jan 2023 04:04:53 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=JHzhq7vpZvkMoS8JN5VM4FtbCVKMQZHUx7juQYITA8LJELNpCco/L8juq4jJBjkljuoHSFq4garPjyv8Aaq17su5A5hIsaNbWpXg/x7cVrPngNMs2XBvxnB7AuDYsu3AD9OB6J6WaUJG5fR+gnVK//wb237cQB8/egBT6dkEJTKlef/naGVgiOl3vRWZlG566TEC0w3xnGiwNTWxrSO8c48IMXvsEjrOHQsX5DAAI6TiD25oTIqVKEhomVdmR1VOgz6NaMDdZNYcHpTrO88vH4oadZzSNFFH9VkrMyEWtp1Fc5OqOV5j1GShgTufXFQmzUHc2TLv/nZH3VKFL7CMcA==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=3u+OnqYbUw4ntvabIzbCzrWA2LEX3f0W8gyrDm6q6YI=; b=ljLa9o0oGJBoEBlBr7qRE235fmByT1yK1Dp7gVjZ0235tEkeUhqZYr7VFj94xl5Fv9cm4VD+fH+UpO6UzqTXxmaN/ccr/0gzsAadnr8s89oQernPcD6CTZ6WItIBlwdyJ5muKXvbWxcfwH2Euw5+6qxwNPdVYTAkxVeU7uXN03mDxidrrvwGfgCDAqHhzIEmqw19qg56DSZa1UqCrqSxfSsUHr87kyd0//pxzZrIu3r2xwjPvfJEIq1FAQ/V85JfrrlR70DlLEwwV7Femei3v/wdOZreG6eibZvGBSJC7gWMYECT1MBn9FiWuSf4XMPojaMkPzP7TFDrJY4xXxAiJQ==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=btconnect.com; dmarc=pass action=none header.from=btconnect.com; dkim=pass header.d=btconnect.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=btconnect.onmicrosoft.com; s=selector2-btconnect-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=3u+OnqYbUw4ntvabIzbCzrWA2LEX3f0W8gyrDm6q6YI=; b=c4X3CuKg3t7FL18y7ze+ee1Gj9aUwbspU4g62OS+fqn8KeQEu6zluEWuqGSUhqv4W+5GeK2ZbHFsiwpG1BEyjZZaM67yWKjF33GJ1FJ0swwfMVX7DYoCDnDcPqGb8tCL1ucscAtUH5TGu19CNitl8IFaOwR2e7Xoam/rIIdxvb0=
Received: from AM7PR07MB6248.eurprd07.prod.outlook.com (2603:10a6:20b:134::11) by AS8PR07MB9260.eurprd07.prod.outlook.com (2603:10a6:20b:61c::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5944.19; Wed, 4 Jan 2023 12:04:50 +0000
Received: from AM7PR07MB6248.eurprd07.prod.outlook.com ([fe80::86cd:e36d:9333:8537]) by AM7PR07MB6248.eurprd07.prod.outlook.com ([fe80::86cd:e36d:9333:8537%5]) with mapi id 15.20.5944.019; Wed, 4 Jan 2023 12:04:50 +0000
From: tom petch <ietfc@btconnect.com>
To: "maqiufang (A)" <maqiufang1=40huawei.com@dmarc.ietf.org>, "draft-ietf-netconf-netconf-client-server@ietf.org" <draft-ietf-netconf-netconf-client-server@ietf.org>
CC: "netconf@ietf.org" <netconf@ietf.org>, Kent Watsen <kent+ietf@watsen.net>
Thread-Topic: [netconf] Shepherd review on draft-ietf-netconf-netconf-client-server-26
Thread-Index: AdjBm3dc3N5rUNyhQlGZ+pRKJ2y9bxelaufd
Date: Wed, 04 Jan 2023 12:04:50 +0000
Message-ID: <AM7PR07MB62484D5C11D88D018576D06EA0F59@AM7PR07MB6248.eurprd07.prod.outlook.com>
References: <1145a6e0302d45f5b35fde9445fd90a9@huawei.com>
In-Reply-To: <1145a6e0302d45f5b35fde9445fd90a9@huawei.com>
Accept-Language: en-GB, en-US
Content-Language: en-GB
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
msip_labels:
authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=btconnect.com;
x-ms-publictraffictype: Email
x-ms-traffictypediagnostic: AM7PR07MB6248:EE_|AS8PR07MB9260:EE_
x-ms-office365-filtering-correlation-id: 3c582f96-44cf-442a-aece-08daee4be1b8
x-ms-exchange-senderadcheck: 1
x-ms-exchange-antispam-relay: 0
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: zHWzZ+sC5kgBz8jlzDc+ahHy8WcobWgZA5pqASe6ZKjq0JR+VdqF7nA9IK2P/ponv1Kov4s+g6CfD5C6ybz2KpOxc16y96OCCVJ83SJqbniTADBVxFRdb1MTmyD0nPqyyJ+XidPZMs9CiKIYEFl1alE3/rjA956t4k2Q5xtwR2G0dNMgNpgkpsRnCCQAKs/bHyt6cQaiyWUo2ikJefNzEZFP6c8GMvNzjC3fk/GX/BBxU6XY0ChZKkY+j+Iq6Rch8R4/I5CZyH5zMp7WQ4sQDbKlQKxutnSEDnWyDdMcULUQt+Z0OytaMj1OffDVk6kDXFBzQvebc1u7v45hMn7AXWcMQTcOZLTnis4bzfx7yPsCQL+f1GkX7x2YQgxc5d0QZKZTYCsSTOdpuUJeHZrlTBM7HC0CE01ybzybDBuU2iDAKS3x2Z3Q7bc6LtmL75CXFHjz9KjgAYRgy6KUOoU78GA2wIk9VHKppaGhrXPCVYiRszea2qmbUloVw2jUpkOgys1z9OC/RP2uFzknTno+nEzyyqRisYKtHvGrr7gGqOLB6VMWLfT9E+cCLl73KKnL/vSBilOfllI1vlMW3pJWdQldGVdPGLoLIVdWAsxj+KrBW94I2Xb4Wr95MUgeAuGDknTrkUb8rkcXI+z+9rXagMBRYVJxCrSxxx812OjMjdMb247G433EbuejJSnShFy3LnRUDBOi2qTXvnJ4Ak644LqKwgzP/56ytm9tyr32GvPtG9FPGyaVj+BpvRypckai
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM7PR07MB6248.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230022)(376002)(366004)(39860400002)(396003)(136003)(346002)(451199015)(5660300002)(52536014)(2906002)(8936002)(41300700001)(4326008)(478600001)(8676002)(316002)(66476007)(54906003)(66946007)(76116006)(110136005)(64756008)(66446008)(91956017)(66556008)(71200400001)(9686003)(26005)(6506007)(33656002)(55016003)(7696005)(83380400001)(122000001)(82960400001)(186003)(38100700002)(86362001)(38070700005); DIR:OUT; SFP:1102;
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: YbLFbeHLNVfIct97LZTG4I7dpFW2zDr2A1Pqoa27wGmkaC4EddESSWISXfdFgezOtD7OGtnNQGA37TPmWeKsOGsOtIRNmD16uH2Uyk4cGWrPZ3NoHd6Vic+qOD+FqV1GVj855xQiFQifYtJa+HYX+30FVkTJAeL2ex2vqxKqaVZUr52hxKWk34oDaE1iXCltX4hwX6da7oPKDe0T4RmLMw/wdkPSEyLx+6YoB/5kEivrqZROunJuRQxmhbilTgoGDv019mapKuviqqyg8/dd4hC0cnvn9pibpjQHQ4ga+4gmVa1tmFjnUtlHuGNKJj1PhEB3u/dVNoufUlEWduQXi2w4i3nEDzjbyz7XFmV5+3c1DBH4QVwvQ8hpxx+WpcruBtgJAt+eJKuD6JNPz4UxZFB8GuDQpFVI/hrdiGkOwQtG/raPh2aje7K48WWzwOgqNYDuu2w9k6rXXZLTWVKr0vBg2FYQoBxxjUltTmOmG5t23KNsfCy/oYyxSqB6yWdtPaL1ouTHLssh+fT1cVjWPlwuHf3CN0CL6E1z8FY7A6wv7D+eVcsQJkPJ7Inqh4nDdsTqwiNz+XKG3GQ8iDMFaXDDBAMqAyFoiJLUzNbBag6tFz052rOdIGLxrZOF4osQQnna+i+o2V2fVLLVpaINei63QiVeFGr9PWHDAnk79bpJpKRwu78uwB8okplIZ1UeydW5bOzczOhhHi1C+jgzWNZDXwUXZbeY7fe/O5mQZ2vB8UnXms8aS0abBtLMNzKwY6nq7C7lnrsNl1HV54E9iIij4L40qLs/JPN77hna1UASTxdPap+bQSnapjP3+0AbN2mkhNiUnm9sDJpflsXhzmYIrK0K5DqB8+UHklGKHTymoB0Fk4b0W/RTAp8RiCLEWjaUfaUbXv2sG1sqPlTNRf7PLquCwQ34lzE0h5w7XGWT+UViITCBkd5rsqIEzY7cc4BcsdOKVj0ayLVU9A+f924GZKfCX9dE7/Y8njJ5iHonxg/z8hZ+XqNpGPe7Dwx0Sin0n6PX6+eThaSvVIClzaIiN8SdgzKJu0f0Xq66LGFUge1YixAxZq6aR442yl4F1HGnHFJkU3BfEzjmJzS99ELOtO4ZvHF5qIVjSHliMNp2yE8EnWvzQDCtEMLOKikgC6LHMurZfkio8JiMAMjRBWBeZ8GXDKs5uHcEYY6OpTT9gIs7s9sMNHDHkduW5dClfOaqXhoPmcByKqG5NHXv8ZIFb1a13gqrSEI1W/GdPfm/pqQfO+g92A01E9H5+FiTswW1NfRk3xLnh6ljDXpC7XcFvv4ag1SN8wBFjLtcbdwoO4vHy0g6oXJMIo3cNKU6cqhQp7nR0S7CxHdoSDJ+ctVFcR7HtR5bjPJHLLxeVcvQNbi4pPsgqfCTr+0bHivugRClA9qlyHHKhmW81avqvJYlaWHcUAGkz1/dVkJ2V39j8kHC8UREO2BCas1GebI9r8dcz7+KD4DjdEO+Xjg8vw7BLdwtYK2lpA0miCqfBmQMUxGMX3bBCFVStiqImRwDogkzT7d1EQboAjmYujLWDPEK1gaSzkQKCngcl0N6jYM=
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: btconnect.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AM7PR07MB6248.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3c582f96-44cf-442a-aece-08daee4be1b8
X-MS-Exchange-CrossTenant-originalarrivaltime: 04 Jan 2023 12:04:50.2967 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: cf8853ed-96e5-465b-9185-806bfe185e30
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: GX/78BUysXKbH1u7JV/qjni7z9ydCvnXncD0GVkfc/aB5V70LtNq80dCmcn517ujEfBRJhFxlv5BKk7jhcXiGQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR07MB9260
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/-QcZEsEKSnRfBbGwQqdRHc4peDE>
Subject: Re: [netconf] Shepherd review on draft-ietf-netconf-netconf-client-server-26
X-BeenThere: netconf@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: NETCONF WG list <netconf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/netconf>, <mailto:netconf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/netconf/>
List-Post: <mailto:netconf@ietf.org>
List-Help: <mailto:netconf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/netconf>, <mailto:netconf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 04 Jan 2023 12:04:58 -0000

Nothing to do with the shepherd review, just some comments on -28

-grouping on all the identifiers of groupings I find clumsy and redundant

      downstream modules can augment nodes into it if needed.
I do not know what downstream is in this context - what is flowing where in this stream?

 *  The "transport" choice node enables both the SSH and TLS
  No, either or, not both; ditto for the server module

leaf name string
is used as a key with no restriction on length or on character set - CJK anyone?

even if due to a reboot
reboot of what?

A NETCONF session will be  dropped if it is idle for an interval longer than this
              number of seconds.  
netconf-restconf-client-server has a similar statement but about the TCP session- why the difference?
ditto for server module

 Sessions that have a notification subscription active are never
              dropped.";
sounds like a MUST NOT to me 
ditto in server module

the same documentation address is used in all examples.  Were I an implementor I would have chosen different ones for different protocols but YMMV  

      choice transport {
         mandatory true;
         case ssh {
           if-feature "ssh-listen";
....
         case tls {
           if-feature "tls-listen";
suggests to me that it is mandatory to have one or other of the listen features which I do not know if YANG can specify otherwise assuming that that is what is intended

               case periodic-connection {
                 container periodic {
                   description
                     "Periodically connect to the NETCONF client.

                      This connection type decreases resource
                      utilization, albeit with increased delay in
                      NETCONF client to NETCONF client interactions.
right and wrong; depends on the resources.  This needs more explanation perhaps in the body of the I-D rather than in the modules

Tom Petch







f-bounces@ietf.org> on behalf of maqiufang (A) <maqiufang1=40huawei.com@dmarc.ietf.org>
Sent: 06 September 2022 13:16
To: draft-ietf-netconf-netconf-client-server@ietf.org
Cc: netconf@ietf.org
Subject: [netconf] Shepherd review on draft-ietf-netconf-netconf-client-server-26

Hi, all

I am glad to be the doc shepherd for this draft as it moves beyond the WG with the other client-server suite of drafts.

v2.23.0 | Report a Bug | By Email