-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Start of an OSPF YANG northbound/mgmtd conversion #18401
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Christian Hopps <[email protected]>
Signed-off-by: Christian Hopps <[email protected]>
c84e72c
to
b26f735
Compare
It is a very nice initiative @choppsv1 ! Could you add a documentation update too ? So its usage will spread quicker. |
Signed-off-by: Christian Hopps <[email protected]>
This simple OSPF test utilizes less "magic" testing infrastructure. This is useful when doing OSPF development as a quick way to enable a feature to help test or develop it. Signed-off-by: Christian Hopps <[email protected]>
b26f735
to
2656eb0
Compare
mgmtd covnersion: https://docs.frrouting.org/projects/dev-guide/en/latest/mgmtd-dev.html#converting-a-daemon-to-mgmtd The mgmtd conversion is pretty straight-forward and that is mostly done by the code that's present here already. The main work here is the more basic northbound conversion which boils down to:
The docs for this are nice in covering everything but may make the task seem harder than it actually is.. it's really pretty simple once you do one command the rest end up being pretty rote. northbound conversion: https://docs.frrouting.org/projects/dev-guide/en/latest/northbound/retrofitting-configuration-commands.html northbound architecture: https://docs.frrouting.org/projects/dev-guide/en/latest/northbound/architecture.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are some of the new ospf files auto-generated from the yang model(s)? If so, could you please separate the tool's output files from the ones you implement, instead of modifying them in-place? That makes it a lot easier to make incremental changes in the models and see those reflected in the tool output.
No description provided.