[mpls] [Technical Errata Reported] RFC8960 (7059)

RFC Errata System <rfc-editor@rfc-editor.org> Fri, 29 July 2022 11:22 UTC

To: tsaad@juniper.net, skraza@cisco.com, rgandhi@cisco.com, xufeng.liu.ietf@gmail.com, vbeeram@juniper.net, aretana.ietf@gmail.com, jgs@juniper.net, andrew-ietf@liquid.tech, loa@pi.nu, n.leymann@telekom.de, tsaad.net@gmail.com
From: RFC Errata System <rfc-editor@rfc-editor.org>
Cc: jlindbla@cisco.com, mpls@ietf.org, rfc-editor@rfc-editor.org
Content-Type: text/plain; charset="UTF-8"
Message-Id: <20220729112223.B2AED4C29E@rfcpa.amsl.com>
Date: Fri, 29 Jul 2022 04:22:23 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/YZJxNjD3PRLJ9SooH34coeY0vzs>
Subject: [mpls] [Technical Errata Reported] RFC8960 (7059)
Precedence: list

The following errata report has been submitted for RFC8960,
"A YANG Data Model for MPLS Base".

--------------------------------------
You may review the report below and at:
https://www.rfc-editor.org/errata/eid7059

--------------------------------------
Type: Technical
Reported by: Jan Lindblad <jlindbla@cisco.com>

Section: 2.5

Original Text
-------------
augment "/rt:routing/rt:ribs/rt:rib/rt:routes/rt:route/"
+ "rt:next-hop/rt:next-hop-options/rt:simple-next-hop" {
description
"Augments the 'simple-next-hop' case in IP unicast routes.";
uses nhlfe-single-contents {
when "/rt:routing/rt:ribs/rt:rib/rt:routes/rt:route"
+ "/mpls:mpls-enabled = 'true'";
}
}

Corrected Text
--------------
augment "/rt:routing/rt:ribs/rt:rib/rt:routes/rt:route/"
+ "rt:next-hop/rt:next-hop-options/rt:simple-next-hop" {
description
"Augments the 'simple-next-hop' case in IP unicast routes.";
uses nhlfe-single-contents {
when "../../../mpls:mpls-enabled = 'true'";
}
}

Notes
-----
The original YANG statements make the "uses" statement apply to all rt:rib and all rt:route instances as soon as there is at least one instance that has mpls:mpls-enabled set to true. I suspect this is not the author's intent.

The corrected YANG statements make the "uses" statement only apply to the specific route instances that have mpls:mpls-enabled set to true. There are also other ways to fix this issue.

Instructions:
-------------
This erratum is currently posted as "Reported". If necessary, please
use "Reply All" to discuss whether it should be verified or
rejected. When a decision is reached, the verifying party
can log in to change the status and edit the report, if necessary.

--------------------------------------
RFC8960 (draft-ietf-mpls-base-yang-17)
--------------------------------------
Title : A YANG Data Model for MPLS Base
Publication Date : December 2020
Author(s) : T. Saad, K. Raza, R. Gandhi, X. Liu, V. Beeram
Category : PROPOSED STANDARD
Source : Multiprotocol Label Switching
Area : Routing
Stream : IETF
Verifying Party : IESG

[mpls] [Technical Errata Reported] RFC8960 (7059) RFC Errata System
Re: [mpls] [Technical Errata Reported] RFC8960 (7… tom petch
Re: [mpls] [Technical Errata Reported] RFC8960 (7… tom petch
Re: [mpls] [Technical Errata Reported] RFC8960 (7… Jan Lindblad (jlindbla)
Re: [mpls] [Technical Errata Reported] RFC8960 (7… tom petch