[netmod] Re: Yangdoctors last call review of draft-ietf-netmod-system-config-06

[[e-mail]]

Hi Qiufang,

This version looks good to me. Thanks. 

Cheers,
Med

> -----Message d'origine-----
> De : maqiufang (A) <[e-mail]>
> Envoyé : mardi 18 juin 2024 11:05
> À : BOUCADAIR Mohamed INNOV/NET <[e-mail]>;
> Michal Vaško <[e-mail]>; [e-mail]
> Cc : [e-mail]; last-
> [e-mail]; [e-mail]
> Objet : RE: [netmod] Yangdoctors last call review of draft-ietf-
> netmod-system-config-06
> 
> 
> Thanks a lot Med. I have submitted a new version to incorporate
> your fixes. You might want to review it at
> https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2
> Fauthor-tools.ietf.org%2Fiddiff%3Furl2%3Ddraft-ietf-netmod-
> system-config-
> 08&data=05%7C02%7Cmohamed.boucadair%40orange.com%7Ca351ee76402d46
> a5674708dc8f75ec6a%7C90c7a20af34b40bfbc48b9253b6f5d20%7C0%7C0%7C6
> 38542983866493391%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJ
> QIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=Yp
> 5MD%2Bg%2F1S4323gKKNTvzvy18l9LdtgLG3f59L%2F%2Fbg0%3D&reserved=0.
> 
> Best Regards,
> Qiufang
> 
> -----Original Message-----
> From: [e-mail]
> [mailto:[e-mail]]
> Sent: Tuesday, June 18, 2024 3:30 PM
> To: maqiufang (A) <[e-mail]>; Michal Vaško
> <[e-mail]>; [e-mail]
> Cc: [e-mail]; last-
> [e-mail]; [e-mail]
> Subject: RE: [netmod] Yangdoctors last call review of draft-ietf-
> netmod-system-config-06
> 
> Hi Qiufang,
> 
> Thanks for taking care of this.
> 
> I submitted right now a PR with some minor fixes (e.g., align
> with 8407bis reco):
> https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2
> Fgithub.com%2Fnetmod-wg%2Fsystem-
> config%2Fpull%2F38&data=05%7C02%7Cmohamed.boucadair%40orange.com%
> 7Ca351ee76402d46a5674708dc8f75ec6a%7C90c7a20af34b40bfbc48b9253b6f
> 5d20%7C0%7C0%7C638542983866502315%7CUnknown%7CTWFpbGZsb3d8eyJWIjo
> iMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7
> C%7C%7C&sdata=hTUH7sYC7r3om%2Fug7NAHC0dWwQFomD3jOBiBSpG8BOQ%3D&re
> served=0
> 
> Other than that, this looks good to me.
> 
> Cheers,
> Med
> 
> > -----Message d'origine-----
> > De : maqiufang (A) <[e-mail]> Envoyé : lundi 17
> juin 2024
> > 14:28 À : BOUCADAIR Mohamed INNOV/NET
> <[e-mail]>;
> > Michal Vaško <[e-mail]>; [e-mail] Cc :
> > [e-mail]; last-
> [e-mail];
> > [e-mail] Objet : RE: [netmod] Yangdoctors last call
> review of
> > draft-ietf-
> > netmod-system-config-06
> >
> >
> > Hi, Med and Michal,
> >
> > Thanks for your review, the authors have submitted a new
> version
> >
> (https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%
> > 2Fdatatracker.ietf.org%2Fdoc%2Fhtml%2Fdraft-ietf-netmod-system-
> > config-
> >
> 07&data=05%7C02%7Cmohamed.boucadair%40orange.com%7C1a59a213597040
> >
> 741c9708dc8ec8ee94%7C90c7a20af34b40bfbc48b9253b6f5d20%7C0%7C0%7C6
> >
> 38542240852698598%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJ
> >
> QIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=am
> > rvfTAx%2Fh2IvNtkbpBjGC8gInnlriPP9bGfYYACvKQ%3D&reserved=0)  to
> resolve
> > the nits you raised below, together with some other updates
> received
> > from the WG. Please review the diff
> >
> (https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%
> > 2Fauthor-tools.ietf.org%2Fiddiff%3Furl1%3Ddraft-ietf-netmod-
> > system-config-06%26url2%3Ddraft-ietf-netmod-system-config-
> > 07%26difftype%3D--
> >
> html&data=05%7C02%7Cmohamed.boucadair%40orange.com%7C1a59a2135970
> >
> 40741c9708dc8ec8ee94%7C90c7a20af34b40bfbc48b9253b6f5d20%7C0%7C0%7
> >
> C638542240852716505%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiL
> >
> CJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7C&sdata=
> > 8N0szV%2FeKhtq3zkood1OXjcfl1uwBL1wbdO1%2FVSAq1o%3D&reserved=0)
> > and let us know if you have further comments. Thanks a lot!
> >
> > Best Regards,
> > Qiufang
> >
> > -----Original Message-----
> > From: [e-mail]
> > [mailto:[e-mail]]
> > Sent: Thursday, June 13, 2024 7:25 PM
> > To: Michal Vaško <[e-mail]>; [e-mail]
> > Cc: [e-mail]; last-
> [e-mail];
> > [e-mail]
> > Subject: RE: [netmod] Yangdoctors last call review of draft-
> ietf-
> > netmod-system-config-06
> >
> > Hi all,
> >
> > On this one:
> >
> > > - all 'local-as' and 'peer-as' nodes are uint32, so in JSON
> > encoding
> > > numbers should be used instead of strings
> >
> > Even if this an example, the authors may consider using
> "inet:as-
> > number" rather than uint32.
> >
> > As I'm there,
> >
> > (1)
> >
> >  the name of this leaf is weird:
> >
> >        leaf name {
> >          type inet:ip-address;
> >        }
> >
> > I would change the name.
> >
> > (2) I would delete from the description of the ietf-system-
> datastore
> > module:
> >
> >         The key words 'MUST', 'MUST NOT', 'REQUIRED', 'SHALL',
> >         'SHALL NOT', 'SHOULD', 'SHOULD NOT', 'RECOMMENDED',
> >         'NOT RECOMMENDED', 'MAY', and 'OPTIONAL' in this
> document
> >         are to be interpreted as described in BCP 14 (RFC 2119)
> >         (RFC 8174) when, and only when, they appear in all
> >         capitals, as shown here.";
> >
> > (3) Please fix this part in the IANA cons:
> >
> > OLD:
> >       name: ietf-system-datastore
> >       prefix: sys
> >
> > NEW:
> >       name: ietf-system-datastore
> >       prefix: sysds
> >
> > (4) Security cons: Please use the sec template in both
> subsections
> >
> > For example,
> >
> > CURRENT:
> >    The Network Configuration Access Control Model (NACM)
> [RFC8341]
> >    provides the means to restrict access for particular NETCONF
> users
> > to
> >    a preconfigured subset of all available NETCONF protocol
> operations
> >    and content.
> >
> > Does not mention RESTCONF.
> >
> > I think other para of the template should make it to these
> sections.
> >
> > Also, you may start each subsection by indicating the name of
> the
> > module instead of "The YANG module defined in this document"
> > because two modules are defined in the doc.
> >
> > Hope this helps.
> >
> > Cheers,
> > Med
> >
> > > -----Message d'origine-----
> > > De : Michal Vaško via Datatracker <[e-mail]> Envoyé :
> > jeudi 13
> > > juin 2024 12:25 À : [e-mail] Cc :
> > > [e-mail]; last-
> > [e-mail];
> > > [e-mail] Objet : [netmod] Yangdoctors last call review
> > of
> > > draft-ietf-
> > > netmod-system-config-06
> > >
> > >
> > > Reviewer: Michal Vaško
> > > Review result: Ready with Nits
> > >
> > > This is my yang-doctor review of draft-ietf-netmod-system-
> > config,
> > > which includes 2 small YANG modules, in addition to a few
> > example
> > > modules.
> > >
> > > ietf-system-datastore:
> > > - small module with a single identity, no issues
> > >
> > > ietf-netconf-resolve-system:
> > > - module with similar simple augments to standard ietf-
> netconf
> > and
> > > ietf-netconf-nmda modules, no issues
> > >
> > > As for the example YANG modules and data, there are a few
> nits:
> > >
> > > example-acl:
> > > - leaf-list application - path is not indented
> > >
> > > Section 8.2 BGP examples:
> > > - 'inet:port' type does not exist in the latest ietf-inet-
> types
> > > (2013) YANG module, only 'port-number' - all 'local-as' and
> > 'peer-as'
> > > nodes are uint32, so in JSON encoding numbers should be used
> > instead
> > > of strings - 'local-port' is using uint16 type so in JSON
> > encoding
> > > numbers should be used instead of strings
> > >
> > > Finally, the examples and their data are using YANG snippets
> > and data
> > > without namespaces or module names, which may be fine for
> > illustration
> > > purposes but possibly confusing.
> > >
> > >
> > > _______________________________________________
> > > netmod mailing list -- [e-mail] To unsubscribe send an
> > email to
> > > [e-mail]
> >
> _________________________________________________________________
> > ___________________________________________
> > Ce message et ses pieces jointes peuvent contenir des
> informations
> > confidentielles ou privilegiees et ne doivent donc pas etre
> diffuses,
> > exploites ou copies sans autorisation. Si vous avez recu ce
> message
> > par erreur, veuillez le signaler a l'expediteur et le detruire
> ainsi
> > que les pieces jointes. Les messages electroniques etant
> susceptibles
> > d'alteration, Orange decline toute responsabilite si ce message
> a ete
> > altere, deforme ou falsifie. Merci.
> >
> > This message and its attachments may contain confidential or
> > privileged information that may be protected by law; they
> should not
> > be distributed, used or copied without authorisation.
> > If you have received this email in error, please notify the
> sender and
> > delete this message and its attachments.
> > As emails may be altered, Orange is not liable for messages
> that have
> > been modified, changed or falsified.
> > Thank you.
> _________________________________________________________________
> ___________________________________________
> Ce message et ses pieces jointes peuvent contenir des
> informations confidentielles ou privilegiees et ne doivent donc
> pas etre diffuses, exploites ou copies sans autorisation. Si vous
> avez recu ce message par erreur, veuillez le signaler a
> l'expediteur et le detruire ainsi que les pieces jointes. Les
> messages electroniques etant susceptibles d'alteration, Orange
> decline toute responsabilite si ce message a ete altere, deforme
> ou falsifie. Merci.
> 
> This message and its attachments may contain confidential or
> privileged information that may be protected by law; they should
> not be distributed, used or copied without authorisation.
> If you have received this email in error, please notify the
> sender and delete this message and its attachments.
> As emails may be altered, Orange is not liable for messages that
> have been modified, changed or falsified.
> Thank you.
____________________________________________________________________________________________________________
Ce message et ses pieces jointes peuvent contenir des informations confidentielles ou privilegiees et ne doivent donc
pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce message par erreur, veuillez le signaler
a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration,
Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci.

This message and its attachments may contain confidential or privileged information that may be protected by law;
they should not be distributed, used or copied without authorisation.
If you have received this email in error, please notify the sender and delete this message and its attachments.
As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified.
Thank you.