Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix issues from Tom Petch review in Opsawg mailing list #112

Closed
oscargdd opened this issue May 21, 2020 · 5 comments
Closed

Fix issues from Tom Petch review in Opsawg mailing list #112

oscargdd opened this issue May 21, 2020 · 5 comments

Comments

@oscargdd
Copy link
Collaborator

Original message:
https://mailarchive.ietf.org/arch/msg/opsawg/Mh-BCY9PZuWGE1QKzBD1rll6_AY/

Looking at the YANG module it would appear that a lot of prior knowledge is needed in order to understand it, such as QinQ, dot1Q, GRE, LDP, VRRP, etc etc ie they are what I think of as Normative References and yet I cannot recall seeing a YANG module with so few reference clauses; and the very few reference clauses that are there refer to RFC that do not appear in the I-D References.
I also note that the IANA Considerations registers a different prefix to that in the YANG module and it is unclear /UNUSED/ is about. And examples in the body of the I-D rather than as an Informative appendix mmmm. I am unenthusiastic about looking any deeper.
Tom Petch

@sbarguil sbarguil assigned sbarguil and unassigned sbarguil May 21, 2020
@oscargdd
Copy link
Collaborator Author

Qin takes care of the references. Do the work after the Split of the module.
Oscar to take care of the editorial issue in IANA Section.

@oscargdd
Copy link
Collaborator Author

Daniel King takes the issue of looking at the best aproach for the examples

@boucadair
Copy link
Collaborator

Daniel King takes the issue of looking at the best aproach for the examples

I would move this section to an appendix.

@boucadair
Copy link
Collaborator

@oscargdd: I would close this one as the only remaining point to check is references. We do already have an issue to track that.

@oscargdd
Copy link
Collaborator Author

oscargdd commented Jul 2, 2020

all the points expect the references are solved. Issue #115 deals with the references

@oscargdd oscargdd closed this as completed Jul 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants