Re: [netconf] RFC 6241 Ambiguity

Kent Watsen <kent@watsen.net> Sat, 11 May 2019 14:43 UTC

Return-Path: <0100016aa75956af-70018fb1-15f8-4394-8ffd-4f4d5b2d7b3f-000000@amazonses.watsen.net>
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 57E57120020 for <netconf@ietfa.amsl.com>; Sat, 11 May 2019 07:43:35 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.889
X-Spam-Level:
X-Spam-Status: No, score=-1.889 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, T_REMOTE_IMAGE=0.01] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=amazonses.com
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 aq6qsd1lfzJl for <netconf@ietfa.amsl.com>; Sat, 11 May 2019 07:43:33 -0700 (PDT)
Received: from a8-33.smtp-out.amazonses.com (a8-33.smtp-out.amazonses.com [54.240.8.33]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id E9BA612001E for <netconf@ietf.org>; Sat, 11 May 2019 07:43:32 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/simple; s=6gbrjpgwjskckoa6a5zn6fwqkn67xbtw; d=amazonses.com; t=1557585811; h=From:Message-Id:Content-Type:Mime-Version:Subject:Date:In-Reply-To:Cc:To:References:Feedback-ID; bh=BzUsK80cMFPY2zi1+8iil6+NHIsWAN7V//18EJ8VL5Q=; b=N2tkMknprZnp/659BwZ3bJy1ZGUCpItaTAechbuLgU4wdSsN+pef8HEftmie0NTQ wBjtcM2BCTHFswrs0utdTIfze2In3tFQbJO2q1sz3A1GH6HiNTsph4nA+Z56CVwGIHn 2hxlkBwQWCsv5kLq82FaeDL7BDY7p9rOMX4n41Ac=
From: Kent Watsen <kent@watsen.net>
Message-ID: <0100016aa75956af-70018fb1-15f8-4394-8ffd-4f4d5b2d7b3f-000000@email.amazonses.com>
Content-Type: multipart/alternative; boundary="Apple-Mail=_74B477DA-78B3-44C9-9096-CD7DFFD4659A"
Mime-Version: 1.0 (Mac OS X Mail 12.2 \(3445.102.3\))
Date: Sat, 11 May 2019 14:43:31 +0000
In-Reply-To: <em35e87021-fa76-4888-a383-8b34e960175f@morpheus>
Cc: "netconf@ietf.org" <netconf@ietf.org>
To: Jonathan Hansford <jonathan@hansfords.net>
References: <em35e87021-fa76-4888-a383-8b34e960175f@morpheus>
X-Mailer: Apple Mail (2.3445.102.3)
X-SES-Outgoing: 2019.05.11-54.240.8.33
Feedback-ID: 1.us-east-1.DKmIRZFhhsBhtmFMNikgwZUWVrODEw9qVcPhqJEI2DA=:AmazonSES
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/KJCplaUdYp73SIo_ML3RVvH9xgA>
Subject: Re: [netconf] RFC 6241 Ambiguity
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: Sat, 11 May 2019 14:43:35 -0000

It seems that the "for any reason" clause should be qualified modulo the "persist" element (please file an errata report).  Note also the last paragraph in Section 8.4.1, which shows RFC 624 straddling capability versions, hence the wonky text. 

To answer your question. it is the latter: the only way to abort a persistent confirmed commit is to let the timer expire, or to use the <cancel-commit> operation.

Kent // contributor


> On May 8, 2019, at 9:33 AM, Jonathan Hansford <jonathan@hansfords.net>; wrote:
> 
> Hi,
> 
> In RFC 6241:
> 
> Section 8.4.1 states "If the session issuing the confirmed commit is terminated for any reason before the confirm timeout expires, the server MUST restore the configuration to its state before the confirmed commit was issued, unless the confirmed commit also  included a <persist> element."
> 
> Section 8.4.5.1 states the persist parameter makes "the confirmed commit survive a session termination".
> 
> Appendix C states the persist parameter "is used to make a confirmed commit persistent. A persistent confirmed commit is not aborted if the NETCONF session terminates. The only way to abort a persistent confirmed commit is to let the timer expire, or to use the <cancel-commit> operation." 
> 
> However:
> 
> Section 7.8, Erratum 5397 states "If a NETCONF server receives a <close-session> request while processing a confirmed commit (Section 8.4) for that session, regardless of whether the confirmed commit included a <persist> element, it MUST restore the configuration to its  state before the confirmed commit was issued."
> 
> Section 7.9, Erratum 5397 states "If a NETCONF server receives a <kill-session> request while processing a confirmed commit (Section 8.4) for that session, regardless of whether the confirmed commit included a <persist> element, it MUST restore the configuration to its  state before the confirmed commit was issued."
> 
> Section 8.4.1 states "If the device reboots for any reason before the confirm timeout expires, the server MUST restore the configuration to its state before the confirmed commit was issued."
> 
> So:
> 
> Is the use of <close-session> or <kill-session>, or the device rebooting, not considered to be the session being "terminated for any reason"? Or is it the case that "the only way to abort a persistent confirmed commit is to let the timer expire, or to use the <cancel-commit> operation"?
> 
> Jonathan
> 
>  <https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient>	Virus-free. www.avast.com <https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient> <x-msg://11/#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>_______________________________________________
> netconf mailing list
> netconf@ietf.org <mailto:netconf@ietf.org>
> https://www.ietf.org/mailman/listinfo/netconf <https://www.ietf.org/mailman/listinfo/netconf>