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://github.com/netmod-wg/system-config/pull/38 Other than that, this looks good to me. Cheers, Med > -----Message d'origine----- > De : maqiufang (A) <maqiufang1@xxxxxxxxxx> > Envoyé : lundi 17 juin 2024 14:28 > À : BOUCADAIR Mohamed INNOV/NET <mohamed.boucadair@xxxxxxxxxx>; > Michal Vaško <mvasko@xxxxxxxxx>; yang-doctors@xxxxxxxx > Cc : draft-ietf-netmod-system-config.all@xxxxxxxx; last- > call@xxxxxxxx; netmod@xxxxxxxx > 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: mohamed.boucadair@xxxxxxxxxx > [mailto:mohamed.boucadair@xxxxxxxxxx] > Sent: Thursday, June 13, 2024 7:25 PM > To: Michal Vaško <mvasko@xxxxxxxxx>; yang-doctors@xxxxxxxx > Cc: draft-ietf-netmod-system-config.all@xxxxxxxx; last- > call@xxxxxxxx; netmod@xxxxxxxx > 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 <noreply@xxxxxxxx> Envoyé : > jeudi 13 > > juin 2024 12:25 À : yang-doctors@xxxxxxxx Cc : > > draft-ietf-netmod-system-config.all@xxxxxxxx; last- > call@xxxxxxxx; > > netmod@xxxxxxxx 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 -- netmod@xxxxxxxx To unsubscribe send an > email to > > netmod-leave@xxxxxxxx > _________________________________________________________________ > ___________________________________________ > 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. -- last-call mailing list -- last-call@xxxxxxxx To unsubscribe send an email to last-call-leave@xxxxxxxx
