Re: [netconf] [netconf-wg/https-notif] Should the receivers container be moved under the augment statement and the leafref renamed? (#2)
Martin Bjorklund <mbj@tail-f.com> Tue, 11 February 2020 13:25 UTC
Return-Path: <mbj@tail-f.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 B9C881200E7 for <netconf@ietfa.amsl.com>; Tue, 11 Feb 2020 05:25:44 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Level:
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_HELO_NONE=0.001, SPF_PASS=-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 1RKvKb1Cx9xM for <netconf@ietfa.amsl.com>; Tue, 11 Feb 2020 05:25:43 -0800 (PST)
Received: from mail.tail-f.com (mail.tail-f.com [46.21.102.45]) by ietfa.amsl.com (Postfix) with ESMTP id 05A83120043 for <netconf@ietf.org>; Tue, 11 Feb 2020 05:25:42 -0800 (PST)
Received: from localhost (unknown [173.38.220.37]) by mail.tail-f.com (Postfix) with ESMTPSA id EA36D1AE018B; Tue, 11 Feb 2020 14:25:35 +0100 (CET)
Date: Tue, 11 Feb 2020 14:24:56 +0100
Message-Id: <20200211.142456.1627117725994673253.mbj@tail-f.com>
To: kent+ietf@watsen.net
Cc: netconf@ietf.org
From: Martin Bjorklund <mbj@tail-f.com>
In-Reply-To: <0100016fa69f59c6-212e569d-fb2b-47cb-a1ff-e897829ba111-000000@email.amazonses.com>
References: <netconf-wg/https-notif/issues/2/555352822@github.com> <20191119.161851.678459934233941550.mbj@tail-f.com> <0100016fa69f59c6-212e569d-fb2b-47cb-a1ff-e897829ba111-000000@email.amazonses.com>
X-Mailer: Mew version 6.8 on Emacs 25.2
Mime-Version: 1.0
Content-Type: Text/Plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/g2loFgO5fb--ldF6Xvq8gwn70OI>
Subject: Re: [netconf] [netconf-wg/https-notif] Should the receivers container be moved under the augment statement and the leafref renamed? (#2)
X-BeenThere: netconf@ietf.org
X-Mailman-Version: 2.1.29
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: Tue, 11 Feb 2020 13:25:45 -0000
Kent Watsen <kent+ietf@watsen.net> wrote: > Hi Martin, > > > > On Nov 19, 2019, at 10:18 AM, Martin Bjorklund <mbj@tail-f.com> wrote: > > > >> The reason to not move the receiver container under the augment is so > >> as to allow the leafref to point to multiple receivers. > > > > I don't understand this reason. > > Each "receiver" (in https-notif) maps to an HTTPS connection from the > publisher to the receiver. > > There is likely to be more than one configured subscription, yet all > notifications should go to the same receiver. > > We'd like to use the same HTTPS "connection" for all, as opposed to > having an HTTPS connection for each. > > The "receiver-ref" leaf provides an indirection enabling this > many-to-one relationship. I can't find the original thread about this issue, so I'll rephrase what I (think I) wrote from the start: I understand the reason for the many-to-one, and I wish we had that in the base model itself (i.e., RFC 8639), so that protocol-documents didn't have to invent this, and so that we could have receiver-specific config in one place. > > Since this is not a stand-alone model, I think it should augment > > /sn:subscriptions. In some way it doesn't matter what nodes are > > called and where they are located, but having descriptive names and > > keep related nodes under common subtrees helps the understanding of > > models. > > Is s/receiver-ref/https-receiver-ref/ what you had in mind? Yes, as a minimum. I would also change the top-level container "receivers" to augment /sn:subscriptions: augemnt /sn:subscriptions { container https-receivers { ... } } /martin
- Re: [netconf] [netconf-wg/https-notif] Should the… Martin Bjorklund
- Re: [netconf] [netconf-wg/https-notif] Should the… Kent Watsen
- Re: [netconf] [netconf-wg/https-notif] Should the… Martin Bjorklund
- Re: [netconf] [netconf-wg/https-notif] Should the… Kent Watsen
- Re: [netconf] [netconf-wg/https-notif] Should the… Martin Bjorklund