Re: [netmod] js review of draft-ietf-netmod-schema-mount-09

2018-04-11 Thread Lou Berger
Hi, I have a suggested minor clarification below. -- On April 11, 2018 2:47:24 AM Martin Bjorklund wrote: Juergen Schoenwaelder wrote: On Tue, Apr 10, 2018 at 10:58:00PM +0200, Martin Bjorklund wrote: > Juergen

Re: [netmod] js review of draft-ietf-netmod-schema-mount-09

2018-04-11 Thread Martin Bjorklund
Juergen Schoenwaelder wrote: > On Tue, Apr 10, 2018 at 10:58:00PM +0200, Martin Bjorklund wrote: > > Juergen Schoenwaelder wrote: > > > On Tue, Apr 10, 2018 at 04:15:46PM +0200, Martin Bjorklund wrote: > > > > > > > > >

Re: [netmod] js review of draft-ietf-netmod-schema-mount-09

2018-04-11 Thread Juergen Schoenwaelder
On Tue, Apr 10, 2018 at 10:58:00PM +0200, Martin Bjorklund wrote: > Juergen Schoenwaelder wrote: > > On Tue, Apr 10, 2018 at 04:15:46PM +0200, Martin Bjorklund wrote: > > > > > > > > Yes, the checksum (which I think is actually a version identifier). > > > >

Re: [netmod] js review of draft-ietf-netmod-schema-mount-09

2018-04-10 Thread Martin Bjorklund
Juergen Schoenwaelder wrote: > On Tue, Apr 10, 2018 at 04:15:46PM +0200, Martin Bjorklund wrote: > > > > > > Yes, the checksum (which I think is actually a version identifier). > > > Anyway, the question is whether a client can draw any conclusions from > >

Re: [netmod] js review of draft-ietf-netmod-schema-mount-09

2018-04-10 Thread Juergen Schoenwaelder
On Tue, Apr 10, 2018 at 04:15:46PM +0200, Martin Bjorklund wrote: > > > > Yes, the checksum (which I think is actually a version identifier). > > Anyway, the question is whether a client can draw any conclusions from > > seeing YANG library instances with the same checksum and whether a > >

Re: [netmod] js review of draft-ietf-netmod-schema-mount-09

2018-04-10 Thread Martin Bjorklund
Juergen Schoenwaelder wrote: > On Fri, Apr 06, 2018 at 03:27:10PM +0200, Martin Bjorklund wrote: > > > > Ok. I tweaked it to: > > > > This document defines a mechanism to add the schema trees defined > > by a set of YANG modules onto a mount point

Re: [netmod] js review of draft-ietf-netmod-schema-mount-09

2018-04-10 Thread Juergen Schoenwaelder
On Fri, Apr 06, 2018 at 03:27:10PM +0200, Martin Bjorklund wrote: > > Ok. I tweaked it to: > > This document defines a mechanism to add the schema trees defined > by a set of YANG modules onto a mount point defined in the schema > tree in some YANG module. > Works for me. > > OK. It

Re: [netmod] js review of draft-ietf-netmod-schema-mount-09

2018-04-06 Thread Martin Bjorklund
Hi, Juergen Schoenwaelder wrote: > On Thu, Apr 05, 2018 at 02:33:40PM +0200, Martin Bjorklund wrote: > > Hi, > > > > Thank you for this review! Comments inline. > > > > Juergen Schoenwaelder wrote: > > > Here is my

Re: [netmod] js review of draft-ietf-netmod-schema-mount-09

2018-04-06 Thread Juergen Schoenwaelder
On Thu, Apr 05, 2018 at 02:33:40PM +0200, Martin Bjorklund wrote: > Hi, > > Thank you for this review! Comments inline. > > Juergen Schoenwaelder wrote: > > Here is my review of draft-ietf-netmod-schema-mount-09. > > > > * Abstract > > > >This

Re: [netmod] js review of draft-ietf-netmod-schema-mount-09

2018-04-05 Thread Martin Bjorklund
Hi, Thank you for this review! Comments inline. Juergen Schoenwaelder wrote: > Here is my review of draft-ietf-netmod-schema-mount-09. > > * Abstract > >This document defines a mechanism to combine YANG modules into the >schema defined in other

[netmod] js review of draft-ietf-netmod-schema-mount-09

2018-03-29 Thread Juergen Schoenwaelder
Here is my review of draft-ietf-netmod-schema-mount-09. * Abstract This document defines a mechanism to combine YANG modules into the schema defined in other YANG modules. I do not know what this says - I think this text is confusing. What does it mean to 'combine' YANG modules? What