Re: [netmod] [Technical Errata Reported] RFC7950 (5157)

[Andy Bierman <[e-mail]>]

On Mon, Oct 23, 2017 at 7:30 AM, Martin Bjorklund <[e-mail]> wrote:

> Andy Bierman <[e-mail]> wrote:
> > On Mon, Oct 23, 2017 at 1:35 AM, Martin Bjorklund <[e-mail]>
> wrote:
> >
> > > Benoit Claise <[e-mail]> wrote:
> > > > Dear all,
> > > >
> > > > Shall I validate this one?
> > >
> > > No I don't think this is correct.  We choose a quoted string to be
> > > able to correctly represent all YANG integer and decimal64 values.
> > > Note that XPath doesn't have integers, just 64-bit floats.  This means
> > > that not all 64-bit integers can be expressed as numbers in XPath.
> > >
> > >
> > It's OK to leave this out since it requires a code change.
> >
> >
> > Read sec. 2.3 again:
>
> Wrong thread?
>

No -- customers who know XPath report this YANG-ism as a bug.
It would require 2 lines of code to be changed in our code to accept
numbers.
I don't think an errata should require a code change.


>
> > The node test text() is true for any text node.
>
> Sure, but a node test is not a predicate!
>

But it returns true/false, not a node-set.
The expression /foo[text() = 'fred'] is not the same as /foo[. = 'fred']



>
>
> /martin
>
>
Andy


>
>
> >
> >
> >
> >
> > >
> > > /martin
> > >
> > >
> >
> > Andy
> >
> >
> > >
> > > >
> > > > Regards, Benoit
> > > > > The following errata report has been submitted for RFC7950,
> > > > > "The YANG 1.1 Data Modeling Language".
> > > > >
> > > > > --------------------------------------
> > > > > You may review the report below and at:
> > > > > http://www.rfc-editor.org/errata/eid5157
> > > > >
> > > > > --------------------------------------
> > > > > Type: Technical
> > > > > Reported by: Andy Bierman <[e-mail]>
> > > > >
> > > > > Section: 14
> > > > >
> > > > > Original Text
> > > > > -------------
> > > > >    key-predicate-expr  = node-identifier *WSP "=" *WSP
> quoted-string
> > > > >
> > > > > Corrected Text
> > > > > --------------
> > > > >    key-predicate-expr  = node-identifier *WSP "=" *WSP
> > > > >          (quoted-string / integer-value / decimal-value)
> > > > >
> > > > > Notes
> > > > > -----
> > > > > An instance identifier is forced to specify every key value to be a
> > > > > string
> > > > > even though the YANG key leaf type could be a numeric type.
> > > > > XPath does not require a quoted string here, just YANG.
> > > > >
> > > > > Old:  /top/list[idx="4"]
> > > > > New: /top/list[idx=4]
> > > > >
> > > > > 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.
> > > > >
> > > > > --------------------------------------
> > > > > RFC7950 (draft-ietf-netmod-rfc6020bis-14)
> > > > > --------------------------------------
> > > > > Title               : The YANG 1.1 Data Modeling Language
> > > > > Publication Date    : August 2016
> > > > > Author(s)           : M. Bjorklund, Ed.
> > > > > Category            : PROPOSED STANDARD
> > > > > Source              : NETCONF Data Modeling Language
> > > > > Area                : Operations and Management
> > > > > Stream              : IETF
> > > > > Verifying Party     : IESG
> > > > > .
> > > > >
> > > >
> > >
>